Skip to content

Commit

Permalink
Added type="button" to <button>s without any type (#23866)
Browse files Browse the repository at this point in the history
  • Loading branch information
Herst authored and mdo committed Oct 3, 2017
1 parent 0c03b70 commit aae11ab
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 13 deletions.
4 changes: 2 additions & 2 deletions docs/4.0/components/badge.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ Badges scale to match the size of the immediate parent element by using relative
Badges can be used as part of links or buttons to provide a counter.

{% example html %}
<button class="btn btn-primary">
<button type="button" class="btn btn-primary">
Notifications <span class="badge badge-light">4</span>
</button>
{% endexample %}
Expand All @@ -41,7 +41,7 @@ Note that depending on how they are used, badges may be confusing for users of s
Unless the context is clear (as with the "Notifications" example, where it is understood that the "4" is the number of notifications), consider including additional context with a visually hidden piece of additional text.

{% example html %}
<button class="btn btn-primary">
<button type="button" class="btn btn-primary">
Profile <span class="badge badge-light">9</span>
<span class="sr-only">unread messages</span>
</button>
Expand Down
2 changes: 1 addition & 1 deletion docs/4.0/components/modal.md
Original file line number Diff line number Diff line change
Expand Up @@ -422,7 +422,7 @@ Modals have two optional sizes, available via modifier classes to be placed on a

{% highlight html %}
<!-- Large modal -->
<button class="btn btn-primary" data-toggle="modal" data-target=".bd-example-modal-lg">Large modal</button>
<button type="button" class="btn btn-primary" data-toggle="modal" data-target=".bd-example-modal-lg">Large modal</button>

<div class="modal fade bd-example-modal-lg" tabindex="-1" role="dialog" aria-labelledby="myLargeModalLabel" aria-hidden="true">
<div class="modal-dialog modal-lg">
Expand Down
16 changes: 8 additions & 8 deletions docs/4.0/examples/tooltip-viewport/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -18,19 +18,19 @@

<body>

<button class="btn btn-secondary float-right tooltip-bottom" title="This should be shifted to the left">Shift Left</button>
<button class="btn btn-secondary tooltip-bottom" title="This should be shifted to the right">Shift Right</button>
<button class="btn btn-secondary tooltip-right" title="This should be shifted down">Shift Down</button>
<button type="button" class="btn btn-secondary float-right tooltip-bottom" title="This should be shifted to the left">Shift Left</button>
<button type="button" class="btn btn-secondary tooltip-bottom" title="This should be shifted to the right">Shift Right</button>
<button type="button" class="btn btn-secondary tooltip-right" title="This should be shifted down">Shift Down</button>

<button class="btn btn-secondary tooltip-right btn-bottom" title="This should be shifted up">Shift Up</button>
<button type="button" class="btn btn-secondary tooltip-right btn-bottom" title="This should be shifted up">Shift Up</button>

<div class="container-viewport">
<button class="btn btn-secondary tooltip-viewport-bottom" title="This should be shifted to the left">Shift Left</button>
<button class="btn btn-secondary tooltip-viewport-right" title="This should be shifted down">Shift Down</button>
<button type="button" class="btn btn-secondary tooltip-viewport-bottom" title="This should be shifted to the left">Shift Left</button>
<button type="button" class="btn btn-secondary tooltip-viewport-right" title="This should be shifted down">Shift Down</button>

<button class="btn btn-secondary float-right tooltip-viewport-bottom" title="This should be shifted to the right">Shift Right</button>
<button type="button" class="btn btn-secondary float-right tooltip-viewport-bottom" title="This should be shifted to the right">Shift Right</button>

<button class="btn btn-secondary tooltip-viewport-right btn-bottom" title="This should be shifted up">Shift Up</button>
<button type="button" class="btn btn-secondary tooltip-viewport-right btn-bottom" title="This should be shifted up">Shift Up</button>
</div>


Expand Down
4 changes: 2 additions & 2 deletions docs/4.0/utilities/clearfix.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ The following example shows how the clearfix can be used. Without the clearfix t

{% example html %}
<div class="bg-info clearfix">
<button class="btn btn-secondary float-left">Example Button floated left</button>
<button class="btn btn-secondary float-right">Example Button floated right</button>
<button type="button" class="btn btn-secondary float-left">Example Button floated left</button>
<button type="button" class="btn btn-secondary float-right">Example Button floated right</button>
</div>
{% endexample %}

0 comments on commit aae11ab

Please sign in to comment.