This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 28697 - BarProp.visible can be readonly
Summary: BarProp.visible can be readonly
Status: RESOLVED FIXED
Alias: None
Product: WHATWG
Classification: Unclassified
Component: HTML (show other bugs)
Version: unspecified
Hardware: Other other
: P3 normal
Target Milestone: Unsorted
Assignee: Ian 'Hixie' Hickson
QA Contact: contributor
URL: https://html.spec.whatwg.org/#browser...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-25 14:43 UTC by contributor
Modified: 2015-09-01 20:28 UTC (History)
4 users (show)

See Also:


Attachments

Description contributor 2015-05-25 14:43:00 UTC
Specification: https://html.spec.whatwg.org/
Multipage: https://html.spec.whatwg.org/multipage/#browser-interface-elements
Complete: https://html.spec.whatwg.org/#browser-interface-elements
Referrer: 

Comment:
BarProp.visible can be readonly

Posted from: 212.116.72.179
User agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/44.0.2376.0 Safari/537.36 OPR/31.0.1857.0 (Edition developer)
Comment 1 Philip Jägenstedt 2015-05-25 14:45:45 UTC
The spec says "On setting, the new value must be discarded" but in Blink and WebKit, BarProp.visible is simply readonly. Since this appears to be web compatible, I propose doing the same in the spec.
Comment 2 Anne 2015-09-01 10:13:28 UTC
It seems in Gecko the setter only does something in Chrome code. So I guess Firefox could make this readonly too, unless I read http://mxr.mozilla.org/mozilla-central/source/dom/base/BarProps.cpp#68 incorrectly. Copying bz for verification.
Comment 3 Boris Zbarsky 2015-09-01 17:29:47 UTC
We could do that, yes.  It's a behavior change in strict mode (to throw instead of silently doing nothing), but as long as we're sure that's web compatible...
Comment 4 Anne 2015-09-01 17:50:38 UTC
Yeah, that seems like a small risk given that Chrome already implements it this way and this is very old.

Thank you.

https://github.com/whatwg/html/commit/78f102944f276919251d8a057052edbca4ee1e0c
Comment 5 Boris Zbarsky 2015-09-01 20:28:14 UTC
Actually, we'll need to figure out how to implement this in Gecko... it'll take some nontrivial binding surgery to do it right, unfortunately.  :(