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

Flexbox enable list-group #18981

Merged
merged 2 commits into from
Oct 30, 2016
Merged

Flexbox enable list-group #18981

merged 2 commits into from
Oct 30, 2016

Conversation

rosskevin
Copy link
Contributor

Resolves #18978

Requires the pull-right and pull-left flex change from PR #18976

/cc @cvrebert

@rosskevin rosskevin changed the title v4 Flexbox list-group implementation v4 Flex enable list-group Jan 20, 2016
Cras justo odio
<span class="label label-default label-pill pull-right">14</span>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's no pull-right class anymore. Stick with pull-xs-right
Also, won't putting the floats last instead of first cause some breakage when flexbox is disabled?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. I'm confused about pull-right/left, because it is still there in on v4-dev tip and I'm not seeing the responsive classes. I can change that back.
  2. It does not appear to have any problems floating in my test (granted it may be limited testing considering I didn't go after a bunch of browsers). The fact is that pull-right appears to work this way in the docs for both flex and normal layouts.

I can change it back, but then we may have questions about a flexed pull-right not working too...

I'll change the pull-*-right back and commit.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(1) There are still pull-left/right mixins, but the pull-left/right classes are gone as of #18340. I just double-checked.
(2) I'm not 100% sure either. Will ask mdo when he reviews this.

@cvrebert cvrebert changed the title v4 Flex enable list-group Flexbox enable list-group Jan 20, 2016
@FezVrasta
Copy link
Contributor

Any news on this?

@mdo mdo added this to the v4.0.0-alpha.6 milestone Oct 28, 2016
@mdo mdo merged commit 4b3c9e4 into twbs:v4-dev Oct 30, 2016
@mdo
Copy link
Member

mdo commented Oct 30, 2016

Merged, but made some changes to this with 7ba4ac0.

Rather than change how our default examples are structured, I've added guidance on how to move from the default mode to the flexbox mode for the list group.

@mdo mdo mentioned this pull request Oct 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants