Skip to content
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

Admin UI : header buttons #8034

Closed
Skrypt opened this issue Dec 16, 2020 · 9 comments · Fixed by #8083
Closed

Admin UI : header buttons #8034

Skrypt opened this issue Dec 16, 2020 · 9 comments · Fixed by #8083

Comments

@Skrypt
Copy link
Contributor

Skrypt commented Dec 16, 2020

Before #7851 :

image

After #7851 :

image

I found out that we are displaying these buttons inside a hamburger menu now.
I'm not a fan of it. They should be displayed all the time.
If we have other menu items then they should be added in the hamburger menu but only if they have a text.
This looks strange needing to open the hamburger menu to toggle the darkmode.

image

/cc @agriffard

@hishamco
Copy link
Member

Seems the icons are wiered in the humberger menu, could we use text instead

@Skrypt
Copy link
Contributor Author

Skrypt commented Dec 17, 2020

No it needs to be removed from that hamburger menu. Make it how it was before.

@hishamco
Copy link
Member

May be the first screenshot is accepted but the other once introduce a spacing issue which is not needed

@Skrypt
Copy link
Contributor Author

Skrypt commented Dec 17, 2020

Well the hamburger menu is not necessary at all I think. We should keep it if people want to extend the top menu with their own menu items (text links). Else I don't think we need it.

@agriffard
Copy link
Member

What about only show text on small?:
media

agriffard added a commit that referenced this issue Dec 17, 2020
agriffard added a commit that referenced this issue Dec 17, 2020
@Skrypt
Copy link
Contributor Author

Skrypt commented Dec 18, 2020

I'm sorry but I have to say that I don't like it. It doesn't look right.
One of the reasons is that the user menu should not open a second modal like this inside the hamburger menu.
Also, the darkmode and frontend website buttons should be accessible directly in the top bar like the notification button is for Github. Also, notice that the page title is never appended in the top navbar either.

image

Maybe you will say that I'm picky but I can't help but noticing these small details...

@Skrypt
Copy link
Contributor Author

Skrypt commented Dec 18, 2020

Frontend hamburger menu :

image

Backend hamburger menu :

image

I think we should follow the second example.

@hishamco
Copy link
Member

That's why I vote for the text in the menu items at the beginning ;)

@sebastienros
Copy link
Member

We can't have this hamburger menu in mobile view. This is a regression, and we closed a similar PR some time ago for the same reason. Sorry I didn't think about asking about the mobile view during the meeting. We need more time to evaluate all the changes before taking it.

Better create multiple PRs too for instance for the content items list so we can give feedback independently, and not merge all or nothing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants