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

javascript.html: add data-ride note #10146

Merged
merged 1 commit into from
Aug 26, 2013
Merged

javascript.html: add data-ride note #10146

merged 1 commit into from
Aug 26, 2013

Conversation

unwiredben
Copy link
Contributor

carousel docs were missing mention of the data-ride attribute
used to automatically start the carousel at load time

@kamui545
Copy link

Just what I was looking for :)

@@ -1755,7 +1755,7 @@ <h2 id="carousel-examples">Examples</h2>
<h2 id="carousel-usage">Usage</h2>

<h3>Via data attributes</h3>
<p>Use data attributes to easily control the position of the carousel. <code>data-slide</code> accepts the keywords <code>prev</code> or <code>next</code>, which alters the slide position relative to its current position. Alternatively, use <code>data-slide-to</code> to pass a raw slide index to the carousel <code>data-slide-to="2"</code>, which shifts the slide position to a particular index beginning with <code>0</code>.</p>
<p>Use data attributes to easily control the position of the carousel. <code>data-slide</code> accepts the keywords <code>prev</code> or <code>next</code>, which alters the slide position relative to its current position. Alternatively, use <code>data-slide-to</code> to pass a raw slide index to the carousel <code>data-slide-to="2"</code>, which shifts the slide position to a particular index beginning with <code>0</code>. The <code>data-ride="carousel"</code> attribute is used to mark a carousel as animating starting at page load.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

The new sentence should probably go in a new paragraph since it doesn't concern positioning.

carousel docs were missing mention of the data-ride attribute
used to automatically start the carousel at load time
@unwiredben
Copy link
Contributor Author

Good feedback, @cvrebert. Just updated my PR.

cvrebert added a commit that referenced this pull request Aug 26, 2013
javascript.html: add data-ride note
@cvrebert cvrebert merged commit d977c99 into twbs:master Aug 26, 2013
@cvrebert
Copy link
Collaborator

Great! Thanks.

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

Successfully merging this pull request may close these issues.

3 participants