Skip to content

Fixed z-index calculation error on IE11#1

Open
allenpiscitello wants to merge 56 commits intomasterfrom
patch-IE-11-z-offset
Open

Fixed z-index calculation error on IE11#1
allenpiscitello wants to merge 56 commits intomasterfrom
patch-IE-11-z-offset

Conversation

@allenpiscitello
Copy link
Copy Markdown
Owner

In IE11, the zIndex is returned as a number, which has parentsZIndex returned as an empty array. This empty array has a max value applied, which is -Infinity, which causes the zIndexOffset to not be applied.

Chrome and other browsers accidentally get correct behavior since the zIndex property is returned a string, which makes !== 0 return true.

To unify the behavior, I added parseInt to the comparison, and set a default value of the parentsZIndex to 0, so that the max value is always at least 0.

jelhan and others added 30 commits August 18, 2016 14:16
Use a notation usable with XHTML
In IE11, the zIndex is returned as a number, which has parentsZIndex returned as an empty array.  This empty array has a max value applied, which is -Infinity, which causes the zIndexOffset to not be applied.

Chrome and other browsers accidentally get correct behavior since the zIndex property is returned a string, which makes !== 0 return true.

To unify the behavior, I added parseInt to the comparison, and set a default value of the parentsZIndex to 0, so that the max value is always at least 0.
[Doc] Keyboard navigation is not supported for embedded / inline type
Occitan : spoken in South France, North Spain and Italy, and Monaco.
just a little change, it's uèi and not ùèi.
resolves bug on days when DST is added.
* Adding Canadian English locale file

Adding Canadian English locale file

* Fixed missing EOF CR
Stichoza and others added 26 commits October 20, 2016 11:10
…shows-wrong-month

Improve defaultViewDate documentation
…ate-multiple-dates

Add notes about multiple dates and examples for update method
Added missing basque language properties
Week starts on monday in Slovenia
Added weekStart to slovenian translation
Fix placeholder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.