Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

refactor(one-dark): export all theme colors for reuse is needed#3

Closed
bRRRITSCOLD wants to merge 5 commits intocodemirror:mainfrom
bRRRITSCOLD:main
Closed

refactor(one-dark): export all theme colors for reuse is needed#3
bRRRITSCOLD wants to merge 5 commits intocodemirror:mainfrom
bRRRITSCOLD:main

Conversation

@bRRRITSCOLD
Copy link
Copy Markdown

Export theme colors for reuse by down stream users to match said theme colors/reuse of said theme colors

@bRRRITSCOLD
Copy link
Copy Markdown
Author

@marijnh If you have some time - I just want to reuse the theme color for some things in an app, don't want to use a forked version or keep local colors up to date with this repo

@marijnh
Copy link
Copy Markdown
Member

marijnh commented Aug 4, 2022

  1. We're definitely not breaking backwards compat or bumping the major version of any of these packages.
  2. This looks entirely broken and untested so it's really hard to take it seriously.

@bRRRITSCOLD
Copy link
Copy Markdown
Author

  1. @marijnh how does this break backwards compat? The things that are exported and renamed now were never exported to begin with, so no one would be using them in their code, therefore there is no possible way we break anyone.
  2. I will gladly update tests now! :)

@bRRRITSCOLD
Copy link
Copy Markdown
Author

bRRRITSCOLD commented Aug 4, 2022

  1. @marijnh you have no tests.... It works completely fine for me locally, so for "tested" idk what more your repo requires, your whole codebase is "untested" isn't it?

@marijnh
Copy link
Copy Markdown
Member

marijnh commented Aug 4, 2022

how does this break backwards compat?

You're the one who moved it to version 7.0.0.

I'm not talking about automated tests. I'm talking about the fact that you're using CSS properties like one-dark-background-color, which are definitely not something the browser will use.

@bRRRITSCOLD
Copy link
Copy Markdown
Author

bRRRITSCOLD commented Aug 4, 2022

backgroundColor to oneDarkBackgroundColor is clearly a mistake on my end.... Its an obvious "ctrl + f" mistake, fixing now.

As for 7.0.0 would your suggestion to be just a minor bump? 6.1.0? 6.0.1?

marijnh added a commit that referenced this pull request Aug 4, 2022
FEATURE: Export a `color` object holding the colors used in the
theme.

Issue #3
@marijnh
Copy link
Copy Markdown
Member

marijnh commented Aug 4, 2022

Would attached patch work as a replacement for this?

@marijnh marijnh closed this Apr 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants