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.

Sign up for GitHub

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

Jump to bottom

Carousel: use buttons, not links, for prev/next controls #32627

Merged
merged 20 commits into from
Jan 27, 2021
Merged

Carousel: use buttons, not links, for prev/next controls #32627

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
f328d01
Carousel: use buttons, not links, for prev/next
patrickhlauke Dec 27, 2020
9116ff2
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Dec 28, 2020
b749662
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Dec 29, 2020
50ac915
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Dec 29, 2020
d753f4a
Use buttons for newly-merged extra carousel
patrickhlauke Dec 29, 2020
7b718ca
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 5, 2021
13eb932
Edit carousel examples
patrickhlauke Jan 7, 2021
eda76bb
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 7, 2021
c72d23b
Clarify that controls can be button or link
patrickhlauke Jan 7, 2021
1493ac0
Merge branch 'patrickhlauke-carousel' of https://github.com/twbs/boot…
patrickhlauke Jan 7, 2021
a2447ba
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 10, 2021
7e1855d
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 13, 2021
80f5812
Revert separate/unrelated change to CTA links
patrickhlauke Jan 13, 2021
c9cbb17
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 15, 2021
6271d10
Merge branch 'main' into patrickhlauke-carousel
rohit2sharma95 Jan 19, 2021
c49d354
Update site/content/docs/5.0/components/carousel.md
patrickhlauke Jan 25, 2021
6373ac1
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 25, 2021
e4414c7
Merge branch 'main' into patrickhlauke-carousel
XhmikosR Jan 26, 2021
47657bf
Merge branch 'main' into patrickhlauke-carousel
patrickhlauke Jan 27, 2021
3f0f00a
Explicitly set padding to 0 to prevent dipping/moving on active in Fi…
patrickhlauke Jan 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions site/content/docs/5.0/examples/carousel-rtl/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
<div class="carousel-caption text-start">
<h1>عنوان المثال.</h1>
<p> نهاية الأوضاع ان أضف, هو مما رجوعهم وقدّموا. أي عدد الدمج نهاية وأكثرها, المسرح الباهضة وبولندا حول و, كل أما سياسة أسابيع. مع حيث قُدُماً الكونجرس, بها و خيار ٢٠٠٤, كلا في مكّن وقام. مع يكن زهاء بالفشل, الجوي الصين الشمال إذ على.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">سجل اليوم</a></p>
<p><a class="btn btn-lg btn-primary" href="#">سجل اليوم</a></p>
</div>
</div>
</div>
Expand All @@ -59,7 +59,7 @@ <h1>عنوان المثال.</h1>
<div class="carousel-caption">
<h1>عنوان مثال آخر.</h1>
<p>ثم تزامناً الفرنسي الإقتصادية دار. لكل عن الضغوط المتّبعة, أن حتى إختار المدن بالإنزال. عن الشمل بالفشل تلك, بل أراض أوزار بلديهما حول. دون لكون والتي ثم, كُلفة ويعزى استطاعوا أن لمّ. جُل بخطوط واحدة البشريةً.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">أعرف أكثر</a></p>
patrickhlauke marked this conversation as resolved.
Show resolved Hide resolved
<p><a class="btn btn-lg btn-primary" href="#">أعرف أكثر</a></p>
</div>
</div>
</div>
Expand All @@ -69,19 +69,19 @@ <h1>عنوان مثال آخر.</h1>
<div class="carousel-caption text-end">
<h1>واحد أكثر لقياس جيد.</h1>
<p>قررت العصبة إيطاليا وتم أن, عن سكان وقامت الحكومة وفي. كان بـ اوروبا اليابانية, ثمّة بوابة يتعلّق عل بعض. عدم رئيس الآلاف أن حدى.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">تصفح المعرض</a></p>
<p><a class="btn btn-lg btn-primary" href="#">تصفح المعرض</a></p>
</div>
</div>
</div>
</div>
<a class="carousel-control-prev" href="#myCarousel" role="button" data-bs-slide="prev">
<button class="carousel-control-prev" type="button" data-bs-target="#myCarousel" data-bs-slide="prev">
<span class="carousel-control-prev-icon" aria-hidden="true"></span>
<span class="visually-hidden">السابق</span>
</a>
<a class="carousel-control-next" href="#myCarousel" role="button" data-bs-slide="next">
</button>
<button class="carousel-control-next" type="button" data-bs-target="#myCarousel" data-bs-slide="next">
<span class="carousel-control-next-icon" aria-hidden="true"></span>
<span class="visually-hidden">التالى</span>
</a>
</button>
</div>


Expand All @@ -97,19 +97,19 @@ <h1>واحد أكثر لقياس جيد.</h1>
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>عنوان</h2>
<p>ان وتم عجّل الأجل, قبل نتيجة المشترك بـ, أي جعل جورج أوزار المسرح. أن وإعلان الساحل تلك, مكن ان استبدال الباهضة التكاليف. الى ماذا اليميني الحكومة في, إجلاء نتيجة قبل تم. مساعدة بولندا، أي هذه الحكم.</p>
<p><a class="btn btn-secondary" href="#" role="button">عرض التفاصيل</a></p>
<p><a class="btn btn-secondary" href="#">عرض التفاصيل</a></p>
</div><!-- /.col-lg-4 -->
<div class="col-lg-4">
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>عنوان</h2>
<p>هو أخر إتفاقية الدولارات الأوروبيّون, ثانية طوكيو و به،, ونتج أعمال مما أم. عن الا يونيو أفريقيا, السيطرة التقليدي ومن ٣٠. هو الغالي الإتفاقية ويكيبيديا، مكن, و الى هُزم اعتداء وايرلندا. وقبل بمباركة الأوروبيّون عن فقد, بتحدّي والفلبين ما كلا.</p>
<p><a class="btn btn-secondary" href="#" role="button">عرض التفاصيل</a></p>
<p><a class="btn btn-secondary" href="#">عرض التفاصيل</a></p>
</div><!-- /.col-lg-4 -->
<div class="col-lg-4">
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>عنوان</h2>
<p>غير عن الثقيل وسمّيت الأوضاع, لم تاريخ بالحرب للسيطرة حين, دار اللا تطوير تم. الى بشرية اليابان في. أما الشهيرة الإثنان وايرلندا ما, لإعلان واشتدّت و مدن. في غير والحزب للسيطرة الإكتفاء. ثانية الكونجرس الا من, جُل ٣٠ وبداية الشرقية.</p>
<p><a class="btn btn-secondary" href="#" role="button">عرض التفاصيل</a></p>
<p><a class="btn btn-secondary" href="#">عرض التفاصيل</a></p>
</div><!-- /.col-lg-4 -->
</div><!-- /.row -->

Expand Down
20 changes: 10 additions & 10 deletions site/content/docs/5.0/examples/carousel/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
<div class="carousel-caption text-start">
<h1>Example headline.</h1>
<p>Some representative placeholder content for the first slide of the carousel.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">Sign up today</a></p>
<p><a class="btn btn-lg btn-primary" href="#">Sign up today</a></p>
</div>
</div>
</div>
Expand All @@ -58,7 +58,7 @@ <h1>Example headline.</h1>
<div class="carousel-caption">
<h1>Another example headline.</h1>
<p>Some representative placeholder content for the second slide of the carousel.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">Learn more</a></p>
<p><a class="btn btn-lg btn-primary" href="#">Learn more</a></p>
</div>
</div>
</div>
Expand All @@ -68,19 +68,19 @@ <h1>Another example headline.</h1>
<div class="carousel-caption text-end">
<h1>One more for good measure.</h1>
<p>Some representative placeholder content for the third slide of this carousel.</p>
<p><a class="btn btn-lg btn-primary" href="#" role="button">Browse gallery</a></p>
<p><a class="btn btn-lg btn-primary" href="#">Browse gallery</a></p>
</div>
</div>
</div>
</div>
<a class="carousel-control-prev" href="#myCarousel" role="button" data-bs-slide="prev">
<button class="carousel-control-prev" type="button" data-bs-target="#myCarousel" data-bs-slide="prev">
<span class="carousel-control-prev-icon" aria-hidden="true"></span>
<span class="visually-hidden">Previous</span>
</a>
<a class="carousel-control-next" href="#myCarousel" role="button" data-bs-slide="next">
</button>
<button class="carousel-control-next" type="button" data-bs-target="#myCarousel" data-bs-slide="next">
<span class="carousel-control-next-icon" aria-hidden="true"></span>
<span class="visually-hidden">Next</span>
</a>
</button>
</div>


Expand All @@ -96,19 +96,19 @@ <h1>One more for good measure.</h1>
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>Heading</h2>
<p>Some representative placeholder content for the three columns of text below the carousel. This is the first column.</p>
<p><a class="btn btn-secondary" href="#" role="button">View details &raquo;</a></p>
<p><a class="btn btn-secondary" href="#">View details &raquo;</a></p>
</div><!-- /.col-lg-4 -->
<div class="col-lg-4">
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>Heading</h2>
<p>Another exciting bit of representative placeholder content. This time, we've moved on to the second column.</p>
<p><a class="btn btn-secondary" href="#" role="button">View details &raquo;</a></p>
<p><a class="btn btn-secondary" href="#">View details &raquo;</a></p>
</div><!-- /.col-lg-4 -->
<div class="col-lg-4">
{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}
<h2>Heading</h2>
<p>And lastly this, the third column of representative placeholder content.</p>
<p><a class="btn btn-secondary" href="#" role="button">View details &raquo;</a></p>
<p><a class="btn btn-secondary" href="#">View details &raquo;</a></p>
</div><!-- /.col-lg-4 -->
</div><!-- /.row -->

Expand Down