fix(modal): change the order we return from show() method (#34087)

In case of a modal with fading enabled, a prevented show event can cause show to not showing the modal anymore.

See #34055

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

Update devDependencies

* @babel/cli                       ^7.14.3  →  ^7.14.8
* @babel/core                      ^7.14.3  →  ^7.14.8
* @babel/preset-env                ^7.14.2  →  ^7.14.8
* @rollup/plugin-commonjs          ^19.0.0  →  ^19.0.1
* @rollup/plugin-node-resolve      ^13.0.0  →  ^13.0.2
* autoprefixer                     ^10.2.5  →  ^10.3.1
* eslint                           ^7.26.0  →  ^7.31.0
* eslint-config-xo                 ^0.36.0  →  ^0.37.0
* eslint-plugin-import             ^2.23.2  →  ^2.23.4
* eslint-plugin-unicorn            ^31.0.0  →  ^34.0.1
* globby                           ^11.0.3  →  ^11.0.4
* hugo-bin                         ^0.71.1  →  ^0.73.0
* karma                             ^6.3.2  →   ^6.3.4
* karma-firefox-launcher            ^2.1.0  →   ^2.1.1
* linkinator                       ^2.13.6  →  ^2.14.0
* nodemon                           ^2.0.7  →   ^2.0.12
* node-sass                         ^6.0.0  →   ^6.0.1
* postcss                           ^8.3.0  →   ^8.3.5
* qunit                            ^2.15.0  →  ^2.16.0
* rollup                           ^2.48.0  →  ^2.53.3
* stylelint-config-twbs-bootstrap   ^2.2.0  →   ^2.2.3
* vnu-jar                           21.4.9  →  21.6.11

CI: make use of the `actions/setup-node`'s `cache` option (#34453)

Bump coverallsapp/github-action from 1.1.2 to 1.1.3 (#34491)

Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](https://github.com/coverallsapp/github-action/compare/v1.1.2...1.1.3)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>

CONTRIBUTING.md: fix broken link to README.md (#34389)

Backport #34336

Remove unneeded `.text-white-50` CSS rule

This is already part of Bootstrap 4.6 and 5.0.

Manually backport #34075 and #34279

Improve and clean up vnu-jar.js

Update Hugo config and sitemap.xml (#34074)

Handle complex expressions in add() & subtract() (#34047)

Replace Freenode with Libera.Chat in the remaining files (#34250)

Remove print thead rule (#34426)

The `thead` rule is safe to remove -- it was a workaround for browsers that didn't conform to this part of the CSS 2.1 spec. (https://www.w3.org/TR/CSS21/tables.html#table-display)
In practice, this meant IE 6 & IE 7.

(thanks to @mattbrundage for this PR upstream for the main.css project (https://github.com/h5bp/main.css/pull/101)

🐛 Enable `0.x` with negative margins @ v4 (#33593)

* 🐛 Enable `0.x` with negative margins @ v4

I set it as below, but no negative margin was added.
`$spacers: (0\.5: ($spacer * .125));`

After investigating, it seems that `0 \ .5` is also evaluated as 0.
So I changed it to string comparison.

HTML example: `<img class="mt-n0.5">`

* 💚 Clear CI errors

Fix repetition in the description.

docs(color): wrong secondary hex code (#34159)

Closes #34146

---

Not sure what happened but indeed, [our current `$secondary` is not what's shown in docs](https://github.com/twbs/bootstrap/blob/c2f949c1fdad09e30793b533e5502808c5e2d423/scss/_variables.scss#L68).

Update devDependencies

* @babel/cli                       ^7.13.14  →   ^7.14.3
* @babel/core                       ^7.14.0  →   ^7.14.3
* @babel/preset-env                 ^7.14.1  →   ^7.14.2
* @rollup/plugin-commonjs           ^18.0.0  →   ^19.0.0
* @rollup/plugin-node-resolve       ^11.2.1  →   ^13.0.0
* clean-css-cli                      ^5.2.2  →    ^5.3.0
* eslint                            ^7.23.0  →   ^7.26.0
* eslint-config-xo                  ^0.35.0  →   ^0.36.0
* eslint-plugin-import              ^2.22.1  →   ^2.23.2
* eslint-plugin-unicorn             ^29.0.0  →   ^31.0.0
* glob                               ^7.1.6  →    ^7.1.7
* hugo-bin                          ^0.70.0  →   ^0.71.1
* node-sass                          ^5.0.0  →    ^6.0.0
* postcss                            ^8.2.9  →    ^8.3.0
* qunit                             ^2.14.1  →   ^2.15.0
* rollup                            ^2.44.0  →   ^2.48.0
* stylelint                        ^13.12.0  →  ^13.13.1
* stylelint-config-twbs-bootstrap    ^2.1.0  →    ^2.2.0
* vnu-jar                            21.2.5  →    21.4.9

Replace freenode with libera IRC server (#34050)

CI: remove Node.js 10 (#33923)

Node.js 10 is EOL since 30/04/2021

Spinner alignment changes

JS tests: add Node.js 16 (#33720)

Update anchor.js to v4.3.1. (#33668)

docs: update clipboard.js to v2.0.8 (#33583)

Backport #33782 to v4

Backport #33634 (#33649)

Dropdown: support `.dropdown-item` wrapped in `<li>` tags

Remove aria-haspopup from dropdowns (#33624)

`aria-haspopup="true"` is really intended to signal that an ARIA `menu` will be opened on activation. as a result, some assistive technologies will announce controls with `aria-haspopup="true"` as a menu or menu item (e.g. JAWS and NVDA).
In addition, `aria-haspopup` seems to trigger a bug in Edge/Narrator where the `aria-expanded` state is not correctly announced at the moment when `aria-haspopup` is present.
This now makes the dropdown button more like a generic disclosure widget control - see also https://www.w3.org/TR/wai-aria-practices-1.2/examples/disclosure/disclosure-navigation.html01

fix typo in docs (#33562)

Update devDependencies

* @babel/cli                    ^7.13.0  →  ^7.13.14
* @babel/core                   ^7.13.8  →  ^7.13.14
* @babel/preset-env             ^7.13.9  →  ^7.13.12
* @rollup/plugin-commonjs       ^17.1.0  →   ^18.0.0
* @rollup/plugin-node-resolve   ^11.2.0  →   ^11.2.1
* autoprefixer                  ^10.2.4  →   ^10.2.5
* clean-css-cli                 ^5.2.0  →    ^5.2.2
* eslint                        ^7.21.0  →   ^7.23.0
* eslint-plugin-unicorn         ^28.0.2  →   ^29.0.0
* globby                        ^11.0.2  →   ^11.0.3
* hugo-bin                      ^0.69.0  →   ^0.70.0
* karma                         ^6.1.1  →    ^6.3.2
* postcss                       ^8.2.6  →    ^8.2.9
* qunit                         ^2.14.0  →   ^2.14.1
* rollup                        ^2.40.0  →   ^2.44.0
* stylelint                     ^13.11.0  →  ^13.12.0

Also lock jQuery to v3.5.1 since 3.6.0 has a bug that affects at least our tests

BrowserStack: test on iPhone 12 (#33398)

README.md: use shields.io for the tests badge too (#33401)

Improve change-version.js (#33148)

* switch to async/await
* remove shelljs
* switch to globby; supports .gitignore out of the box
* don't write any files to the disk if they are indentical
* add two more cmd switches: --dry and --verbose
* strip leading "v" from version arguments so that we don't end up with duplicate `v`s

Delete the unused build/ship.sh (#33326)

Manually backport #33386 to v4 to properly document Sass default variables

Manual backport of #32627 (#33165)

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

Update .bundlewatch.config.json

Adjust feedback icon position and padding for select.form-control

Update devDependencies

* @babel/cli             ^7.12.16  →   ^7.13.0
* @babel/core            ^7.12.16  →   ^7.13.8
* @babel/preset-env      ^7.12.16  →   ^7.13.9
* eslint                  ^7.20.0  →   ^7.21.0
* eslint-config-xo        ^0.34.0  →   ^0.35.0
* eslint-plugin-unicorn   ^28.0.1  →   ^28.0.2
* hugo-bin                ^0.68.0  →   ^0.69.0
* linkinator              ^2.13.4  →   ^2.13.6
* lockfile-lint            ^4.3.7  →    ^4.6.2
* rollup                  ^2.39.0  →   ^2.40.0
* stylelint              ^13.10.0  →  ^13.11.0

CodeQL: skip dependabot PRs too (#33130)

Bundlewatch: stop ignoring dependabot branches (#33192)

Rename _gh_pages to _site. (#33129)

linkinator: replace `--silent` with `--verbosity` (#33100)

Adjust `SAFE_URL_PATTERN` regex for use with `test` method. (#33153)

The `test` method on regexes does not behave like `match` on strings
for checks if the regex matches when the global modifier (g) is present.

Also adds a unit test on tooltips for sanitizing the same template twice.

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

Update clean-css-cli to v5.2.0 (#33144)

* Update clean-css-cli to v5.2.0

* Enable clean-css's batch flag

Partially backport #33045

Simplify karma.conf.js

`customLaunchers` does not seem to be needed.

Update devDependencies

* @babel/cli                   ^7.12.13  →  ^7.12.16
* @babel/core                  ^7.12.13  →  ^7.12.16
* @babel/preset-env            ^7.12.13  →  ^7.12.16
* @rollup/plugin-babel           ^5.2.3  →    ^5.3.0
* @rollup/plugin-node-resolve   ^11.1.1  →   ^11.2.0
* eslint                        ^7.19.0  →   ^7.20.0
* eslint-plugin-unicorn         ^27.0.0  →   ^28.0.1
* karma                          ^6.0.4  →    ^6.1.1
* postcss                        ^8.2.5  →    ^8.2.6
* rollup                        ^2.38.5  →   ^2.39.0
* stylelint                     ^13.9.0  →  ^13.10.0

Kept clean-css-cli to 4.x for the time being.

Backport #32940

* Update the default value for $enable-deprecation-messages

As seen in the https://github.com/twbs/bootstrap/blob/v4-dev/scss/_variables.scss,
the `$enable-deprecation-messages` variable is set to `true` by default.

Add a callout for the sanitizer in popovers and tooltips (#32272)

* Add a callout for the sanitizer in popovers and tooltips

* Add second reference to sanitizer in the options

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

vnu-jar.js: quote vnu path (#33016)

Should work now with folders with spaces

CI: remove the cache fallback and shorten cache key (#33012)

CI: switch to the coverallsapp/github-action tag (#32919)

Unfortunately, they don't offer a `v1` tag

change-version: remove vendor, add resources (#32917)

Fix minor visual bug in FF caused by moz-focusring (v4) (#32821)

Avoid bug in Firefox where text appears "noisy" due to text-shadow applying to focused form fields.

fix(forms): input-group and validation icons (#32968)

Fixes #30131

Issue mentions `form-control` but it impacts `.custom-select` — but not `textarea` since it has a qualifying type selector that increases its specificty.

So another way could be to set `select.custom-select` and `input.form-control`. Would there be any other tag concerned by those classes?

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

Simplify bugify.html (#33063)

* use `in` instead of `strings.Contains` since the latter is not documented
* use `cond`

Update devDependencies (#32967)

* @babel/cli                  ^7.12.10  →  ^7.12.13
* @babel/core                 ^7.12.10  →  ^7.12.13
* @babel/preset-env           ^7.12.11  →  ^7.12.13
* @rollup/plugin-babel          ^5.2.2  →    ^5.2.3
* @rollup/plugin-commonjs      ^17.0.0  →   ^17.1.0
* @rollup/plugin-node-resolve  ^11.1.0  →   ^11.1.1
* autoprefixer                 ^10.2.3  →   ^10.2.4
* eslint                       ^7.18.0  →   ^7.19.0
* karma                         ^5.2.3  →    ^6.0.4
* karma-qunit                   ^4.1.1  →    ^4.1.2
* linkinator                    2.11.2  →   ^2.13.4
* postcss                       ^8.2.4  →    ^8.2.5
* rollup                       ^2.38.0  →   ^2.38.5
* vnu-jar                      20.6.30  →    21.2.5

_variables.scss: add missing newline before new section (#32868)

Switch to jsDelivr for jQuery too. (#32918)

Update devDependencies (#32852)

Pin karma and linkinator due to failures

Release v4.6.0. (#32726)

* Bump version to v4.6.0.

* Dist

Backport #32037 (#32833)

* Backport #32037

Add `.navbar-nav-scroll` for vertical scrolling of navbar content

* drop the css var for a Sass css

Co-authored-by: Mark Otto <markdotto@gmail.com>

Backport #32759 (#32832)

Mention CSP and embedded SVGs in v4 docs

Backport #32760 (#32827)

Add a live toast example to the docs

Docs: Sass implementation and rounding precision (#32791)

* Backport of changes to v5 docs that updated and clarified information about the Sass compiler used and the minimum rounding precision we recommend with it.

Co-authored-by: Mark Otto <otto@github.com>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>

Lighten disabled dropdown text to $gray-500

Fixes #32474

Backport #32469 and #32799

Improve zip-examples.js by only including the assets we need

Document some basic code conventions for us (#32778)

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

CI: change Node.js to `14` (#32796)

This is so that it's consistent with the JS workflow thus the cache should match.

Backport #32795 (#32809)

* Clarify Sass import and customize docs for how to modify variable defaults

* Add an npm starter project callout to a few pages

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"`

Update devDependencies

Stylelint: disallow some property values (#32756)

* `border: none`
* `outline: none`

Mention stretched-link constraints with table elements (#32761)

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

GitHub Actions cleanup (#32755)

* lowercase `runner.os`
* remove `CI` environment variable since it's already set by the runner

Darken dropdown item hover style

Backport of https://github.com/twbs/bootstrap/pull/32754

Consistently use outline:0 rather than outline:none

Manual backport of https://github.com/twbs/bootstrap/pull/32751 (which has not been merged yet, but assume it will shortly)

Suppress focus outline for buttons when it shouldn't be visible in Chromium

Manual backport of https://github.com/twbs/bootstrap/pull/32689

Remove old/unnecessary reboot bug fix

Manual backport of https://github.com/twbs/bootstrap/pull/32631

Backport #32737

package.json: move `version_short` variable under the `config` object

This is so that it works on npm 7.x too.

Update karma config

* rename constants
* enforce the constants to be Boolean
* use the coverage plugin only one time, on the latest jQuery run
* fix karma `logLevel`

karma: run tests in parallel

docs: add v5.0 in versions (#32728)

Update devDependencies

* @babel/cli                   ^7.12.8  →  ^7.12.10
* @babel/core                  ^7.12.8  →  ^7.12.10
* @babel/preset-env            ^7.12.7  →  ^7.12.11
* @rollup/plugin-babel          ^5.2.1  →    ^5.2.2
* @rollup/plugin-commonjs      ^16.0.0  →   ^17.0.0
* @rollup/plugin-node-resolve  ^10.0.0  →   ^11.0.1
* autoprefixer                 ^10.0.2  →   ^10.2.1
* cross-env                     ^7.0.2  →    ^7.0.3
* eslint                       ^7.14.0  →   ^7.17.0
* eslint-config-xo             ^0.33.1  →   ^0.34.0
* eslint-plugin-unicorn        ^23.0.0  →   ^25.0.1
* find-unused-sass-variables    ^3.0.0  →    ^3.1.0
* hugo-bin                     ^0.66.2  →   ^0.68.0
* linkinator                    ^2.2.2  →   ^2.11.2
* nodemon                       ^2.0.6  →    ^2.0.7
* postcss                      ^8.1.10  →    ^8.2.3
* postcss-cli                   ^8.3.0  →    ^8.3.1
* qunit                        ^2.12.0  →   ^2.13.0
* rollup                       ^2.33.3  →   ^2.36.1

Backport #32423

Add two new variables for pagination border-radius values

Backport #32632

Docs: Tweak the wording for collapse to indicate button is preferred/more semantic

Backport #32685

Clarify the `$enable-shadows` option in our docs

Backport #32713

Bump copyright year to 2021

Remove incorrect mention of dropdowns for dynamic tab behavior (#32634)

While technically it still works, we're already saying authors shouldn't do it later on, and we removed any examples of this. This was a straggler.

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

CI: update to actions/setup-node@v2 (#32635)

Tooltips: add missing space (#32529)

Backport #32489

Fix popover docs example

BrowserStack: test on macOS Catalina instead of High Sierra (#32486)

Docs: style clipboard button on `:focus`, not just `:hover` (#32461)

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

Update CodeQL Action (#32452)

CI: add a 30-min timeout for BrowserStack (#32238)

CONTRIBUTING: mention dist files (#32314)

* CONTRIBUTING: mention dist files

* Update CONTRIBUTING.md

Co-authored-by: Mark Otto <markd.otto@gmail.com>

Remove duplicated 'follow Bootstrap on Twitter' link in Community section (#32399)

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

build-plugins: switch to "bundled" for babel helpers (#32312)

README: fix path to search.js (#32257)

karma: add `v4` in Browserstack build ID (#32502)

docs: Add Russian translation (#32253)

v4: Add an actual `data-touch="false"` example in the carousel docs (#32639)

* v4: Add an actual `data-touch="false"` example in the carousel docs

Backport of https://github.com/twbs/bootstrap/pull/32638

* Remove data-ride from example carousel

* Correct attribute table explanation and remove slide class from example

i believe the correct option is `slide` here, not `ride`.
the `if "carousel"` bit was rather unclear/confusing

* Edits...

sorry, trying to make sense of our quirky carousel stuff, it seems some illogical things have been there ever since

* Add data-interval to suppress autoplay

* Tweak carousel options formatting

* Update site/content/docs/4.5/components/carousel.md

Co-authored-by: Mark Otto <markd.otto@gmail.com>
* Tweak language

mostly based on @mdo's suggestion, but also adds the word "attribute" after `data-ride` for better flow/legibility

Co-authored-by: Mark Otto <markd.otto@gmail.com>

v4 backport: Replace Lorem Ipsum placeholder text with more representative (or at least english language) text (#32462)

Backport and expansion (for v4) of #32246 and #32479