diff options
author | Dominik Kremer <dominik.kremer@yatta.de> | 2020-05-10 16:59:22 +0300 |
---|---|---|
committer | XhmikosR <xhmikosr@gmail.com> | 2020-05-12 07:51:09 +0300 |
commit | d59de33fc09b83c708d40690a595eab64aca0b30 (patch) | |
tree | 122267e37c5407053a7c26281a7dc389837db809 /js/tests/unit/modal.spec.js | |
parent | 3e7566f2c3f0d7dff368e16c0427a13900f3cf84 (diff) |
Add a test about the scrollbar issue on non-integer width
Diffstat (limited to 'js/tests/unit/modal.spec.js')
-rw-r--r-- | js/tests/unit/modal.spec.js | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/js/tests/unit/modal.spec.js b/js/tests/unit/modal.spec.js index ff8102f444..a72e93ca85 100644 --- a/js/tests/unit/modal.spec.js +++ b/js/tests/unit/modal.spec.js @@ -649,6 +649,38 @@ describe('Modal', () => { modal.show() }) + it('should not adjust the inline body padding when it does not overflow, even on a scaled display', done => { + fixtureEl.innerHTML = '<div class="modal"><div class="modal-dialog"></div></div>' + + const modalEl = fixtureEl.querySelector('.modal') + const modal = new Modal(modalEl) + const originalPadding = window.getComputedStyle(document.body).paddingRight + + // Remove body margins as would be done by Bootstrap css + document.body.style.margin = '0' + + // Hide scrollbars to prevent the body overflowing + document.body.style.overflow = 'hidden' + + // Simulate a discrepancy between exact, i.e. floating point body width, and rounded body width + // as it can occur when zooming or scaling the display to something else than 100% + document.documentElement.style.paddingRight = '.48px' + + modalEl.addEventListener('shown.bs.modal', () => { + const currentPadding = window.getComputedStyle(document.body).paddingRight + + expect(currentPadding).toEqual(originalPadding, 'body padding should not be adjusted') + + // Restore overridden css + document.body.style.removeProperty('margin') + document.body.style.removeProperty('overflow') + document.documentElement.style.paddingRight = '16px' + done() + }) + + modal.show() + }) + it('should enforce focus', done => { fixtureEl.innerHTML = '<div class="modal"><div class="modal-dialog"></div></div>' |