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.

Sign up for GitHub

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

Jump to bottom

[Grid] Column width doesn't behave predictably when using a base width combined with responsive widths #791

Closed
noinkling opened this issue Jun 2, 2019 · 2 comments
Closed

[Grid] Column width doesn't behave predictably when using a base width combined with responsive widths #791

noinkling opened this issue Jun 2, 2019 · 2 comments
Labels
lang/css Anything involving CSS state/wont-fix Anything which isn't going to be fixed type/usage Any support issues asking for help

Comments

@noinkling
Copy link
Contributor

noinkling commented Jun 2, 2019 •
edited
Loading

Bug Report

Essentially the same report as made here: Semantic-Org/Semantic-UI-React#2869

Read my comment in that issue for why I think it should be considered a bug.

Version

2.7.5
@noinkling
Copy link
Contributor Author

noinkling commented Jun 2, 2019 •
edited
Loading

As for suggestions how you might fix it I'm not too sure. Link to the relevant stylesheet.

You could require the base width to come directly before column and then change the attribute selectors to read [class*="one wide column"], etc. But that introduces order-dependent behaviour which I believe kinda goes against the SUI philosophy.

Otherwise maybe you could put the base width declarations below the media query ones and remove !important - but it's probably there for a reason, right? Never mind, that wouldn't help.

@lubber-de lubber-de added state/awaiting-investigation Anything which needs more investigation lang/css Anything involving CSS labels Jun 3, 2019
@lubber-de
Copy link
Member

lubber-de commented Feb 24, 2023 •
edited
Loading

I dont see the need for a change, at least in the current codebase. It is working as expected
An "eight wide two wide mobile six wide tablet column" will display

@lubber-de lubber-de added state/wont-fix Anything which isn't going to be fixed type/usage Any support issues asking for help and removed state/awaiting-investigation Anything which needs more investigation labels Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lang/css Anything involving CSS state/wont-fix Anything which isn't going to be fixed type/usage Any support issues asking for help
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@noinkling @lubber-de