-
Notifications
You must be signed in to change notification settings - Fork 661
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
[css-pseudo] highlights and decoration propagation #6829
Labels
Comments
@frivoal, @fantasai, any thoughts? We are starting to implement this in Blink (e.g. CL:3344847), but it would be good to agree on how this should work before we enable it everywhere. |
The CSS Working Group just discussed
The full IRC log of that discussion<TabAtkins> Topic: highlights and decoration propagation<TabAtkins> github: https://github.com//issues/6829 <TabAtkins> delan: some background, text-decs have this thing called decoration propagation <TabAtkins> delan: which is normally how descendants ensure they're also decorated <TabAtkins> delan: the text-dec props aren't inherited normally, they use this propagation instead <TabAtkins> delan: however, with highlight pseudos, we've saying all properties are inherited, even if they're not normally defined to inherit <TabAtkins> delan: So how do these two things interact? Do we still have decor propagation anyway? <TabAtkins> delan: If we do, it seems like you'd get a second underline, which might not be desirable <TabAtkins> delan: on the other hand, if we say no, then we'll be relying on inheritance to ensure children are decorated, which also has negative side effects <TabAtkins> delan: So: do decorations propagate into highlight pseudos? <astearns> ack fantasai_ <TabAtkins> fantasai: I think when decors propagate it's from one element to another, and for highlight pseudos, each piece of a highlight pseudo is applying decors on its own <TabAtkins> fantasai: so i think they don't propagate if they're set above the highlight pseudo <TabAtkins> fantasai: say you ahve a <p> with an <em> <TabAtkins> fantasai: If you decor the <p> if propagates to the <em> <TabAtkins> fantasai: if you highlight across it, there'll be two pieces, inside and outside the em <iank_> what is the interaction when highlights come across an inline which is a stacking-context? <TabAtkins> fantasai: each one will have an underline decl, you don't want it to be twice. you don't want to propagate since each piece takes care of the underline itself <TabAtkins> fantasai: I think that's what happens, but might give different results. <TabAtkins> fantasai: Propagation and inheritance can give different thickness <TabAtkins> fantasai: but we probably want it to look like a normal underline, so we *do* want it to propagate... <TabAtkins> delan: for example, a <sup> insdie the <p>. Normally there's a single underline that doesn't jump up. (Blink does jump up, but we do it wrong.) <TabAtkins> delan: We don't actually do the decorating box, we do an inheritance hack. <TabAtkins> fantasai: I think what you want is to render as if it's propagated, but the existence of the underline is based on inheritance, but then...... <TabAtkins> fantasai: Dunno if this makes sense. Inherits like every other property, but instead of each part drawing its own, they look up to see if they would propagate from their parent. <florian> q+ <TabAtkins> fantasai: I think that would get us the results we want. <astearns> ack florian <TabAtkins> florian: I agree in general. I wonder if it's that bad in the case of highlight pseudos if we do jump up and down. <TabAtkins> florian: In a whole paragraph with some <sup> you do want something continuous and homogenous. <TabAtkins> florian: But in highlights it might depend on hwo you get there. If you highlight the whole para, yeah you might want it to be the same. But if you start within the sup, maybe you do want it to be up. And as you drag out to the normal text, maybe the whole highlight moves down, or just the part in the normal text. <TabAtkins> florian: it's not hierarchical. <TabAtkins> florian: So if we don't fix the inconsistency for highlights, maybe that's fine, unlike in the normal case. <TabAtkins> delan: If we rely on inheritance, wouldn't we... <TabAtkins> delan: Now that I think about it, I'm not sure that saying there's not propagation is really all that feasible. <TabAtkins> delan: Becuase the parent will draw its decoration, and there will have to be additional work done to ensure that it stops painting its decor when there's a child. that would still require changes to impls. <TabAtkins> astearns: Ian had a question about fi there's anything interesting in the interaction with stacking contexts? <TabAtkins> iank_: if you've got an inline which is self-painting, a stacking context, is there any interesting interaction there? <TabAtkins> delan: It sounds like if we relied on inheritance to "propagate", then that would cause problems. <TabAtkins> delan: Is how it's supposed to work normally that when you have a stacking context propagation doesn't happen? <TabAtkins> iank_: Unsure, is why I was asking. <florian> q+ <TabAtkins> iank_: Is it possible to have inlines that paint independenty from the rest of the inline content, and if there ar eimplications. <TabAtkins> iank_: Fine if we don't have an answer. <TabAtkins> fantasai: When you're doing hihglighting normally, if you're hilighting the para and there's an inline-block inside, i don't think you highlight the inline-block, you just skip it or run the line across it, depending. <TabAtkins> (all the "highlight"s mean underlines) <TabAtkins> fantasai: So maybe it just makes sense to use the inheritance model even if it doesn't work as well for sups. <TabAtkins> florian: One thing I wanted to ask - is the worry you have actually going to pan out like this? <TabAtkins> florian: Unlike the normal text-decor we're not applying to the whole para. <fantasai_> fantasai^: because in the case of highlights, we'd want to go into the inline block to indicate what's highlighted <TabAtkins> florian: If we highliht from the start of the para, thru a superscript, to the end of the para, we actually have three highlight pseudos, right? <iank_> specificially thinking about inlines which are self painting, e.g. <div>text <span style="position: relative; z-index: 2; opacity: 0.5; background: lime;">self<br>painting</span> <astearns> q+ <TabAtkins> florian: You're not highlighting the parent "behind" the sup, you're highlighting three adjacent chunks <astearns> ack florian <TabAtkins> delan: That's how we do it in Blink, we paint chunks. <TabAtkins> delan: Not sure if that's correct. <TabAtkins> iank_: at the IFC level we have a data structure that determines where to paint decoarations for the whole IFC <TabAtkins> iank_: We can chat offline about this. <TabAtkins> astearns: So what I heard was, there is a reason for text-decors in highlights to render differently than in normal elements <TabAtkins> astearns: At first I thought this was wrong, but the scenario about starting a highlight in a sup and then dragging out of it, means the highlight decors change as you drag. I think that's wrong, they should be stable as you do the highlight gesture. <TabAtkins> astearns: So that suggests to me we should only be using inheritance, no propagation. <TabAtkins> delan: Not sure I follow, that's not a problem outside of highlight land <fantasai_> astearns++ <TabAtkins> astearns: Outside of highlights you have the decor set on an element, and it propagates to the children, and there's a stable rendering of that <TabAtkins> astearns: In the highlighting gesture you can get an unstable decoration state if you're using propagation painting rules <TabAtkins> delan: Agree that's not desirable. What if we went the other way and said they do have propagation, but the decoration properties aren't inherited? <florian> q+ <astearns> ack astearns <TabAtkins> delan: Everything else continues inheriting, so things like background-color works. But not text-decor props. Would that work? <TabAtkins> astearns: I think that might still ahve problems. Say I start a highlight within a sup. How do you assess the propagation rules? You don't know how far up the parent chain to go, so you just propagate from the sup. Now you drag it out, you've got a shared parent, the propagation changes? <TabAtkins> delan: I think we could use the same rules as for originating elements. <TabAtkins> delan: Whether you've highlighted something determines whether we actualy paint the decor there. <TabAtkins> delan: But we actually figur eit out as if highlighted <astearns> ack astearns <TabAtkins> florian: Abspos example - if you highlight something that's been absposed elsewhere. <TabAtkins> florian: Classically, when you underline a para, the abspos child doesn't get underlined. <TabAtkins> florian: Here if you start the selection in the abspos you want to see the highlight styles in the abspos, and if you drag into the para text you don't want it to suddeny change. <TabAtkins> florian: In general we're just not highlighting one span of text, we're highlighting lots of bits. <astearns> s/At first I thought this was wrong/At first I thought the answer was no/ <TabAtkins> delan: That makes sense <florian> q- <TabAtkins> astearns: So I think we can get a resolution to have text-decor properties in highlight pseudos inherit only, not propagate any decorations? <TabAtkins> delan: Sounds good. <TabAtkins> RESOLVED: Highlight pseudos continue to inherit the text-decor properties, but do not propagate decorations the normal document. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When selecting the following (demo):
We want to be careful not to inadvertently favour the incorrect decoration impl in Blink^ and WebKit, where descendants paint propagated decorations as if they were specified directly — equivalent to 3(b) in non-highlight content.
(cc @frivoal, @fantasai, @mrego, @MatsPalmgren)
^ fixed for underlines in Chromium 105
The text was updated successfully, but these errors were encountered: