Skip to content

[4.2] [Smart Search] Moved statistics in the toolbar#38371

Merged
fancyFranci merged 5 commits intojoomla:4.2-devfrom
Kostelano:4-2-smart-stats
Sep 6, 2022
Merged

[4.2] [Smart Search] Moved statistics in the toolbar#38371
fancyFranci merged 5 commits intojoomla:4.2-devfrom
Kostelano:4-2-smart-stats

Conversation

@Kostelano
Copy link
Copy Markdown
Contributor

Pull Request for Issue # .

Summary of Changes

The statistics button is right in the middle between the actions, which I found a little illogical. I suggest moving it to the very end of the toolbar.

Screenshot_1

Testing Instructions

Pay attention to the screenshot, after applying the PR, check pages Smart Search: Indexed Content, Smart Search: Content Maps, Smart Search: Search Filters.

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Aug 2, 2022

I have tested this item ✅ successfully on af8b36f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38371.

1 similar comment
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 3, 2022

I have tested this item ✅ successfully on af8b36f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38371.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 3, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38371.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 3, 2022
@fancyFranci fancyFranci merged commit d93cc6b into joomla:4.2-dev Sep 6, 2022
@fancyFranci fancyFranci added this to the Joomla! 4.2.3 milestone Sep 6, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 6, 2022
@fancyFranci
Copy link
Copy Markdown
Contributor

Thats really better, thank you!

@Kostelano Kostelano deleted the 4-2-smart-stats branch September 6, 2022 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants