visual-studio-code icon indicating copy to clipboard operation
visual-studio-code copied to clipboard

add editorActiveLineNumber.foreground

Open klebercode opened this issue 7 years ago • 14 comments

klebercode avatar Mar 18 '18 02:03 klebercode

Thanks for the contribution!

I'm inclined to say we should keep as-is since the yellow could be jarring to some users who have gotten used to the theme in its current form.

I'll keep this open for now to see if it draws any further feedback.

A solution for you in the interim is just to add an override for this to your user settings.

dsifford avatar Mar 29 '18 19:03 dsifford

Okay guy.

Thanks for the feedback.

klebercode avatar Apr 04 '18 16:04 klebercode

How about #95aeff?

screenshot from 2018-04-22 05-56-36

ajitid avatar Apr 22 '18 00:04 ajitid

I like this!

klebercode avatar Apr 22 '18 14:04 klebercode

I'd prefer to not add a non-spec color if we chose to try this.

The yellow is a fine idea, but let's wait until the PR has a few more :+1: before giving it a try. Want to make sure this is something everybody would want, rather than just a few people.

dsifford avatar Apr 22 '18 15:04 dsifford

On-spec colors comparison below.

Näyttökuva 2019-04-12 kello 21 19 34 Näyttökuva 2019-04-12 kello 21 20 01 Näyttökuva 2019-04-12 kello 21 20 11 Näyttökuva 2019-04-12 kello 21 20 19 Näyttökuva 2019-04-12 kello 21 20 29 Näyttökuva 2019-04-12 kello 21 20 41 Näyttökuva 2019-04-12 kello 21 20 51 Näyttökuva 2019-04-12 kello 21 21 00

ghost avatar Apr 12 '19 18:04 ghost

Personally I would go with simple white (foreground color). Regardless of what color, I do think that this is an useful addition as it helps you gauge which line you're on even better. Reason my line highlight is so dark is because I had real trouble seeing the active line with vanilla Dracula. Having a highlight in the margin makes sense.

ghost avatar Apr 12 '19 18:04 ghost

Another comparison, with vanilla theme (no custom modifications, no line highlight):

Näyttökuva 2019-04-12 kello 21 28 36 Näyttökuva 2019-04-12 kello 21 28 46 Näyttökuva 2019-04-12 kello 21 28 54 Näyttökuva 2019-04-12 kello 21 29 03 Näyttökuva 2019-04-12 kello 21 29 10 Näyttökuva 2019-04-12 kello 21 29 17 Näyttökuva 2019-04-12 kello 21 29 29 Näyttökuva 2019-04-12 kello 21 29 36

ghost avatar Apr 12 '19 18:04 ghost

I liked of purple version to keep the UI consistency

Purple Version

leandrocunha avatar Mar 12 '20 14:03 leandrocunha

I vote for either Cyan or Purple.

JimmyMultani avatar May 18 '20 18:05 JimmyMultani

This is picking up a bit of traction. Thoughts @zenorocha?

dsifford avatar May 19 '20 13:05 dsifford

@dsifford - Adding a comment here to keep this thread active. I'm currently working on the Dracula theme for Nova and am struggling to pick a color for the active line number. When I took over it was a non-spec color so to have an 'official' color would take the guess work out of it.

FWIW - Vim uses yellow for the active line number though that seems to be determined by the editor itself rather than the maintainers.

bdielman avatar Dec 16 '20 15:12 bdielman

@bdielman Thanks! Ya we should prob make a decision here. I use vim as my daily driver and I've grown accustomed to yellow, but not opposed to go with majority rule here.

dsifford avatar Dec 16 '20 20:12 dsifford