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

Implement Highlight Pseudo-elements (::selection) #8077

Closed
paulrouget opened this issue Oct 19, 2015 · 6 comments
Closed

Implement Highlight Pseudo-elements (::selection) #8077

paulrouget opened this issue Oct 19, 2015 · 6 comments
Labels
A-content/css Interacting with CSS from web content (parsing, serializing, introspection)

Comments

@paulrouget
Copy link
Contributor

I'm not sure we can select text yet in a regular element, but it's possible in an <input> tag.The selection is not highlighted though.

See: https://drafts.csswg.org/css-pseudo-4/#selectordef-selection

@nox
Copy link
Contributor

nox commented Oct 19, 2015

This will require changes in rust-selectors.

@nox nox added A-content/css Interacting with CSS from web content (parsing, serializing, introspection) S-blocked-on-external Something, somewhere else, needs to happen before this PR can be merged. labels Oct 19, 2015
@paulrouget
Copy link
Contributor Author

Bumping to bhtml P2. Now we have an awesomebar, we need to be able to highlight the selected text in an <input>.

@mbrubeck
Copy link
Contributor

mbrubeck commented Mar 2, 2016

See also #4410 and #4409.

@paulrouget
Copy link
Contributor Author

@SimonSapin - can you confirm you're working on this issue?

@SimonSapin
Copy link
Member

@paulrouget As we’ve discussed in person I think #9993 can be separated from styling support. I haven’t started on this yet as I’d like to wrap up servo/rust-url#176 and its fallout first.

With a recent change to rust-selectors, parsing a new pseudo-element can be done in components/style/selector_impl.rs without changing rust-selectors.

@SimonSapin SimonSapin removed the S-blocked-on-external Something, somewhere else, needs to happen before this PR can be merged. label Mar 14, 2016
@mbrubeck
Copy link
Contributor

Fixed by #10252.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-content/css Interacting with CSS from web content (parsing, serializing, introspection)
Projects
None yet
Development

No branches or pull requests

4 participants