-
Notifications
You must be signed in to change notification settings - Fork 512
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 dropdowns from Mimir Admin's landing #1549
Conversation
I didn't like the dropdowns, and Peter gave the same feedback. The issue about dropdowns is that you have to click stuff to find information that should be in the first level. This is also more accessible. Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
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.
To add some separation without too much added vertical space (it seems that padding inside boxes plus margins between boxes are higher than the text on the page), I would try simple horizontal lines between sections.
But this is still an improvement over hiding links by default, so 👍
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
83ec229
to
11722a3
Compare
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Love it, thank you very much! |
What this PR does
I didn't like the dropdowns, and @pstibrany gave the same feedback. The issue about dropdowns is that you have to click stuff to find information that should be in the first level. This is also more accessible.
This is how it looks like now:
And it gracefully scales to mobile:
Which issue(s) this PR fixes or relates to
None
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]