]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Dynamic tabs: use buttons rather than links (#32630)
authorPatrick H. Lauke <redux@splintered.co.uk>
Tue, 9 Feb 2021 05:23:45 +0000 (05:23 +0000)
committerGitHub <noreply@github.com>
Tue, 9 Feb 2021 05:23:45 +0000 (07:23 +0200)
commit96be06e32bf68de22d8d14362909dd688070ca40
treeb18f4c583be94a4f75c63785d5b4cf217ae5635d
parentc93d754d35eadf6cb81a8fccef9ce091ea4f28c9
Dynamic tabs: use buttons rather than links (#32630)

* Dynamic tabs: use buttons rather than links

- change docs
- add mention that tabs should be <button> elements
- tweak styles to neutralise border and background

* Update js unit and visual test accordingly

- replace links with buttons
- make one specific test that uses links instead of buttons, as we still want to support it despite it being non-semantically appropriate
- Leaving a couple of tests for now. The test for removed tabs should be redone so that tabs are removed programmatically (as the approach of having that close button inside the link is invalid and broken markup). The test for dropdowns should be removed together we actually ripping out the handling for dropdowns in the tab.js code (arguably a breaking change, though we discouraged this for a few versions and effectively "deprecated" it)

* Add isolation:isolate to prevent focus being overlapped

https://github.com/twbs/bootstrap/pull/32630#issuecomment-756015766
js/tests/unit/tab.spec.js
js/tests/visual/tab.html
scss/_nav.scss
site/content/docs/5.0/components/navs-tabs.md