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

LibWeb: Update Element::directionality() for bdi elements to match spec #4031

Merged
merged 1 commit into from
Mar 23, 2025

Conversation

Noah55-maker
Copy link
Contributor

This fixes three WPT test cases at html/dom/elements/global-attributes/dir-assorted.window.html

@gmta
Copy link
Collaborator

gmta commented Mar 21, 2025

Welcome to the project, thank you for implementing this!

@gmta gmta enabled auto-merge (rebase) March 21, 2025 07:42
@gmta
Copy link
Collaborator

gmta commented Mar 21, 2025

@Noah55-maker the Text/input/wpt-import/css/selectors/dir-pseudo-on-bdi-element.html test is failing; quite possibly you need to update the expectations. Check out our documentation on testing for more pointers.

@gmta gmta disabled auto-merge March 21, 2025 07:59
@ladybird-bot
Copy link
Collaborator

Hello!

One or more of the commit messages in this PR do not match the Ladybird code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

This fixes three WPT test cases at html/dom/elements/global-attributes/dir-assorted.window.html

Update test expectations for Tests/LibWeb/Text/expected/wpt-import/css/selectors/dir-pseudo-on-bdi-element.txt
@gmta gmta enabled auto-merge (rebase) March 23, 2025 07:35
@gmta gmta merged commit add380d into LadybirdBrowser:master Mar 23, 2025
7 checks passed
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.

3 participants