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

Mark the current heading in the dropdown list #2422

Closed
Reinmar opened this issue Aug 17, 2016 · 13 comments · Fixed by ckeditor/ckeditor5-heading#60
Closed

Mark the current heading in the dropdown list #2422

Reinmar opened this issue Aug 17, 2016 · 13 comments · Fixed by ckeditor/ckeditor5-heading#60
Assignees
Labels
package:heading type:improvement This issue reports a possible enhancement of an existing feature.
Milestone

Comments

@Reinmar
Copy link
Member

Reinmar commented Aug 17, 2016

Currently, the current option is only visible in the dropdown button. It should also be marked inside the panel.

@oleq
Copy link
Member

oleq commented Mar 9, 2017

What would you say about it?

dropdown-active-item

@oleq oleq self-assigned this Mar 9, 2017
@Reinmar
Copy link
Member Author

Reinmar commented Mar 9, 2017

I always have problems with active/inactive states. If you have just two options to choose from and one of them is active in a lot of apps you have no idea which of these options is the active one and which is inactive.

One notable example here – Sublime Text:

image

I'm working with this app every day and I still don't know which option is selected.

What's worse – this happens to me even if there are more options than 2. Since the style itself is pretty indistinguishable there's no mental "tick" when I notice it. It usually takes me a couple of arrow down/up key presses to figure out where the damn selection is in Sublime's panel.

In the example you showed the focus is clearly marked using an intuitive glow, so that's, I think, ok.

However, the active/inactive styling would fail the same test which Sublime fails. This isn't too important now because most of the time you'll have more than 2 options in that panel, but if you have a better idea for that style, I'd be happy to see it :D.

PS. There's also https://github.com/ckeditor/ckeditor5-heading/issues/55 to fix (I mention it because it's visible on the screencast).

@oleq
Copy link
Member

oleq commented Mar 10, 2017

We could invert active selection. This is what OSs usually do:
dropdown-invert

@Reinmar
Copy link
Member Author

Reinmar commented Mar 10, 2017

Slack uses similar styling (color for active, slight shade change for hover) in its channel list and I think I like it. But macOS uses a bit different styling for its top menu AFAICS – hover is color and active has a check mark :D. But a similar to Slack's styling is used e.g. in Messages (on the left) and it's fine.

OTOH, personally, I'll have to get used to this blue background. ATM, it stands out pretty strongly. I guess that I'd just need time.

@oleq
Copy link
Member

oleq commented Mar 10, 2017

Some experiments with ticks

image

@oleq
Copy link
Member

oleq commented Mar 10, 2017

And hued color without text inversion
image

TBH, this is the one I like the most.

@oleq
Copy link
Member

oleq commented Mar 10, 2017

Combined:
combined

@Reinmar
Copy link
Member Author

Reinmar commented Mar 10, 2017

All fine for me. Maybe just small doubt about the tick – that it moves the text to the right. In native macOS menus it has a reserved space on the left, so adding the check mark doesn't move the item.

@oleq
Copy link
Member

oleq commented Mar 10, 2017

A good point.

image

Needs some tuning though.


Edit:

image

@Reinmar
Copy link
Member Author

Reinmar commented Mar 10, 2017

screen shot 2017-03-10 at 11 34 36

I noticed on the early screenshots that the items are not aligned well. Is this some styling issue?

@oleq
Copy link
Member

oleq commented Mar 10, 2017

Needs some tuning though.

I updated my response.

@Reinmar
Copy link
Member Author

Reinmar commented Mar 10, 2017

So, I like:

image

And:

image

And (if the items were aligned):

image

In the first solution, I'm not sure about the bright, panty color (maybe if it was the same as the focused border of the editable?). E.g. the stronger one in the second solution looks better to me, though, it may stand out too much. In the 3rd option there's no problem with color, cause it's grey so it may be the easiest option :D.

@oleq
Copy link
Member

oleq commented Mar 10, 2017

In the first solution, I'm not sure about the bright, panty color (maybe if it was the same as the focused border of the editable?). E.g. the stronger one in the second solution looks better to me, though, it may stand out too much.

This one is pretty rough IMO, but it also makes sense.

image

I'll make up my mind and create a PR ;-)

szymonkups referenced this issue in ckeditor/ckeditor5-heading Mar 13, 2017
Feature: Active heading is marked in the drop-down list. Closes #26.
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-heading Oct 9, 2019
@mlewand mlewand added this to the iteration 9 milestone Oct 9, 2019
@mlewand mlewand added status:confirmed type:improvement This issue reports a possible enhancement of an existing feature. package:heading labels Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:heading type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants