- From: CSS Meeting Bot via GitHub <sysbot+gh@w3.org>
- Date: Tue, 07 Nov 2017 17:20:43 +0000
- To: public-css-archive@w3.org
The Working Group just discussed `logical/physical property cascade and the .style API`, and agreed to the following resolutions: * `RESOLVED` * `RESOLVED: Make it so that setters to the .style api are appended rather than put in place` <details><summary>The full IRC log of that discussion</summary> <gregwhitworth> Topic: logical/physical property cascade and the .style API<br> <astearns> github: https://github.com/w3c/csswg-drafts/issues/1898<br> <gregwhitworth> TabAtkins: someone pointed out that there are problems with the way logical properties are defined when trying to use the DOM APIs<br> <gregwhitworth> TabAtkins: they get resolved in the normal order and then go through the cascade and we deal with the physical and logical based on ordering and writing mode<br> <gregwhitworth> TabAtkins: when you set a property it gets appended to the list of properties in the style attribute<br> <gregwhitworth> TabAtkins: and if you set one that's already there it just replaces it<br> <gregwhitworth> TabAtkins: so this can cause it to be out of sync with what would happen in the cascade<br> <gregwhitworth> TabAtkins: the proposal is to amend the CSSOM that if something is there it is always appended<br> <gregwhitworth> TabAtkins: related to this<br> <gregwhitworth> TabAtkins: we had an issue with !important and we resolved to append to make this work<br> <gregwhitworth> dbaron: I'm happy to make that change<br> <gregwhitworth> dbaron: we had a long discussion and resolved the opposite way<br> <gregwhitworth> dbaron: a setter should append<br> <gregwhitworth> dbaron: at the time implementations were inconsistent due to optimizations<br> <gregwhitworth> dbaron: for existing properties in the fast path they would replace and others they would append<br> <gregwhitworth> dbaron: one question, would having to do that mess up optimizations?<br> <gregwhitworth> TabAtkins: when we discussed it internally we were fine making the change<br> <gregwhitworth> dbaron: I found the minutes from 2013 but I'm looking for the other one<br> <gregwhitworth> astearns: so I'm clear, the suggestion to append or remove and then append?<br> <gregwhitworth> TabAtkins: those are equivelant<br> <gregwhitworth> dbaron: I think it may turn out that it's not equivelant in the future<br> <gregwhitworth> TabAtkins: we should figure out which one we want, it will need to have a note to determine this<br> <gregwhitworth> dholbert: you could inspect the style attr and determine that way<br> <gregwhitworth> TabAtkins: the current model limits to only one<br> <gregwhitworth> astearns: I'm hearing two engines happy to change<br> <gregwhitworth> astearns: any compat concerns?<br> <gregwhitworth> dbaron: possibly<br> <gregwhitworth> astearns: the compat issue would be limited to logical props shorthands?<br> <gregwhitworth> TabAtkins: no<br> <gregwhitworth> fantasai: it would only affect people looking at .style expecting a particular order<br> <gregwhitworth> fantasai: which doesn't make much sense, so it's probably not very common<br> <gregwhitworth> florian: I doubt it's common<br> <gregwhitworth> Rossen: tools and editors might be doing this<br> <gregwhitworth> TabAtkins: I'd be surprised if the editor was reading the style and writing to .style<br> <gregwhitworth> Rossen: they could<br> <gregwhitworth> astearns: The proposed resolution is to change CSSOM to append values via the .style API and add a note<br> <gregwhitworth> TabAtkins: yeah, and try to figure that out in the future<br> <dbaron> Things I found were https://lists.w3.org/Archives/Public/www-style/2013Sep/0469.html and https://lists.w3.org/Archives/Public/www-style/2013Oct/0007.html but I think there was further followup after the latter<br> <gregwhitworth> astearns: any other opinions? objections?<br> <gregwhitworth> RESOLVED<br> <gregwhitworth> RESOLVED: Make it so that setters to the .style api are appended rather than put in place<br> </details> -- GitHub Notification of comment by css-meeting-bot Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/1898#issuecomment-342556321 using your GitHub account
Received on Tuesday, 7 November 2017 17:20:47 UTC