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

Alert dismissable fix example #10207

Closed
NinoD opened this issue Aug 27, 2013 · 1 comment
Closed

Alert dismissable fix example #10207

NinoD opened this issue Aug 27, 2013 · 1 comment

Comments

@NinoD
Copy link

NinoD commented Aug 27, 2013

Just a small fix in docs.
http://getbootstrap.com/components/#alerts-dismissable

<div class="alert alert-dismissable">

should have .alert-contextual

<div class="alert alert-warning alert-dismissable">

In your own words:
"No default class
Alerts don't have default classes, only base and modifier classes. A default gray alert doesn't make too much sense, so you're required to specify a type via contextual class. Choose from success, info, warning, or danger."

clbn added a commit to clbn/bootstrap that referenced this issue Aug 27, 2013
@mdo
Copy link
Member

mdo commented Aug 27, 2013

Already been addressed in 3.0.1 (in master).

@mdo mdo closed this as completed Aug 27, 2013
mdo added a commit that referenced this issue Aug 27, 2013
Fixes #10207: alert-dismissable example requires contextual class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants