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: Make :open target color and file inputs with an open picker #3419

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AtkinsSJ
Copy link
Member

This works when testing it manually, but I can't seem to figure out how to make an automated test for it unfortunately.

I've imported the existing WPT test for :open but it doesn't cover <input>, and possibly never will as the behaviour of different input types is largely UA-defined. We fail the subtest for <select> too, and that also works when done manually.

Copy link

github-actions bot commented Feb 5, 2025

Your pull request has conflicts that need to be resolved before it can be reviewed and merged. Make sure to rebase your branch on top of the latest master.

@github-actions github-actions bot added the conflicts Pull request has merge conflicts that need resolution label Feb 5, 2025
@AtkinsSJ AtkinsSJ force-pushed the css-open-pseudoclass branch from 6d24844 to 198ad55 Compare February 5, 2025 14:29
@AtkinsSJ
Copy link
Member Author

AtkinsSJ commented Feb 5, 2025

Fixed conflicts.

@github-actions github-actions bot removed the conflicts Pull request has merge conflicts that need resolution label Feb 5, 2025
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.

1 participant