From 80aefd862c18a2f44a2870788312aa2a0f0f7091 Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Thu, 14 Jan 2021 10:19:19 +0000 Subject: [PATCH] Remove role="button" from CTA links in carousel example (#32789) They're links, acting as links. Just because they're styled as buttons visually (as is often the case for "Call to action" (CTA) links) doesn't mean they need/get `role="button"` --- site/content/docs/4.5/examples/carousel/index.html | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/site/content/docs/4.5/examples/carousel/index.html b/site/content/docs/4.5/examples/carousel/index.html index c231a92843..96f71f7729 100644 --- a/site/content/docs/4.5/examples/carousel/index.html +++ b/site/content/docs/4.5/examples/carousel/index.html @@ -46,7 +46,7 @@ extra_css: @@ -56,7 +56,7 @@ extra_css: @@ -66,7 +66,7 @@ extra_css: @@ -94,19 +94,19 @@ extra_css: {{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}

Heading

Some representative placeholder content for the three columns of text below the carousel. This is the first column.

-

View details »

+

View details »

{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}

Heading

Another exciting bit of representative placeholder content. This time, we've moved on to the second column.

-

View details »

+

View details »

{{< placeholder width="140" height="140" background="#777" color="#777" class="rounded-circle" >}}

Heading

And lastly this, the third column of representative placeholder content.

-

View details »

+

View details »