-
-
Notifications
You must be signed in to change notification settings - Fork 136
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
Remove overhang logo and user when viewing media #1514
Conversation
It looks like the image quality went down a bit when I cropped these in paint 😢 We should recreate the images from the SVG sources. |
this is also more in line with web, which is nice. |
a33741e
to
b41e08f
Compare
This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity. |
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.
Functionality looks good now. 1 small code style change.
Why remove the logo?
Branding is good but too much branding is bad. At the very minimum, the user will still see the logo on the home screen of the app as well as the server and user select screens. This just removes the logo from screens that display media (the home screen shows media too but is more of a dashboard) to allow the focus to be on the media itself. The user knows what app they are using at this point.
Why remove the user?
As mentioned above, the home screen is basically the dashboard for the app and this is the one screen where the currently logged in user is shown. If the user ever isn't sure who is the logged in user, then can navigate to the home screen. This also allows more focus to be on the media itself while viewing it.
Before:
After:
Changes