-
Notifications
You must be signed in to change notification settings - Fork 716
[css-anchor-position-1] position-area
: self
keyword inconsistency
#11371
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
For some perspective, Chome implements it as exactly as the syntax definition says, and the WPT tests reflect it. |
position-area
: self
keyword inconsistencyposition-area
: self
keyword inconsistency
The position fo the Put another way, just as "block-start" is a logical direction, "self-block-start" is a logical direction, and we're just combining "x-" with a logical direction. |
The CSS Working Group just discussed
The full IRC log of that discussion<fantasai> TabAtkins: We have both normal logical keywords and self-* logical keywords<fantasai> TabAtkins: using either containing block writing mode or box's own writing mode <fantasai> TabAtkins: Kiet pointed out that there's an inconsistency in the keywords about the placement of 'self' <fantasai> TabAtkins: This was intentional on our part, because self- is not modifying the axis, but the direction <fantasai> TabAtkins: x-self-start is combining physical direction and logical direction <fantasai> TabAtkins: whereas self-inline-start is using self writingmode for both axis and direction <fantasai> Kiet: Wanted to understand why it's the case. once I read Tab's comment, it makes sense <fantasai> Kiet: So I think it's fine to keep it like that <fantasai> Kiet: Another issue which is editorial is, in some places we use self-block and other places we use block-self <fantasai> TabAtkins: Oh, indeed we do! <fantasai> astearns: Is it just in anchorpos spec or others? <fantasai> TabAtkins: just here <fantasai> TabAtkins: but that is definitely an editorial mistake <fantasai> fantasai: That raised the question of what was intended. <fantasai> TabAtkins: Ohh, yeah. <fantasai> TabAtkins: I think it still needs to be self-block-start, since it's interpreting the axis also per the self writing mode <fantasai> TabAtkins: ... <fantasai> TabAtkins: We did a copy-paste error probably <ntim> I heard block as a verb :D <ntim> block self start <fantasai> astearns: Is that an argument to make the change? <fantasai> TabAtkins: just an editing error <fantasai> TabAtkins: more general argument that self-{block-start} is what we have here <fantasai> TabAtkins: other way around could imply you're taking the axis (block) from the container and the direction (start) from the self <fantasai> RESOLVED: x-self-start and self-block-start are correct, fix the spec |
The syntax definition of
position-area
says theself
version ofblock-start
,block-end
,inline-start
,inline-end
is the same keyword but with theself-
prefix (soself-block-start
,self-block-end
, ...).However in the section explaining what the keywords mean,
self
is placed in the middle of the keyword instead of (block-self-start
,block-self-end
, ...)So which of these is the correct convention to use?
This also leads to a relevant issue - in the syntax definition, we both have
self-
as the prefix (e.g:self-block-start
,self-inline-start
), andself-
in the middle (e.g:x-self-start
). Should we fix the syntax to be consistent whereself-
could be in the keyword?The text was updated successfully, but these errors were encountered: