-
Notifications
You must be signed in to change notification settings - Fork 693
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
[selectors] More intuitive names for selector performance profiles #1694
Comments
Briefly discussed in today's working group meeting. |
manual vs auto ? |
The CSS Working Group just discussed The full IRC log of that discussion<dauwhe> Topic: renaming selector profiles<astearns> https://github.com//issues/1694 <dauwhe> astearns: dbaron added the issue, there was some discussion during lunch, but no resolution <dbaron> github: https://github.com//issues/1694 <dauwhe> ... let's try to put realistic suggestions in the issue |
Suggestion: 'snapshot' for script, 'tracking' for stylesheet. |
static vs live-updated ? |
inactive vs reactive ? |
There are some analogous concepts in DOM:
The obvious static vs. live seems just as bad as static vs. dynamic, but snapshot vs. live works. |
"Live" for CSS-related profile sounds great to me. But first of all, I would ask if the "non-live" profile — for just one selector — is really necessary? Limiting the I believe that the most use cases for Wasn't the whole concept of "non-live" selector profile a kind of premature optimization that was relevant in the era of slower CSS engines, but isn't very helpful in our days when the performance of CSS selectors is usually negligible (comparing to other parts of the rendering process)? Wouldn't it be better to drop it at all rather that to rename it? |
@SelenIT Of course lots of people would love You mention other selectors which depend on nested or following elements. An implementator would know better than me, but I'm not sure these are that expensive. For What I mean is that it can be reasonable to add some specific selectors to the live profile even if they depend on nested or following elements, but |
Yup, I used
Mostly it's just that elements don't tend to have very many siblings, so recomputing isn't too expensive. This is not the case for descendants, which elements often have a lot of (and some obvious use-cases for
It's been suggested (and vaguely approved of by bz, as in he said it wasn't obviously horrible) to have something like
Yup! |
Wait, isnt't it? I've been pretty sure it is held back by implementors (at least, according to the MDN's compatibility table). I suppose it might be because browser vendors aren't very interested in implementing it because the same result in JS can already be achieved with existing tools (including Selector API — for I didn't mean that the functionality of I like the idea of separate |
Agenda+ to consider the latest suggestion, in #1694 (comment) ; add a comment if there are other suggestions you think we should consider. :) |
…pshot” per Richard Gibson's suggestion in #1694 (comment)
The Working Group just discussed
The full IRC log of that discussion<dael> Topic: [selectors] More intuitive names for selector performance profiles<dael> github: https://github.com//issues/1694 <dael> fantasai: Latest was to use live for the css profile and snapshot for the one shot API calls. <aja> curious...they don't lie in @supports, too, do they? <dael> Wanted to ask if WG accepts that for if there's better ideas. <dael> florian: Sounds reasonable <dael> Rossen_: To me as well. <dael> dbaron: Do we have consensus that the snapshot profile will be used for things like query selector? <dael> fantasai: Thought it was but I'm not sure. <dael> Rossen_: Have we discussed this? Is there an issue about it? <dael> fantasai: Whole point of having that profile was to do that. I'm pretty sure given the number of times we talked about the name there was consesnus on having them. <dael> dbaron: I just remember it as a thing for selecotrs that print impl wanted. <dael> dbaron: impl producing pdfs, not interactive <dael> florian: My memory is this was for the selectors everyone wants but are deemed too slow to use in normal CSS. I think we made it forbidden to support in pdf impl. It needs to be web compat so if web can't they can't. <dael> dbaron: Makes sense. It just wasn't how I remembered the discussion. <dael> Rossen_: I suggest we resolve on issue of naming. dbaron if you feel the snapshot part needs further discussion let's bring that sep. <dael> Rossen_: In terms of naming, any obj? <dael> RESOLVED: Accept the names Live & Snapshot |
Thanks, “Live”/“Snapshot” are much more clear and intuitive than previous “Dynamic”/“Static”. |
The "dynamic" and "static" profile for selectors are renamed to "live" and "snapshot". https://www.w3.org/TR/selectors-4/#profiles See the CSSWG discussion/resolution at w3c/csswg-drafts#1694 Change-Id: Ic86ba4a87292e2ab7d425a1ff3223e00d7167efd Reviewed-on: https://chromium-review.googlesource.com/1015442 Reviewed-by: Hayato Ito <hayato@chromium.org> Commit-Queue: Takayoshi Kochi <kochi@chromium.org> Cr-Commit-Position: refs/heads/master@{#551966}
@Marat-Tanalin sent an email to www-style saying:
The text was updated successfully, but these errors were encountered: