Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[VarDumper] fix .sf-dump z-index (debug bar conflict)#16825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
hansstevens commentedDec 5, 2015
peterpacket commentedDec 5, 2015
KoenraadM commentedDec 5, 2015
99998 Still shows the dump over the bar in Chrome |
brpauwels commentedDec 5, 2015
wpottier commentedDec 5, 2015
after some investigation it's related to webkit animation that cause the animated element to ignore z-index behavior. (reports in chromium track :https://code.google.com/p/chromium/issues/detail?id=501596) so PR if fine. 👍 |
stof commentedDec 5, 2015
👍 |
nicolas-grekas commentedDec 5, 2015
👍 as bug fix on 2.7 |
stof commentedDec 5, 2015
Thank you@monsty. |
…oine LA)This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes#16825).Discussion----------[VarDumper] fix .sf-dump z-index (debug bar conflict)| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#16824| License | MIT| Doc PR | -Just changed the z-index because the twig dump is actually displaying over the debug bar (#16824).Commits-------a72efb9 [VarDumper] fixed .sf-dump z-index
beoboo commentedDec 5, 2015
Why is the z-index needed there? Removing the z-index property (for pre.sf-dump) at all the issue seems to be solved both in Chrome and Firefox... |



Just changed the z-index because the twig dump is actually displaying over the debug bar (#16824).