Bug 162263
| Summary: | style.removeProperty not working for "overflow: hidden !important" | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Mike Sidorov <mikes.ekb> |
| Component: | CSS | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | koivisto, m.goleb+bugzilla, simon.fraser |
| Priority: | P2 | ||
| Version: | Safari 9 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=49059 https://bugs.webkit.org/show_bug.cgi?id=129994 |
||
Mike Sidorov
http://jsbin.com/requfofayi/edit?js,console,output
Try to set "overflow: hidden !important" style property and then call style.removeProperty('overflow');
No error if thrown but still everything remains intact.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexey Proskuryakov
Mike, can you clarify what the expected behavior is for the test case? It seems like displaying red on button click is expected, and what the test actually verifies is getting the following console output after two clicks:
"hidden"
""
Is this accurate?
Mike Sidorov
Alexey, sorry, had to explain this earlier.
The test case is: clicking a button toggles to properties - background-color: red and overflow: hidden. These properties are being set/removed on every button click.
With background-color everything is fine - it gets set to red and removed on the next click.
But with overflow there's an issue: once you set it, you cannot remove it at all. I'm getting the following output in the console:
"hidden"
"hidden"
What's expected is to remove that property (to be displayed "hidden" - "").
Simon Fraser (smfr)
Seems similar to bug 49059.
Mike Sidorov
Still actual for Safari 10.
Simon Fraser (smfr)
Please test in the next Safari Tech Preview (20).
Michał Gołębiowski-Owczarek
(In reply to comment #5)
> Please test in the next Safari Tech Preview (20).
WebKit Nightly is still affected so I assume next Safari TP won't change anything.
Simon Fraser (smfr)
I see:
"hidden"
""
so it seems this works now (STP 72).