-
Notifications
You must be signed in to change notification settings - Fork 659
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-writing-modes][css-pseudo][css-inline] text-combine-upright and initial-letter #653
Comments
Spec-wise, 9.1.2. Layout Rules has the following sentences:
I suppose initial-letter belongs to "other text layout purposes"? |
Selector matching and If we consider the whole run as a single U+FFFC, that would mean I think I'm ok with either solution 2 or 3, so fine. Testing a bit further how |
Ah, ok, sorry I was confused. So initial-letter is ok but not ::first-letter? My comment above was on initial-letter. |
One part of the problem is that as far as I can tell, initial-letter does not make sense if you apply it to part of the run of characters grouped by text-combine-upright. It can only make sense it you apply to all of it (or none of it). In turn, that problem may only be relevant if it is possible to select only a part of the run of characters grouped by text-combine-upright. I find the rules in https://drafts.csswg.org/css-writing-modes-3/#text-combine-runs hard to understand, so I think, but I am not sure, that it would not be possible to try and apply initial-letter (or any other property for that matter) to part of a run of characters grouped by text-combine-upright using a regular element (e.g.
If it is the case that you cannot apply a property to part of a run of characters grouped by text-combine-upright using a regular element, then it would make sense for
Either way, it is possible that the spec already defines this, but the text is tricky enough that I am not sure. My best bet is that Chrome is doing the right thing with If the specs are already defining this, rephrasing and/or more examples and/or more notes would be very welcome. If they're not, they should. |
Learned initial-letter spec quickly, apologies in advance if I missed something obvious. We should discuss For The
I think how I don't know how to solve this use case. One possible idea is to change |
Agree with Koji that the way this works isn't ideal for the use cases. The ideal solution here really is #2: (Koji, |
Note that |
From the response, I think I didn't explain well enough, sorry. Let me try to rewrite my points:
|
This looks reasonable to me
Maybe. I don't think it is a good, idea, but instead of having
Maybe we could try to extend that logic to earlier than layout, and since It's not obvious if this can happen at box creation time, but if we consider that the composition done by That said, even if we can make this work, I still prefer your suggestion above. |
I have a quick question:
I added double quotations around 13. In this situation, where these punctuations should be displayed? |
@realskk Not thinking form the spec, but from what's desirable, this is an interesting question. I am not sure why you think this is specific to initial letter though. Don't you have the exact same problem of quote orientation with regularly sized TCY? Moreover, the initial letter probably means you cannot get either of the results you showed, since the initial letter has to be on the first inline-level child of the block container, and in your example, there is something before it: the quote. Initial letter can have quotes inside it, but not before it. If we leave initial letter aside, and want to deal with automatic quote orientation, and presuming that you find the first example nicer, we'd have to deal with a codepoint based solution, since |
Looking at
Does this look correct? /cc editors of css-pseudo-4: @fantasai @astearns @therealglazou |
@realskk 's comment looks like a different issue, could you raise it separately? |
Ah, yes. My question applies to regular TCY situation also. As @kojiishi wrote, I'll raise another issue. |
This issue was closed, no change, by the CSSWG in Seattle. |
It is not clear what this should do
::first-letter
should match the1
at the beginning of the<p>
, but that means it selects half of the text-combine-upright, and I have no idea of how you'd turn that into a 3-line drop cap.I think there are three changes we could make to the specs to deal with that:
initial-letter
has no effect when applied to some-but-not-all the characters in a run of characters grouped by text-combine-upright::first-letter
needs to treat a run of characters grouped by text-combine-upright as single character and select all of it if it is the first thing in the element.::first-letter
does not select anything if the element starts with run of characters grouped by text-combine-uprightJust for the record, I believe that text-combine-upright itself isn't an issue, and that the following ought to work just fine, highlighting that the issue is just due to selecting part of this orthogonal bit of content.
The text was updated successfully, but these errors were encountered: