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

Make menus scrollable #882

Closed
KamasamaK opened this Issue Jul 6, 2018 · 10 comments

Comments

Projects
None yet
3 participants
@KamasamaK
Contributor

KamasamaK commented Jul 6, 2018

Some of the top-level menus, like "Other", are quite long and can get cut off.

@shaedrich

This comment has been minimized.

Show comment
Hide comment
@shaedrich

shaedrich Jul 7, 2018

Contributor

I suggested to group the guides into sub menus

Contributor

shaedrich commented Jul 7, 2018

I suggested to group the guides into sub menus

@shaedrich

This comment has been minimized.

Show comment
Hide comment
@shaedrich

shaedrich Oct 12, 2018

Contributor

@KamasamaK I don't know, what @pfreitag and you think about that…

Contributor

shaedrich commented Oct 12, 2018

@KamasamaK I don't know, what @pfreitag and you think about that…

@pfreitag

This comment has been minimized.

Show comment
Hide comment
@pfreitag

pfreitag Oct 12, 2018

Member

Sounds ok to me.

Member

pfreitag commented Oct 12, 2018

Sounds ok to me.

@shaedrich

This comment has been minimized.

Show comment
Hide comment
@shaedrich

shaedrich Oct 12, 2018

Contributor

scrolling our sub menus?

Contributor

shaedrich commented Oct 12, 2018

scrolling our sub menus?

@pfreitag

This comment has been minimized.

Show comment
Hide comment
@pfreitag

pfreitag Oct 12, 2018

Member

sub menus would probably be better than scrolling I think.

Member

pfreitag commented Oct 12, 2018

sub menus would probably be better than scrolling I think.

@pfreitag

This comment has been minimized.

Show comment
Hide comment
@pfreitag

pfreitag Oct 12, 2018

Member

did you already put in a PR to make scrollable? That can work - it's not a huge concern for me.

Member

pfreitag commented Oct 12, 2018

did you already put in a PR to make scrollable? That can work - it's not a huge concern for me.

@shaedrich

This comment has been minimized.

Show comment
Hide comment
@shaedrich

shaedrich Oct 12, 2018

Contributor

No worries, I can close that pr and make a new one.

Contributor

shaedrich commented Oct 12, 2018

No worries, I can close that pr and make a new one.

@pfreitag

This comment has been minimized.

Show comment
Hide comment
@pfreitag

pfreitag Oct 12, 2018

Member

Whatever you think is fine with me, I'm ok with scrolling if it is done and looks good to you.

Member

pfreitag commented Oct 12, 2018

Whatever you think is fine with me, I'm ok with scrolling if it is done and looks good to you.

@shaedrich

This comment has been minimized.

Show comment
Hide comment
@shaedrich

shaedrich Oct 12, 2018

Contributor

Hmmm… I could test how sub menus would look like when I'm at home this evening. I guess, it's best if I post two screenshots so we can decide which one actually looks better.

Contributor

shaedrich commented Oct 12, 2018

Hmmm… I could test how sub menus would look like when I'm at home this evening. I guess, it's best if I post two screenshots so we can decide which one actually looks better.

@KamasamaK

This comment has been minimized.

Show comment
Hide comment
@KamasamaK

KamasamaK Oct 12, 2018

Contributor

Submenus sound fine too if you can find a good organization/grouping scheme.

Contributor

KamasamaK commented Oct 12, 2018

Submenus sound fine too if you can find a good organization/grouping scheme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment