-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Update shown logo in the app #14860
Update shown logo in the app #14860
Conversation
@Findus23 or @diosmosis do you maybe have an idea why the icons would slightly change when nothing else changes and the PR shouldn't change it? There seem to be minor changes will icons in the all the screenshots https://builds-artifacts.matomo.org/matomo-org/matomo/3.x-dev/35792/UIIntegrationTest_dashboard1.png |
Before I update all screenshots and they become outdated again etc it be good to review this issue. Looking at https://builds-artifacts.matomo.org/matomo-org/matomo/14795/35795/ |
The icons look the same to my eye, maybe a change here subtly changes how chrome renders the icons. The size is the same and it basically looks the same, so I suppose it's ok to just copy over the screenshots? |
Updated the screenshots @diosmosis |
fix #14795