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

Highlight text selection #9993

Closed
SimonSapin opened this issue Mar 14, 2016 · 3 comments
Closed

Highlight text selection #9993

SimonSapin opened this issue Mar 14, 2016 · 3 comments
Assignees

Comments

@SimonSapin
Copy link
Member

We already support keyboard text selection (with Shift + arrows) in <input>, but it’s invisible. This issue is about changing the text color and background color of selected text, like other browsers do. Colors can be hard-coded at first, #8077 is about adding a ::selection pseudo-element to change them with CSS.

I don’t know if we’re better off splitting text fragments in layout at selection boundaries, or only splitting display items. @pcwalton, do you have an opinion on this?

CC @paulrouget

@jdm
Copy link
Member

jdm commented Mar 14, 2016

See also #4410 and #4409.

@pcwalton
Copy link
Contributor

The right thing is to split fragments but not text runs, I believe. That's because anything you can style as one unit in CSS needs to be a fragment, but anything ligatures work across needs to be one text run.

@mbrubeck mbrubeck self-assigned this Mar 23, 2016
@mbrubeck
Copy link
Contributor

I've started working on the layout parts of this (fragment splitting and display list generation).

mbrubeck added a commit to mbrubeck/servo that referenced this issue Mar 24, 2016
Fixes servo#9993.  This does not yet allow stylesheets to set the selection colors;
instead it uses a hard-coded orange background and white foreground.
mbrubeck added a commit to mbrubeck/servo that referenced this issue Mar 24, 2016
Fixes servo#9993.  This does not yet allow stylesheets to set the selection colors;
instead it uses a hard-coded orange background and white foreground.
mbrubeck added a commit to mbrubeck/servo that referenced this issue Mar 24, 2016
Fixes servo#9993.  This does not yet allow stylesheets to set the selection colors;
instead it uses a hard-coded orange background and white foreground.
mbrubeck added a commit to mbrubeck/servo that referenced this issue Mar 24, 2016
Fixes servo#9993.  This does not yet allow stylesheets to set the selection colors;
instead it uses a hard-coded orange background and white foreground.
mbrubeck added a commit to mbrubeck/servo that referenced this issue Mar 25, 2016
Fixes servo#9993.  This does not yet allow stylesheets to set the selection colors;
instead it uses a hard-coded orange background and white foreground.
bors-servo pushed a commit that referenced this issue Mar 25, 2016
Highlight selected text in input fields

Fixes #9993.  This does not yet allow stylesheets to set the selection colors; instead it uses a hard-coded orange background and white foreground.

r? @pcwalton

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10176)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Mar 26, 2016
Highlight selected text in input fields

Fixes #9993.  This does not yet allow stylesheets to set the selection colors; instead it uses a hard-coded orange background and white foreground.

r? @pcwalton

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10176)
<!-- Reviewable:end -->
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

No branches or pull requests

4 participants