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

Rename virtual selection to highlight #4151

Closed
pjasiun opened this issue Aug 18, 2017 · 4 comments · Fixed by ckeditor/ckeditor5-engine#1091
Closed

Rename virtual selection to highlight #4151

pjasiun opened this issue Aug 18, 2017 · 4 comments · Fixed by ckeditor/ckeditor5-engine#1091
Assignees
Milestone

Comments

@pjasiun
Copy link

pjasiun commented Aug 18, 2017

I was explaining @scofalik what virtual selection is and it takes a while to get that virtual selection and fake selection are different mechanisms. In fact, the selection has 2 meanings:

  • the place where the user is editing and
  • visual marker over some text/content.

While the fake selection is focused mostly on the first mechanism, the virtual selection is only about the second. And I think it's wrong. We used to associate selection with the document selection, with the first meaning. Virtual selection is misleading since it has nothing in common with the selection as the place where the user is editing.

So, since virtual selection is not a selection we should find a better name.

Maybe just 'marker'? It describes very well what it really is and model markers will be mostly converted to it. What I don't like in calling it "marker", is the "mostly" part. We can convert model markers to elements and it might be strange that model markers do not need to be a view marker.

But it's Friday, 17:40 and I have no better ideas.

@fredck
Copy link
Contributor

fredck commented Aug 18, 2017

Probably it's important to avoid confusion with markers.

"Highlight" or "Custom Highlight"?

@szymonkups
Copy link
Contributor

szymonkups commented Aug 21, 2017

I like "highlight". I think it describes very well what it is in the view: it makes part of the content to stand out by applying some visual effect.

.fromMarker( 'search' ).toHighlight( { class: 'search-highlight' } );

It looks beautiful, isn't it? :D

@scofalik
Copy link
Contributor

I love highlight.

@pjasiun pjasiun changed the title Rename virtual selection Rename virtual selection to highlight Aug 21, 2017
@pjasiun
Copy link
Author

pjasiun commented Aug 21, 2017

Let's go with "highlight"!

@szymonkups szymonkups self-assigned this Aug 21, 2017
pjasiun referenced this issue in ckeditor/ckeditor5-engine Aug 22, 2017
Other: Renamed virtual selection to highlight. Closes #1085.
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-engine Oct 9, 2019
@mlewand mlewand added this to the iteration 11 milestone Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants