mirror of https://github.com/twbs/bootstrap.git
				
				
				
			Fix invalid `<nav ... role="tablist">` in docs.
This commit is contained in:
		
							parent
							
								
									fb4c9fae17
								
							
						
					
					
						commit
						f9188de4af
					
				| 
						 | 
				
			
			@ -27,7 +27,6 @@ childProcess.exec('java -version', (error, stdout, stderr) => {
 | 
			
		|||
    // Firefox's non-standard autocomplete behavior - see https://bugzilla.mozilla.org/show_bug.cgi?id=654072
 | 
			
		||||
    'Attribute “autocomplete” is only allowed when the input type is.*',
 | 
			
		||||
    'Attribute “autocomplete” not allowed on element “button” at this point.',
 | 
			
		||||
    'Bad value “tablist” for attribute “role” on element “nav”.',
 | 
			
		||||
    // We use holder.js with `data-src` and no `src` to avoid 404 errors;
 | 
			
		||||
    // we could work around this, but I'm not sure it's worth it.
 | 
			
		||||
    'Element “img” is missing required attribute “src”.',
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -347,13 +347,15 @@ Note that dynamic tabbed interfaces should <em>not</em> contain dropdown menus,
 | 
			
		|||
</div>
 | 
			
		||||
{% endhighlight %}
 | 
			
		||||
 | 
			
		||||
To help fit your needs, this works with `<ul>`-based markup, as shown above, as well as `<nav>`-based markup shown below.
 | 
			
		||||
To help fit your needs, this works with `<ul>`-based markup, as shown above, or with any arbitrary "roll your own" markup. Note that if you're using `<nav>`, you shouldn't add `role="tablist"` directly to it, as this would override the element's native role as a navigation landmark. Instead, switch to an alternative element (in the exammple below, a simple `<div>`) and wrap the `<nav>` around it.
 | 
			
		||||
 | 
			
		||||
<div class="bd-example bd-example-tabs">
 | 
			
		||||
  <nav class="nav nav-tabs" id="nav-tab" role="tablist">
 | 
			
		||||
  <nav>
 | 
			
		||||
    <div class="nav nav-tabs" id="nav-tab" role="tablist">
 | 
			
		||||
      <a class="nav-item nav-link active" id="nav-home-tab" data-toggle="tab" href="#nav-home" role="tab" aria-controls="nav-home" aria-selected="true">Home</a>
 | 
			
		||||
      <a class="nav-item nav-link" id="nav-profile-tab" data-toggle="tab" href="#nav-profile" role="tab" aria-controls="nav-profile" aria-selected="false">Profile</a>
 | 
			
		||||
      <a class="nav-item nav-link" id="nav-contact-tab" data-toggle="tab" href="#nav-contact" role="tab" aria-controls="nav-contact" aria-selected="false">Contact</a>
 | 
			
		||||
    </div>
 | 
			
		||||
  </nav>
 | 
			
		||||
  <div class="tab-content" id="nav-tabContent">
 | 
			
		||||
    <div class="tab-pane fade show active" id="nav-home" role="tabpanel" aria-labelledby="nav-home-tab">
 | 
			
		||||
| 
						 | 
				
			
			@ -369,10 +371,12 @@ To help fit your needs, this works with `<ul>`-based markup, as shown above, as
 | 
			
		|||
</div>
 | 
			
		||||
 | 
			
		||||
{% highlight html %}
 | 
			
		||||
<nav class="nav nav-tabs" id="myTab" role="tablist">
 | 
			
		||||
<nav>
 | 
			
		||||
  <div class="nav nav-tabs" id="nav-tab" role="tablist">
 | 
			
		||||
    <a class="nav-item nav-link active" id="nav-home-tab" data-toggle="tab" href="#nav-home" role="tab" aria-controls="nav-home" aria-selected="true">Home</a>
 | 
			
		||||
    <a class="nav-item nav-link" id="nav-profile-tab" data-toggle="tab" href="#nav-profile" role="tab" aria-controls="nav-profile" aria-selected="false">Profile</a>
 | 
			
		||||
    <a class="nav-item nav-link" id="nav-contact-tab" data-toggle="tab" href="#nav-contact" role="tab" aria-controls="nav-contact" aria-selected="false">Contact</a>
 | 
			
		||||
  </div>
 | 
			
		||||
</nav>
 | 
			
		||||
<div class="tab-content" id="nav-tabContent">
 | 
			
		||||
  <div class="tab-pane fade show active" id="nav-home" role="tabpanel" aria-labelledby="nav-home-tab">...</div>
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue