rooty/Bootstrap
0
0
Fork 0
mirror of https://github.com/twbs/bootstrap.git synced 2025-04-10 02:02:26 +02:00
Code Issues Actions 12 Packages Releases Activity

Consistent usage of $(document.body) instead of $('body') (#25671)

Browse Source
This commit is contained in:
Martijn Cuppens 2018-02-21 09:01:02 +01:00 committed by Johann-S
parent 99379f3843
commit 8cfbf6933b
3 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
js/src/dropdown.js
View File

@ -183,7 +183,7 @@ const Dropdown = (($) => {
// https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html // https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html
if ('ontouchstart' in document.documentElement && if ('ontouchstart' in document.documentElement &&
$(parent).closest(Selector.NAVBAR_NAV).length === 0) { $(parent).closest(Selector.NAVBAR_NAV).length === 0) {
$('body').children().on('mouseover', null, $.noop) $(document.body).children().on('mouseover', null, $.noop)
} }
this._element.focus() this._element.focus()
@ -365,7 +365,7 @@ const Dropdown = (($) => {
// If this is a touch-enabled device we remove the extra // If this is a touch-enabled device we remove the extra
// empty mouseover listeners we added for iOS support // empty mouseover listeners we added for iOS support
if ('ontouchstart' in document.documentElement) { if ('ontouchstart' in document.documentElement) {
$('body').children().off('mouseover', null, $.noop) $(document.body).children().off('mouseover', null, $.noop)
} }
toggles[i].setAttribute('aria-expanded', 'false') toggles[i].setAttribute('aria-expanded', 'false')

8
js/src/modal.js
View File

@ -453,8 +453,8 @@ const Modal = (($) => {
// Adjust body padding // Adjust body padding
const actualPadding = document.body.style.paddingRight const actualPadding = document.body.style.paddingRight
const calculatedPadding = $('body').css('padding-right') const calculatedPadding = $(document.body).css('padding-right')
$('body').data('padding-right', actualPadding).css('padding-right', `${parseFloat(calculatedPadding) + this._scrollbarWidth}px`) $(document.body).data('padding-right', actualPadding).css('padding-right', `${parseFloat(calculatedPadding) + this._scrollbarWidth}px`)
} }
} }
@ -476,9 +476,9 @@ const Modal = (($) => {
}) })
// Restore body padding // Restore body padding
const padding = $('body').data('padding-right') const padding = $(document.body).data('padding-right')
if (typeof padding !== 'undefined') { if (typeof padding !== 'undefined') {
$('body').css('padding-right', padding).removeData('padding-right') $(document.body).css('padding-right', padding).removeData('padding-right')
} }
} }

4
js/src/tooltip.js
View File

@ -317,7 +317,7 @@ const Tooltip = (($) => {
// only needed because of broken event delegation on iOS // only needed because of broken event delegation on iOS
// https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html // https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html
if ('ontouchstart' in document.documentElement) { if ('ontouchstart' in document.documentElement) {
$('body').children().on('mouseover', null, $.noop) $(document.body).children().on('mouseover', null, $.noop)
} }
const complete = () => { const complete = () => {
@ -375,7 +375,7 @@ const Tooltip = (($) => {
// If this is a touch-enabled device we remove the extra // If this is a touch-enabled device we remove the extra
// empty mouseover listeners we added for iOS support // empty mouseover listeners we added for iOS support
if ('ontouchstart' in document.documentElement) { if ('ontouchstart' in document.documentElement) {
$('body').children().off('mouseover', null, $.noop) $(document.body).children().off('mouseover', null, $.noop)
} }
this._activeTrigger[Trigger.CLICK] = false this._activeTrigger[Trigger.CLICK] = false