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

Fix selectmenu's button for writing-mode #39135

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 22, 2023

Before this patch, the selectmenu's button would not rotate when
writing-mode is asking it to, but based on the mockups from this issue
we should be rotating the button:
openui/open-ui#600

Fixed: 1426702
Bug: 1427573
Change-Id: Ib2b4884efa518d2a445638f99d87be66e87138f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4359392
Commit-Queue: Joey Arhar <[email protected]>
Reviewed-by: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1121829}

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 changed the title i really hope this test fails on at least one bot Fix selectmenu's button for writing-mode Mar 22, 2023
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-4359392 branch 2 times, most recently from e90d51a to b3ed46d Compare March 24, 2023 17:35
Before this patch, the selectmenu's button would not rotate when
writing-mode is asking it to, but based on the mockups from this issue
we should be rotating the button:
openui/open-ui#600

Fixed: 1426702
Bug: 1427573
Change-Id: Ib2b4884efa518d2a445638f99d87be66e87138f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4359392
Commit-Queue: Joey Arhar <[email protected]>
Reviewed-by: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1121829}
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.

3 participants