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

Rename position-anchor:implicit to position-anchor:auto #46269

Merged
merged 1 commit into from
May 15, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented May 14, 2024

This was resolved here:

w3c/csswg-drafts#10312 (comment)

and will be added to the spec shortly.

This is observable, but barely. After this CL, implicit won't be a
valid value for position-anchor, so it'll resolve to the default
value of auto. Meaning the behavior will be the same. But via the
CSSOM, getComputedStyle(element).positionAnchor will now return "auto"
instead of "implicit".

Fixed: 340206288
Change-Id: I75b05310638dca70aa6aa8807da8cfe7c9624a32
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5540275
Reviewed-by: Rune Lillesveen <[email protected]>
Commit-Queue: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1301585}

This was resolved here:

w3c/csswg-drafts#10312 (comment)

and will be added to the spec shortly.

This is observable, but barely. After this CL, `implicit` won't be a
valid value for `position-anchor`, so it'll resolve to the default
value of `auto`. Meaning the behavior will be the same. But via the
CSSOM, getComputedStyle(element).positionAnchor will now return "auto"
instead of "implicit".

Fixed: 340206288
Change-Id: I75b05310638dca70aa6aa8807da8cfe7c9624a32
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5540275
Reviewed-by: Rune Lillesveen <[email protected]>
Commit-Queue: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1301585}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 6bdb665 into master May 15, 2024
19 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5540275 branch May 15, 2024 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants