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

Fix mobile display for multiple categories #36

Open
ArtOfCode- opened this issue May 3, 2020 · 3 comments
Open

Fix mobile display for multiple categories #36

ArtOfCode- opened this issue May 3, 2020 · 3 comments
Labels
area: components Changes to design components type: bug Something isn't working

Comments

@ArtOfCode-
Copy link
Member

When you have multiple (more than 3) categories, the category header works fine on desktop:

categories

When you use it on mobile, it completely screws the layout:

categories-mob

That should probably wrap onto the next line. We can look at collapsing multiple categories into a "see more" link in QPixel too, but Co-Design should still avoid breaking on this case.

@cellio
Copy link
Member

cellio commented May 3, 2020

If the length of the category list exceeds the available space, I think we want to show the first however many fit and then have a "more" affordance. Whether that's something with a dropdown or a link to the categories list or something else, I don't know (want input from better UX folks than I).

@ArtOfCode-
Copy link
Member Author

Alternative that I've just stumbled on by complete accident: simply hide the overflow and make the bar scroll horizontally. Looks terrible on a desktop, but won't happen on a desktop (unless a site has a ridiculous number of categories); looks just fine on a phone:

categories-scroll

For added bonus, the half-hidden name is exactly the affordance it needs to indicate that it's scrollable.

@cellio
Copy link
Member

cellio commented May 3, 2020

Horizontal scrolling seems fine for mobile, though I think conventionally there would be '<' and '>' indicators instead of relying on the luck of truncation?

Ultimately, though, we're going to have to deal with possible category overflow on non-touch interfaces too, and when we do we might end up using the same solution on both. I really want @mattjbrent to weigh in on that.

@cellio cellio added area: components Changes to design components type: bug Something isn't working labels Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: components Changes to design components type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants