Skip to content

Prevent overflowing static backdrop modal animation#30326

Merged
XhmikosR merged 13 commits into
mainfrom
master-ysds-static-modal-animation
Jun 25, 2020
Merged

Prevent overflowing static backdrop modal animation#30326
XhmikosR merged 13 commits into
mainfrom
master-ysds-static-modal-animation

Conversation

@ysds

@ysds ysds commented Mar 2, 2020

Copy link
Copy Markdown
Contributor

Fixes #30322

This is a JavaScript fix for respects the current style, but also can be changed to shrink animation by CSS.

demo: https://codepen.io/fellows3/pen/vYOZwqR

Comment thread js/src/modal.js
@Johann-S

Johann-S commented Mar 9, 2020

Copy link
Copy Markdown
Member

I would love to see a unit test @ysds, do you think it's possible? 🤔

@ysds ysds force-pushed the master-ysds-static-modal-animation branch from f2e9781 to 84de27f Compare March 13, 2020 18:59
@ysds

ysds commented Mar 13, 2020

Copy link
Copy Markdown
Contributor Author

@Johann-S I tried add unit a test, but sorry I don't understand why fail on Firefox. Done!

@ysds ysds force-pushed the master-ysds-static-modal-animation branch from bf9f95f to 6bf1beb Compare March 16, 2020 13:44
@XhmikosR XhmikosR requested a review from Johann-S March 16, 2020 14:31
@ysds ysds force-pushed the master-ysds-static-modal-animation branch from d7c530a to 2bf7c2c Compare March 18, 2020 12:38
Comment thread js/src/modal.js
@XhmikosR

XhmikosR commented Jun 4, 2020

Copy link
Copy Markdown
Member

@ysds ping :)

@mdo mdo changed the base branch from master to main June 16, 2020 20:08
@XhmikosR XhmikosR requested a review from Johann-S June 23, 2020 06:26

@Johann-S Johann-S left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@XhmikosR

Copy link
Copy Markdown
Member

This will probably need to be manually backported in v4-dev, so any help welcome :)

@XhmikosR XhmikosR merged commit fb4efb4 into main Jun 25, 2020
@XhmikosR XhmikosR deleted the master-ysds-static-modal-animation branch June 25, 2020 08:35
XhmikosR added a commit that referenced this pull request Jun 25, 2020
Prevent overflowing static backdrop modal animation

TODO: backport the test too
ysds added a commit that referenced this pull request Jun 25, 2020
XhmikosR pushed a commit that referenced this pull request Jun 30, 2020
XhmikosR added a commit that referenced this pull request Jul 6, 2020
Prevent overflowing static backdrop modal animation

TODO: backport the test too
XhmikosR pushed a commit that referenced this pull request Jul 6, 2020
dsynr added a commit to dsynr/bootstrap that referenced this pull request Aug 21, 2020
* Remove backdrop-filter from toasts

* BrowserStack: test on Edge 15

* Backport twbs#31135

* Move color utility callouts to start of page

Hierarchically/structurally, in the position they are currently at, the two callouts seem like they "belong" just to the "background color" section. Moving them to the start makes it clearer that those two callouts relate to everything in the page (both "Color" and "Background color" classes.

* Change heading level

otherwise the assistive technology callout looks like it's hierarchically under the "Dealing with specificity" heading

* Backport twbs#30326

Prevent overflowing static backdrop modal animation

TODO: backport the test too

* Backport twbs#30326 (Unit test)

* Update Babel config (twbs#31011)

* remove plugin-proposal-object-rest-spread
* add bugfixes
* `exclude: ['transform-typeof-symbol']` did nothing with our config

* Update devDependencies and gems

* Update dependencies, gems and regenerate package-lock.json (twbs#31261)

* @rollup/plugin-node-resolve 8.1.0
* popper.js 1.16.1
* qunit 2.10.1
* rollup 2.21.0

* Docs: forms accessibility cleanup (backport from v5) (twbs#31234)

* Expand advice for anchor-based controls

* Expand accessibility note in input group

* Correct statement about validation, fix server example

* Tweak label > accessible name

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

* Turn off scroll anchoring for accordions (twbs#31347)

New default behavior for scroll anchoring (rolled out in Chrome 84?) leads to unsightly/odd accordion interactions - see twbs#31341
This rule suppresses this new behavior and reverts back to the old way.

See https://drafts.csswg.org/css-scroll-anchoring/

* Update to Ruby 2.7/Bundler 2.x. (twbs#31296)

* Clear timeout before showing the toast (twbs#31155)

* clear timeout before showing the toast

* Add unit test

* Remove the check for timeout

* Check for clearTimeout to have been called

Co-authored-by: XhmikosR <xhmikosr@gmail.com>
# Conflicts:
#	js/tests/unit/toast.spec.js

* Add unit test for toast to check clearTimeout to have been called (twbs#31298)

* docs(skippy): prevent skip links from overlapping header

* Backport twbs#31344

Add toasts to the components requiring JavaScript

* Update devDependencies and gems

* @babel/cli                   ^7.10.4  →  ^7.11.0
* @babel/core                  ^7.10.4  →  ^7.11.0
* @rollup/plugin-babel          ^5.0.4  →   ^5.1.0
* @rollup/plugin-commonjs      ^13.0.0  →  ^14.0.0
* @rollup/plugin-node-resolve   ^8.1.0  →   ^8.4.0
* autoprefixer                  ^9.8.4  →   ^9.8.6
* eslint                        ^7.4.0  →   ^7.6.0
* karma                         ^5.1.0  →   ^5.1.1
* rollup                       ^2.21.0  →  ^2.23.0

* Remove overflow: hidden from toasts (twbs#31381) (twbs#31407)

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

* Backport twbs#31339 (twbs#31414)

* Backport twbs#31339

Add view on GitHub links for easier content editing from the docs

* Prepare v4.5.1. (twbs#31408)

* Remove flex: 1 0 100% from rows (twbs#31439) (twbs#31445)

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

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

* Restore make-container-max-widths mixin

* Deprecate the `make-container-max-widths` mixin

* Remove undefined `$ignore-warning`

* Prepare v4.5.2. (twbs#31444)

Co-authored-by: Mark Otto <markd.otto@gmail.com>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>
Co-authored-by: ysds <ysds.code@gmail.com>
Co-authored-by: Patrick H. Lauke <redux@splintered.co.uk>
Co-authored-by: Rohit Sharma <rohit2sharma95@gmail.com>
Co-authored-by: Gaël Poupard <gael.poupard@orange.com>
Co-authored-by: Martijn Cuppens <martijn.cuppens@gmail.com>
@roxas0zero

Copy link
Copy Markdown

this issue is still happening for Chrome and Edge(Chromium) but not in FireFox, I'm using bootstrap 4.5.2

olsza pushed a commit to olsza/bootstrap that referenced this pull request Oct 3, 2020
Co-authored-by: XhmikosR <xhmikosr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Click on Static Modal Backdrop Scaling up the Modal Content and Adding Scrollbar to Backdrop

6 participants