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

Multi-column Layout #417

Closed
bcorcoran opened this issue Dec 20, 2020 · 4 comments
Closed

Multi-column Layout #417

bcorcoran opened this issue Dec 20, 2020 · 4 comments
Labels
bug Something isn't working

Comments

@bcorcoran
Copy link

What component (if applicable)

Describe the bug
Additional column is only visible on xl viewport.

Expected behavior
I was not expecting the additional column to be completely hidden on anything below XL viewport. Why is it this way? From a usability perspective, it makes no sense... what if there's important content there? I was kind of expecting it to behave like a second sidebar so that you could expand it on smaller viewports. Even stacked would be OK. Not sure what the ultimate solution is but right now it feels incomplete.

@bcorcoran bcorcoran added the bug Something isn't working label Dec 20, 2020
@EdorianDark
Copy link

It was very confusing to open it and see only one column.
The icon is also misleading.

@xt0rted
Copy link

xt0rted commented Dec 23, 2020

This layout mimics the GitHub dashboard, but I agree I didn't realize there was a 3rd column until I came back to the site later and my browser was wider.

@davidluhr
Copy link
Contributor

After talking with the team about this pattern, this is working as intended. It seems counter-intuitive at first, but this is how several major sites handle responsive content with a secondary column.

As mentioned, this matches GitHub, as well as Twitter, Reddit, and others. This layout is used in the new Detail Screens examples: https://tailwindui.com/components/application-ui/page-examples/detail-screens

In these examples, the secondary content is either not shown on smaller screens, or presented as part of the main content for smaller breakpoints. I will make note that it may be useful to have multi-column layouts that retain a secondary column in some fashion across viewport sizes.

@artworkad
Copy link

@davidluhr I understand that one can hide secondary content for smaller breakpoints. But when the column is part of a tiered navigation, there has to be a responsive version for that navigation. Right now there is none as you can see from the pictures:

multi-column-layout-desktop

multi-column-layout

Is there a pattern for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants