WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
162263
style.removeProperty not working for "overflow: hidden !important"
https://bugs.webkit.org/show_bug.cgi?id=162263
Summary
style.removeProperty not working for "overflow: hidden !important"
Mike Sidorov
Reported
2016-09-20 03:24:07 PDT
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
Comment 1
2016-09-20 09:43:17 PDT
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
Comment 2
2016-09-20 09:53:52 PDT
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)
Comment 3
2016-09-20 10:45:26 PDT
Seems similar to
bug 49059
.
Mike Sidorov
Comment 4
2016-12-20 23:32:13 PST
Still actual for Safari 10.
Simon Fraser (smfr)
Comment 5
2016-12-21 08:36:33 PST
Please test in the next Safari Tech Preview (20).
Michał Gołębiowski-Owczarek
Comment 6
2016-12-21 10:58:33 PST
(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)
Comment 7
2018-12-18 17:22:53 PST
I see: "hidden" "" so it seems this works now (STP 72).
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug