-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Docs cleanup #14166
Docs cleanup #14166
Conversation
|
||
<h3>Methods</h3> | ||
|
||
<h4>$().alert()</h4> | ||
<p>Wraps all alerts with close functionality. To have your alerts animate out when closed, make sure they have the <code>.fade</code> and <code>.in</code> class already applied to them.</p> | ||
<p>Makes an alert listen for click events on descendant elements which have the <code>data-dismiss="alert"</code> attribute. (Not necessary when using the auto-initialization of the data-api.)</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cvrebert Mind handing me the eloquence for one second here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"when using the data-api's auto-initialization" seems slightly better. Looks good overall.
Otherwise, LGTM 👍 |
Just getting started, thanks for the feedback. |
Wasn't done but I guess I can put the other changes in different PR. |
Oh, oops. 😁 The more PRs the better I suppose. I just want to move things along quickly because v4 is so much more interesting to work on right now lol. |
That's what I thought 😄 Had thought about splitting these changes up into separate PRs for each section for a while anyway. |
More to come. Posting already to get feedback on wording/certain changes.