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

Selected theme isn't permanent #98

Closed
nbaroiller opened this issue Aug 1, 2014 · 15 comments
Closed

Selected theme isn't permanent #98

nbaroiller opened this issue Aug 1, 2014 · 15 comments

Comments

@nbaroiller
Copy link

I've just upgraded to Sprint 42 0.42.0-13957 (release f9fcb884f) on Win8 and I'm now encountering this issue with Themes for Brackets 1.7.3: the theme I choose isn't applied to other opened files and whenever I hit F5, I get the standard one for the current file though it's not selected in the dropdown menu. Restarting Brackets didn't help either.

@wormeyman
Copy link
Collaborator

Any thoughts @Jacse ? I'm guessing this is because of native themes now.

@MikeSpock
Copy link

Yes, since native themes, themes for brackets stopped being permananet, and when Brackets is restarted, default theme is active.
imho, native themes are ok, but Themes for Brackets is far superior. They should have just integrated this plugin. Anyways, I really miss my dark soda :D

@MikeSpock
Copy link

So I asked Brackets devs what's up with this. The reply i got was:
"I think most of the Themes have been converted into separate plugins now, eg. Dark Sodar, Solarized, Delkos, are now separate plugins. So if say you liked Dark Sodar you can install individually now and it appears in the standard Themes menu. I think some have not yet been converted, eg. Obsidian.. So they might only be 2 built in themes, but it's very easy to extend.."

Maybe if @Jacse had the time, he should start moving this project into the native theme selector panel instead of a separate plugin in a separate Brackets dropdown menu. That would be awesome. I mean I tried Dark Soda extension, but Jacse's is much better. Also, hunting down separate themes is a pain in the a, if this plugin could install all its 12 themes into the native theme selection menu, it would stay the #1 themepack for brackets.

edit: 'monokai dark soda' plugin looks exactly like this plugin's dark soda, but my point still stands I guess.

@Jacse
Copy link
Owner

Jacse commented Aug 5, 2014

Yes, it is definetely a priority to rebuild this project into a theme-pack, now that native themes have finally landed.

@Jacse
Copy link
Owner

Jacse commented Aug 5, 2014

I'm thinking this could become version 2?

@MikeSpock
Copy link

v2 makes sense. I think Brackets users are likely to update quickly to newer versions, so I guess time is key. Btw, thanks for the awesome work you've done with this plugin so far.

@Jacse
Copy link
Owner

Jacse commented Aug 5, 2014

No problem, glad you like it. I will hopefully make the necessary changes Thursday

@stgamarra
Copy link

Erm, @Jacse theres any way to make your Dark Soda's theme permanent again? D: i really miss it, i cant get along with the Monokai's Dark Soda, please tell me theres a way to do it! Greetings!

@Jacse
Copy link
Owner

Jacse commented Aug 7, 2014

Working on it @moo--

@Jacse
Copy link
Owner

Jacse commented Aug 7, 2014

Ok guys, want to hear your take on this

As far as I can tell, two things are not supported natively yet:

  1. Multiple themes per extension
  2. Styling "UI" - and not just the editor.

@MikeSpock
Copy link

As per the package.json schema, it does seem that one theme extension can contain only a single style. Styling the UI - it looks like there is a 'dark':true switch which can be used to indicate that your theme has a dark appearance, but that's about it.

I'm not an expert on Brackets Extensions, so I'm just gonna throw in: Would it be possible to create a standard extension, which would then extract/install multiple theme extensions in the brackets plugin folder?

@Jacse
Copy link
Owner

Jacse commented Aug 7, 2014

@MikeSpock Technically I think so, but I imagine it would be a bitch to maintain

@Jacse
Copy link
Owner

Jacse commented Aug 7, 2014

adobe/brackets#8423 Theme packs will not be supported (natively).

@Jacse
Copy link
Owner

Jacse commented Aug 7, 2014

Ok, I think I've figured it out. Now Themes-for-Brackets works with native themes!! 4f3b3ed

Still a lot of work to do on the themes, but I'm extremely busy so we'll just have to see about that. Scrollbars doesn't work for example.

The Dark Soda theme is very popular, I'm thinking about releasing a standalone with just that, thoughts?

@Jacse Jacse closed this as completed Aug 7, 2014
@wormeyman
Copy link
Collaborator

Dark Soda has been released twice i think:

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

No branches or pull requests

5 participants