Skip to content
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-position-3] link to absolute-position containing block #11239

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gitspeaks
Copy link

In this context, 'the original containing block' is the 'absolute-position containing block'

@gitspeaks gitspeaks changed the title link to absolute-position containing block [css-position-3] link to absolute-position containing block Nov 19, 2024
@w3cbot
Copy link

w3cbot commented Nov 19, 2024

svgeesus marked as non substantive for IPR from ash-nazg.

Copy link
Contributor

@Loirooriol Loirooriol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you changing CSS2 in a PR that says [css-position-3]?

AFAIK CSS2 is in a state where we are not sure what changes have happened since REC, and further divergence would make things messier.

Better stick to CSS Position.

@gitspeaks
Copy link
Author

Why are you changing CSS2 in a PR that says [css-position-3]?

AFAIK CSS2 is in a state where we are not sure what changes have happened since REC, and further divergence would make things messier.

Better stick to CSS Position.

This was not intentional. I thought GitHub would prompt me. How can I now exclude the CSS2 changes from the pull request?

@Loirooriol
Copy link
Contributor

Reset your branch to the right commit and force-push it.

@gitspeaks
Copy link
Author

Reset your branch to the right commit and force-push it.

Done.

@gitspeaks
Copy link
Author

AFAIK CSS2 is in a state where we are not sure what changes have happened since REC, and further divergence would make things messier.
Better stick to CSS Position.

The CSS2 section contains relevant information not present in the CSS Position section

Copy link
Contributor

@Loirooriol Loirooriol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two concepts:

It seems like the spec was trying to link to the former one. However, testcase:

<!DOCTYPE html>
<div style="display: inline-grid; grid-template: 50px 50px 50px / 50px 50px 50px; border: solid; position: relative">
  <div style="grid-area: 2 / 2 / 3 / 3; background: cyan; position: absolute; width: 100%; height: 100%"></div>
</div>

The percentages resolve to the grid area in Gecko, Blink and WebKit, so it's not the "original containing block".

So the edit seems good but it has behavior implications. Then I think it becomes substantive for IPR, and requires approval by editor discretion or CSSWG.

Copy link
Contributor

@Loirooriol Loirooriol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, so some archeology:

Therefore, despite using the same words, it wasn't referring to the original containing block. Then I think this PR isn't affecting conformance after all.

But better let the editors confirm, I guess.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants