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

lost-flex-coontainer needs nowrap option #390

Open
isGabe opened this issue Jul 11, 2017 · 2 comments
Open

lost-flex-coontainer needs nowrap option #390

isGabe opened this issue Jul 11, 2017 · 2 comments

Comments

@isGabe
Copy link

isGabe commented Jul 11, 2017

Is this a feature request or a bug report?
Feature request

What is the current behavior?
lost-flex-container: row outputs:

display: flex;
flex-flow: row wrap;

What is the desired behavior?
It would be nice to be able to pass a nowrap parameter in, like:
lost-flex-container: row nowrap;, instead of having to manually override it.

What's the motivation or use-case behind changing the behavior?
In order to get nowrap you have to override with flex-wrap: nowrap, which is easy enough. But if you have stylelint set up to order selectors alphabetically (team decision, not mine), it will bark at you, wanting the flex-wrap: nowrap declaration to come first, which will then be overridden by the lost-flex-column declaration.

What version of LostGrid, browser and browser version are affected by this issue? Did this happen in previous versions?
Lost: 7.1.1
browser is not relevant in this case...

@isGabe
Copy link
Author

isGabe commented Jul 11, 2017

This may have been a mistake on my part, feel free to close if it doesn't make sense.

@peterramsing
Copy link
Owner

@fStop, is this still an issue. Adding more params is something that I'd like to avoid but I see the potential need here. Let me know. Thank!

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

No branches or pull requests

2 participants