-
Notifications
You must be signed in to change notification settings - Fork 12
Correct mobile vs desktop inconsistency around showing the layer picker button when already in layer picker #2101
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
base: master
Are you sure you want to change the base?
Conversation
Deployed to Cloudflare Pages
|
526dbe0 to
e0d0229
Compare
| </TabletBackButton> | ||
| )} | ||
| </div> | ||
| <MobileNetworkButton |
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.
I expect this was intentional design for visual balance
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.
Maybe so. I spelled out my rationale in the PR description, and asked @juresobocan for input.
e0d0229 to
6052003
Compare
lubej
left a comment
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.
I would rather see, that we keep it.
|
I'm not convinced this is an improvement tbh. I agree that we are missing consistency with desktop, but if anything, we should go the other way around - showing the badge on desktop when the menu is active as well. My argument would be that we should provide the user with clarity on which network they are on at all times. It's part of the usability heuristics of Nielsen and Norman - basically a fundamental requirement in UX. I do not believe it hurts the user to show what network they are currently on. |
OK, let's do that! |
Both in desktop and in mobile mode, when using the app, we have a button at the top, for launching the layer picker dialog.
Clicking this button brings up this dialog.
For some reason, in the mobile view, even when this dialog was up, the button (for launching the dialog) was still visible.
I think this serves no useful purpose.
Therefore I propose removing it.