Change adjustDialog's if conditions to improve readability

Add test for modal-content

Respect modal's initial overflowY

Remove some uncovered code that seems to be unused

Change check for dynamic modal

Tweak methods

Name them to be more descriptive and have agnostic functionality

Some refactoring on modal, to improve readability and generic functionality

Add a template factory helper to handle all template cases (#34519)

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

build/change-version.js: fix wrong map usage (#35395)

build: read & dynamically resolve `imports` on plugins build (#34509)

Our individual js/dist files are now deduplicated properly thus resulting in a size reduction, which varies from ~25% to ~60% depending on the components used. The average savings are 20% uncompressed and ~15% with gzip.

This will mostly benefit cases that more than one component is imported from js/dist. In all other cases it doesn't have any effect.

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

Make footer examples responsive to mobile screens (#35365)

Add missing scss/maps imports (#35373)

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

Update devDependencies (#35368)

* @babel/preset-env      ^7.16.0  →  ^7.16.4
* cspell                 ^5.12.6  →  ^5.13.0
* eslint                  ^8.2.0  →   ^8.3.0
* eslint-plugin-unicorn  ^38.0.1  →  ^39.0.0
* hugo-bin               ^0.77.2  →  ^0.77.4
* karma                   ^6.3.8  →   ^6.3.9
* rollup                 ^2.60.0  →  ^2.60.1

Update devDependencies (#35356)

* eslint-plugin-import  ^2.25.2  →  ^2.25.3
* linkinator            ^2.14.5  →  ^2.16.2
* nodemon               ^2.0.14  →  ^2.0.15
* rollup                ^2.59.0  →  ^2.60.0
* terser                 ^5.9.0  →  ^5.10.0

Move reassigned Sass maps for colors to another stylesheet (#34942)

Fix typo in vertical alignment of table cells (#35348)

Update devDependencies (#35322)

* cspell                 ^5.12.4  →  ^5.12.6
* eslint                  ^8.1.0  →   ^8.2.0
* eslint-plugin-unicorn  ^37.0.1  →  ^38.0.1
* hugo-bin               ^0.76.1  →  ^0.77.2
* karma                   ^6.3.7  →   ^6.3.8
* linkinator             ^2.14.4  →  ^2.14.5
* postcss-cli             ^9.0.1  →   ^9.0.2
* rtlcss                  ^3.4.0  →   ^3.5.0

floating-labels: remove aria-label in the select example (#35327)

While it is understood that this is just an example, the visible text (label) of "Works with selects" and the `aria-label="Floating label select example"` created a [WCAG 2.5.3 Label in name](https://www.w3.org/WAI/WCAG21/quickref/#label-in-name) failure.

As the `aria-label` isn't necessary here since this `select` is already provided an accessible name by its `label` element, removing the unnecessary `aria-label` seems the best course of action as:
* removing it solves the WCAG issue
* it removes the potential implication to developers that they'd even _need_ an `aria-label` here, let alone indirectly suggesting that it's ok for the visible text and accessible name to be out of alignment

Fix spacing utility classes mentioned in navbar supported content documentation (#35328)

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

README.md remove broken "David DM" dependency badges (#35313)

Update import stacks required for modifying utilities (#35320)

Add top placement info to offcanvas docs

Update devDependencies (#35270)

* @babel/cli                ^7.15.7  →  ^7.16.0
* @babel/core               ^7.15.8  →  ^7.16.0
* @babel/preset-env         ^7.15.8  →  ^7.16.0
* autoprefixer              ^10.3.7  →  ^10.4.0
* cspell                    ^5.12.3  →  ^5.12.4
* eslint                     ^8.0.1  →   ^8.1.0
* karma                      ^6.3.5  →   ^6.3.7
* karma-firefox-launcher     ^2.1.1  →   ^2.1.2
* rollup                    ^2.58.0  →  ^2.59.0
* sass                      ^1.43.2  →  ^1.43.4

`make-col-ready()`: remove the unused `$gutter` variable (#34334)

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

Always set the CSS variables for gutters in containers (#34644)

We already do this in rows, so to best support our containers, we need
to do it at the container level as well.

Fixes #32658, fixes #34614, closes #32658.

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

Add missing `role="search"` and `type="search"` in navbar doc and examples (#35223)

* Add missing `role="search"` and `type="search"` in navbar doc and examples

* Update site/content/docs/5.1/components/navbar.md

* Remove warning callout about ensuring correct search role in navbar doc

Co-authored-by: Gaël Poupard <ffoodd@users.noreply.github.com>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>

update grid example docs (#35308)

Don't override CSS `direction` in code elements (#35230)

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

Ensure sufficient contrast in `accordion-item` (#35231)

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

Drop prefixed version of `::file-selector-button` (#35232)

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

images.md: remove zero-width space (#35234)

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

fix(offcanvas): Adding titles class where missing (#35264)

Co-authored-by: louismaximepiton <louismaxime.piton@orange.com>

Update devDependencies (#35233)

* @rollup/plugin-commonjs      ^21.0.0  →  ^21.0.1
* @rollup/plugin-node-resolve  ^13.0.5  →  ^13.0.6
* clean-css-cli                 ^5.4.1  →   ^5.4.2
* eslint                        ^8.0.0  →   ^8.0.1
* karma                         ^6.3.4  →   ^6.3.5
* nodemon                      ^2.0.13  →  ^2.0.14
* postcss                       ^8.3.9  →  ^8.3.11
* rtlcss                        ^3.3.0  →   ^3.4.0
* sass                         ^1.42.1  →  ^1.43.2

Fix typo in RTL Hello World! sentence (#35236)

* docs(RTL): fix typo in Hello World! sentence

* chore(docs): Cspell arabic words update

tests: try to fix a few random failures (#35184)

* Change `Swipe` dispose spy on EventHandler
* Modal hide spy on backdrop hide

Change the way collapse handles its children on opening

collapse: extract duplicate code to a function

Update devDependencies (#35174)

* eslint                 ^7.32.0  →    ^8.0.0
* eslint-plugin-import   ^2.24.2  →   ^2.25.2
* eslint-plugin-unicorn  ^36.0.0  →   ^37.0.1
* vnu-jar                21.10.8  →  21.10.12

Switch to Node.js 16 and npm 8. (#35178)

JS: minor refactoring (#35183)

* add missing comments
* shorten block comments
* reorder constants
* reorder public/private methods
* sort exports alphabetically in util/index.js
* fix a couple of typos

Swipe: add test to ensure that it ignores `pinch` events (#35161)

Update nuget docs to specify only .NET Framework projects are supported. (#35124)

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

Update devDependencies (#35155)

* @babel/core                  ^7.15.5  →  ^7.15.8
* @babel/preset-env            ^7.15.6  →  ^7.15.8
* cspell                       ^5.12.0  →  ^5.12.3
* linkinator                   ^2.14.3  →  ^2.14.4
* vnu-jar                       21.9.2  →  21.10.8

Add color and border-color css variables to tables (#35055)

Extract Carousel's swipe functionality to a separate Class (#32999)

Reset z-index on .navbar-expand .offcanvas, plus prevent box-shadow issues (#35153)

tooltip.js: ignore a LGTM error (#35147)

The code on this line is either sanitized or the user chose to not sanitize it.

Merge remote-tracking branch 'remotes/origin/v513'

Bump version to 5.1.3.

Revert "Add `align-self: center` to buttons for improved rendering in flex containers" (#35143)

This reverts commit 94c80ff613dce6620591acd7f9930ce8f40c5ed0.

ESLint: disable `no-negated-condition` rule (#35137)

tests: minor cleanup (#35138)

* tests: minor cleanup

* tests: use the util noop function

scrollspy.js: chain functions (#35139)

Sanitizer: fix logic and add a test. (#35133)

This was broken in 2596c97 inadvertently.
Added a test so that we don't hit this in the future.

docs: remove 'and' in middle of list (#35113)

Add Sass variables for hr background-color and border

Co-Authored-By: Houssam Hammouda <11141564+devhoussam@users.noreply.github.com>

util/sanitizer.js: use `Array.every()` (#35120)

Use `for...of` in visual tests too

Enable `unicorn/numeric-separators-style` rule.

This is taken care of by babel via @babel/plugin-proposal-numeric-separator

Fix a `unicorn/no-array-callback-reference` issue

Enable `unicorn/no-array-for-each` rule

Enable `unicorn/no-for-loop` rule

Enable `unicorn/prefer-prototype-methods` rule

Separate container classes from `$enable-grid-classes` option (#35005)

* Separate container classes from enable-grid-classes optoin

* Document the new option

* Mention in migration guide

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

Bump postcss from 8.3.8 to 8.3.9 (#35110)

Bumps [postcss](https://github.com/postcss/postcss) from 8.3.8 to 8.3.9.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.3.8...8.3.9)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:development
  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>

Bump autoprefixer from 10.3.6 to 10.3.7 (#35109)

Bumps [autoprefixer](https://github.com/postcss/autoprefixer) from 10.3.6 to 10.3.7.
- [Release notes](https://github.com/postcss/autoprefixer/releases)
- [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/autoprefixer/compare/10.3.6...10.3.7)

---
updated-dependencies:
- dependency-name: autoprefixer
  dependency-type: direct:development
  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>

Bump linkinator from 2.14.0 to 2.14.3 (#35119)

Bumps [linkinator](https://github.com/JustinBeckwith/linkinator) from 2.14.0 to 2.14.3.
- [Release notes](https://github.com/JustinBeckwith/linkinator/releases)
- [Commits](https://github.com/JustinBeckwith/linkinator/compare/v2.14.0...v2.14.3)

---
updated-dependencies:
- dependency-name: linkinator
  dependency-type: direct:development
  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>

Release v5.1.2 (#35114)

Add workaround for dart sass compile error (#35033)

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

Add cSpell custom dictionary and docs-spellcheck script (#34940)

Fix capitalization after period in `contents.md` (#35111)

rollup: specify `generatedCode: 'es2015'` (#35070)

This will make use of `const` and modern features (which we are already using in our code) in the generated rollup code.

Revert "Bundlewatch: stop ignoring dependabot branches (#33192)" (#35069)

This reverts commit 5560c86070c8be08f76801bbc7e525eea4fc0cf6.

Update devDependencies (#35102)

* @popperjs/core                   ^2.10.1  →  ^2.10.2
* @rollup/plugin-commonjs          ^20.0.0  →  ^21.0.0
* clean-css-cli                     ^5.3.3  →   ^5.4.1
* eslint-config-xo                 ^0.38.0  →  ^0.39.0
* rollup                           ^2.57.0  →  ^2.58.0
* stylelint-config-twbs-bootstrap   ^2.2.3  →   ^2.2.4

Bump autoprefixer from 10.2.6 to 10.3.6 (#35076)

Bumps [autoprefixer](https://github.com/postcss/autoprefixer) from 10.2.6 to 10.3.6.
- [Release notes](https://github.com/postcss/autoprefixer/releases)
- [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/autoprefixer/compare/10.2.6...10.3.6)

---
updated-dependencies:
- dependency-name: autoprefixer
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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>

README.md: fix capitalization after period

Update terser to v5.9.0 (#34995)

sanitizer: add `sms` in the `SAFE_URL_PATTERN` (#35074)

Update README.md

update readme github issue new links

Fix syntax error, update description

Change from issue templates to issue forms

Co-Authored-By: XhmikosR <xhmikosr@gmail.com>
Co-Authored-By: GeoSot <geo.sotis@gmail.com>

Bump postcss-cli from 9.0.0 to 9.0.1 (#35075)

Bumps [postcss-cli](https://github.com/postcss/postcss-cli) from 9.0.0 to 9.0.1.
- [Release notes](https://github.com/postcss/postcss-cli/releases)
- [Changelog](https://github.com/postcss/postcss-cli/blob/master/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss-cli/compare/9.0.0...9.0.1)

---
updated-dependencies:
- dependency-name: postcss-cli
  dependency-type: direct:development
  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>

docs: remove `data-bs-dismiss="modal"` leftover (#35042)

Regenerate package-lock.json (#35071)

Use Sass variable instead of RGB components (#35017)

All other `rgba()` calls use a Sass variable, except for the ones fixed in this commit

docs: fix Popper version number in config (#35067)

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

Bump rollup from 2.56.3 to 2.57.0 (#35065)

Bumps [rollup](https://github.com/rollup/rollup) from 2.56.3 to 2.57.0.
- [Release notes](https://github.com/rollup/rollup/releases)
- [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rollup/rollup/compare/v2.56.3...v2.57.0)

---
updated-dependencies:
- dependency-name: rollup
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

Bump sass from 1.42.0 to 1.42.1 (#35064)

Bumps [sass](https://github.com/sass/dart-sass) from 1.42.0 to 1.42.1.
- [Release notes](https://github.com/sass/dart-sass/releases)
- [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sass/dart-sass/compare/1.42.0...1.42.1)

---
updated-dependencies:
- dependency-name: sass
  dependency-type: direct:development
  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>

Bump postcss from 8.3.6 to 8.3.8 (#35063)

Bumps [postcss](https://github.com/postcss/postcss) from 8.3.6 to 8.3.8.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.3.6...8.3.8)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:development
  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>

Bump postcss-cli from 8.3.1 to 9.0.0 (#35061)

Bumps [postcss-cli](https://github.com/postcss/postcss-cli) from 8.3.1 to 9.0.0.
- [Release notes](https://github.com/postcss/postcss-cli/releases)
- [Changelog](https://github.com/postcss/postcss-cli/blob/master/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss-cli/compare/8.3.1...9.0.0)

---
updated-dependencies:
- dependency-name: postcss-cli
  dependency-type: direct:development
  update-type: version-update:semver-major
...

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>

Bump @rollup/plugin-node-resolve from 13.0.4 to 13.0.5 (#35060)

Bumps [@rollup/plugin-node-resolve](https://github.com/rollup/plugins/tree/HEAD/packages/node-resolve) from 13.0.4 to 13.0.5.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/node-resolve/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/node-resolve-v13.0.5/packages/node-resolve)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-node-resolve"
  dependency-type: direct:development
  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>

Bump glob from 7.1.7 to 7.2.0 (#35059)

Bumps [glob](https://github.com/isaacs/node-glob) from 7.1.7 to 7.2.0.
- [Release notes](https://github.com/isaacs/node-glob/releases)
- [Changelog](https://github.com/isaacs/node-glob/blob/master/changelog.md)
- [Commits](https://github.com/isaacs/node-glob/compare/v7.1.7...v7.2.0)

---
updated-dependencies:
- dependency-name: glob
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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>

Add offcanvas to the components requiring JavaScript (#35013)

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

examples: Add `role="switch"` to switches (#35022)

Bump nodemon from 2.0.12 to 2.0.13 (#35066)

Bumps [nodemon](https://github.com/remy/nodemon) from 2.0.12 to 2.0.13.
- [Release notes](https://github.com/remy/nodemon/releases)
- [Commits](https://github.com/remy/nodemon/compare/v2.0.12...v2.0.13)

---
updated-dependencies:
- dependency-name: nodemon
  dependency-type: direct:development
  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>

Update devDependencies (#35026)

* @babel/cli    ^7.15.4  →  ^7.15.7
* sass          ^1.41.0  →  ^1.42.0

Bump nth-check from 2.0.0 to 2.0.1 (#35025)

Bumps [nth-check](https://github.com/fb55/nth-check) from 2.0.0 to 2.0.1.
- [Release notes](https://github.com/fb55/nth-check/releases)
- [Commits](https://github.com/fb55/nth-check/compare/v2.0.0...v2.0.1)

---
updated-dependencies:
- dependency-name: nth-check
  dependency-type: indirect
...

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

Fix Collapse regression of handling toggling between sibling chilldren (#34951)

Minor refactoring (#34980)

Rename variables and remove a temporary one