New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: adds a refresh button #19261
feat: adds a refresh button #19261
Conversation
this commit introduces a refresh button to every page in the CoreHome module. When the user clicks the button, the data of the current page is refreshed without reloading the page. Refs matomo-org#11047
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we use reload icon, I believe other places are using that one, a couple of test screenshots needs to be updated which I can do, plugins/CoreHome/lang/en.json should be reverted, it does translate automatically by the system but other than that looks good.
@justinvelluppillai I feels like this one is good to merge, I will fix the UI tests and lang in another PR. |
Sounds good @peterhashair - you can go ahead and do that. It might be better to create a branch off this branch and fix the UI tests and remove the de translation before merging if you know how. |
@justinvelluppillai sounds good. |
* feat: adds a refresh button this commit introduces a refresh button to every page in the CoreHome module. When the user clicks the button, the data of the current page is refreshed without reloading the page. Refs #11047 * Update _headerMessage.twig update icon and some css * revert de lang revert de lang * update screenshots update screenshots * update screenshots update screenshots Co-authored-by: Andre <aeckardt@outlook.com>
Description:
this commit introduces a refresh button to every page in the CoreHome module. When the user clicks the button, the data of the current page is refreshed without reloading the page.
I've added a shortcut. So if this PR makes it into production, i think the documentation needs to be upgraded at https://matomo.org/faq/how-to/faq_24340/
Refs #11047
Screencast_from_05-24-2022_09.01.27_PM.mp4
Review