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

Move color scheme fixes to User directory #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Move color scheme fixes to User directory #77

wants to merge 2 commits into from

Conversation

tdhsmith
Copy link
Collaborator

@tdhsmith tdhsmith commented Feb 4, 2015

Package Control users utilizing the 'fix color scheme' feature would
have their schemes overwritten on every update, leading to an annoying
'missing color scheme' popup. This moves future fixes to the User folder
to avoid deletion.

Since the issue affects this update as well, there is also a small
function to fail a bit more gracefully, reverting missing themes to
Monokai rather than generating endless dialog boxes (which is Sublime's
ridiculous default behavior).

Fixes #68

Package Control users utilizing the 'fix color scheme' feature would
have their schemes overwritten on every update, leading to an annoying
'missing color scheme' popup. This moves future fixes to the User folder
to avoid deletion.

Since the issue affects this update as well, there is also a small
function to fail a bit more gracefully, reverting missing themes to
Monokai rather than generating endless dialog boxes (which is Sublime's
ridiculous default behavior).
@tdhsmith
Copy link
Collaborator Author

tdhsmith commented Feb 4, 2015

(I'm going to test this on other OSes first.)

@jbrooksuk
Copy link
Collaborator

Let me know once tested. Looks fine though.

@jbrooksuk jbrooksuk self-assigned this Feb 4, 2015
@tdhsmith
Copy link
Collaborator Author

tdhsmith commented Feb 6, 2015

Seems good. Yeah it wasn't a big change but I want to test whenever I introduce a file system manipulation. 😉

@tdhsmith
Copy link
Collaborator Author

tdhsmith commented Feb 6, 2015

I don't know if you prefer each change to have its own version, but FYI I'll be working on the HSL parsing issue in an hour or so.

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.

GC was updated and ST load this error
2 participants