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

Allow to use Tab.js with list-group #21756

Merged
merged 2 commits into from
Mar 18, 2017
Merged

Allow to use Tab.js with list-group #21756

merged 2 commits into from
Mar 18, 2017

Conversation

Johann-S
Copy link
Member

Allow Tab.js selector to accept list-group

Ref : #20620

@mdo mdo added this to the v4.0.0-beta milestone Jan 21, 2017
@mdo
Copy link
Member

mdo commented Jan 21, 2017

Hell yeah, thanks!

@Johann-S
Copy link
Member Author

You're welcome 😀👌

@pvdlg
Copy link
Contributor

pvdlg commented Jan 22, 2017

Is there really a point in supporting list-group with tabs ? As there is already vertical navs.

As far as I can tell both achieve the same thing.

The original issue #20620 mention "an example from v3". I think it's intended in v4 to offer vertical nav for this case while in v3 list-group were the way to go.

@Johann-S
Copy link
Member Author

It's the result of this discussion #20620, because we think (@mdo and myself) that it's better to allow Tab.js to work with every lists provided by BS

@Johann-S
Copy link
Member Author

Due to this comment : #21755 (comment)

Allow to use this structur of .list-group :

<div class="list-group">
  <a href="#home" class="list-group-item active" data-toggle="tab">Home</a>
  <a href="#profil" class="list-group-item list-group-item-action" data-toggle="tab">Profil</a>
</div>

@mdo mdo merged commit 1a46d8c into twbs:v4-dev Mar 18, 2017
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