-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Overhaul of UI for unprivileged users #11908
Overhaul of UI for unprivileged users #11908
Conversation
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
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.
This looks excellent! I think the overall visual style is better - long sections that are broken out in a big page-scrolly thing is generally going to obscure information more than having tabs with nice badges on them. I threw a few tiny little nits about some minor stylistic issues, but regardless, this is great and I'd love to see it.
I wonder if a weird side-effect is that we might see some of our administrators not wanting to prevent logins to view their own profile quite so much? That'd be nice!
Anywhoo - this looks awesome and I approve!
Signed-off-by: snipe <snipe@snipe.net>
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.
This looks so CRISP now. Double-approve!
This continues our process of trying to compact some of these UI sections, while still being limited enough to not expose functionality or elements that would link to things the user viewing their profile doesn't have access to do (so no sad pandas.)
Before
After