-
Notifications
You must be signed in to change notification settings - Fork 843
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
Provide some default design for expanding table rows #587
Comments
I agree with all points except for the last. I do think we should continue the expansion concept into mobile. I can't imagine how much extra content could be spilt in that area and if we expose it all, it will become a HUGE table. I've seen it work well enough in mobile. |
@cchaos sounds good to me. |
@AdnanBoota It's using a simple |
@snide Thanks a lot, it worked 👍 |
@snide Do you think this still needs to be addressed. Consumers can add any content they want in the expanded portion so I'm not entirely sure what we can do from the EUI side. |
I think it's closable. Originally we wanted to have some pattern work here, but at this point we've seen these views used so differently it's best to keep them as a generic "throw stuff in here" option. |
We should be able to clean up @cjcenizal's new expanding row tables with some nice defaults. Quick read is we should...
We'll also need to factor this into @cchaos' responsive table work. Gut there is we just expose the values since we're now working in more vertical space.
The text was updated successfully, but these errors were encountered: