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

Fix a few colors to make this look more like Atom #95

Closed
wants to merge 1 commit into from
Closed

Fix a few colors to make this look more like Atom #95

wants to merge 1 commit into from

Conversation

emroussel
Copy link

I use this theme and added a couple color customizations in my settings to make a couple small things look more like Atom, and thought I'd share them here.

I'm happy to discuss or make changes if you spot something I didn't, or feel like some changes aren't appropriate.

Changes:

"editor.findMatchHighlightBackground": "#314166",
"editor.foreground": "#ABB2C0",
"editorWidget.border": "#373D48",
"editorLineNumber.activeForeground": "#777D88",
"editorLineNumber.foreground": "#4B5365",
"editorRuler.foreground": "#abb2bf26",
"tab.activeForeground": "#D7DAE0"

With before/after and Atom screenshots:

"editor.findMatchHighlightBackground": "#314166"

VSCode (before)
screen shot 2018-05-12 at 18 38 02
Atom
screen shot 2018-05-12 at 18 31 52
VSCode (after)
screen shot 2018-05-12 at 18 31 47

--

"editor.foreground": "#ABB2C0"

VSCode (before)
screen shot 2018-05-12 at 18 41 14
Atom
screen shot 2018-05-12 at 18 40 06
VSCode (after)
screen shot 2018-05-12 at 18 41 28

--

"editorWidget.border": "#373D48"

VSCode (before)
screen shot 2018-05-12 at 18 43 57
Atom
No Atom equivalent, but this color fits better into Atom's color palette

VSCode (after)
screen shot 2018-05-12 at 18 43 42

--

"editorLineNumber.activeForeground": "#777D88"

VSCode (before)
screen shot 2018-05-12 at 18 49 56
Atom
screen shot 2018-05-12 at 18 50 11
VSCode (after)
screen shot 2018-05-12 at 18 51 35

--

"editorLineNumber.foreground": "#4B5365"

VSCode (before)
screen shot 2018-05-12 at 18 52 43
Atom
screen shot 2018-05-12 at 18 52 54
VSCode (after)
screen shot 2018-05-12 at 18 53 09

--

"editorRuler.foreground": "#abb2bf26"

VSCode (before)
screen shot 2018-05-12 at 18 57 06
Atom
screen shot 2018-05-12 at 18 57 24
VSCode (after)
screen shot 2018-05-12 at 18 56 52

--

"tab.activeForeground": "#D7DAE0"

VSCode (before)
screen shot 2018-05-12 at 18 59 03
Atom
screen shot 2018-05-12 at 18 59 22
VSCode (after)
screen shot 2018-05-12 at 18 59 16

@akamud
Copy link
Owner

akamud commented May 13, 2018

Wow, this is the kind of PR that makes open source totally worth it. ❤️
Thank you for your contribution, I also maintain the One Light theme, so I'll be merging your changes as soon as I replicate them to the light theme.

Thank you!

@emroussel
Copy link
Author

@akamud any updates on this?

@akamud
Copy link
Owner

akamud commented Jun 27, 2018

I'm so sorry for taking so long, I still haven't got the time to make a new release. I plan to work on it this weekend.

@emroussel
Copy link
Author

No worries, thanks!

@matthew-e-brown
Copy link

@akamud any plans/updates on merging this? Thanks!

@emroussel
Copy link
Author

@matthew-e-brown I ended up creating a new theme with those changes (+ more) because I wasn't sure if this repo is still maintained. Still happy to see this PR getting merged, but feel free to use Atomize if you want to have these changes now and (in my opinion) a more atom like experience: https://github.com/emroussel/atomize

@matthew-e-brown
Copy link

@emroussel Fantastic! Thanks. Will check it out.

@akamud
Copy link
Owner

akamud commented Feb 13, 2020

Hi everyone, indeed I haven't use VSCode for a while now (doing stuff on JetBrains Rider atm), that's why I didn't put much effort into maintaining this project anymore. I can't just accept the PR because I generate the theme automatically and directly from the Atom One Dark Syntax. So any PR I accept would be overridden, what I need to do is replicate the changes to my generator. I'm sorry about that.

Please remember that VSCode allows for any customization to be applied on top of any theme, like I mention in the README, so that takes away the burden of releasing new versions so frequently.

About forks, I use a MIT license exactly to allow anyone to keep on going from where I stopped. Just remember that MIT requires you to reference the original copyright. I see a lot of forks from this theme that just ignores this completely and is, therefore, breaking the license.

@akamud
Copy link
Owner

akamud commented Mar 15, 2020

Thank you again for your contributions, all your fixes are released in 2.2.0.

@akamud akamud closed this Mar 15, 2020
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

Successfully merging this pull request may close these issues.

None yet

3 participants