-
Notifications
You must be signed in to change notification settings - Fork 143
[css-typed-om] Describe that StylePropertyMaps are not live objects #149
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
What does this mean? If a CSS declaration or inline style is otherwise modified (say, through JS), does the specified or inline StylePropertyMap update? What happens if we then call StylePropertyMap::append, for example? Do we append to the declaration or inline style based on its current value while appending to the StylePropertyMap's value based on what it was created with? What does the iterator give us for that property? |
Found a definition of "live object" here: https://dom.spec.whatwg.org/#concept-collection-live . |
To clarify, this is saying that StylePropertyMaps won't update based on style changes from other sources:
It might actually be reasonably simple to support live maps for specified objects but for computed styles this gets quite expensive, and it seems best to be consistent. |
feedback from jyc on irc: this should probably be true only for computed maps, not specified maps. |
Ah, that's probably fine. I was getting a little confused why .set() worked but wasn't live - there's clearly still a communication channel there. (This doesn't really need to be "live", per se, it's just that .get() goes and gets the value off of the element.) So yeah, the Readonly version is non-live. |
Resolved to be live for specified and for computed style maps. |
The only map that's left is the inline map. We didn't specifically talk about this during the spetember f2f but it seems it should be live too. OK for me to close this issue as invalid? |
The CSS Working Group just discussed #149, and agreed to the following resolutions:
The full IRC log of that discussion
|
No description provided.
The text was updated successfully, but these errors were encountered: