netatoo / phoenix-theme

Dark & Light custom UI themes with colors variations for Sublime Text 2
890 stars 61 forks source link

Light and Dark theme appear visually broken #38

Closed Piderman closed 9 years ago

Piderman commented 10 years ago

Hey guys, loaded up sublime today and both themes appear to be visually broken, getting the following error

Error trying to parse file: Trailing comma before closing bracket ... Phoenix Light.sublime-theme:1605:1, same error in dark: ...Phoenix Dark.sublime-theme:2180:1

Looks like it happened in the latest commit

Removing the last comma seemed to work for me.

Cheers

handycomm commented 10 years ago

I solved. Thanks for comment.

ccoley commented 10 years ago

Removing the last comma worked for me too.

darylknight commented 10 years ago

My theme is broken too. I can't find the file to remove the comma from, where can I find it please?

ccoley commented 10 years ago

It's located in the Packages folder.

On Windows: C:\Users\{USER}\AppData\Roaming\Sublime Text 2\Packages\Theme - Phoenix\Phoenix Dark.sublime-theme

If you're using the Light theme, replace 'Dark' in the file name with 'Light'.

darylknight commented 10 years ago

Thanks, I'm on OSX though and it doesn't appear to be accessible in the same way. I've got:

/Users/{user}/Library/Application Support/Sublime Text 3/Installed Packages/Theme - Phoenix.sublime-package

but can't actually get into the files inside there. There is nothing that refers to the Phoenix theme inside /Packages. Will have to wait for a fix or use something else for now :)

foxx commented 10 years ago

I'm still having this same problem, looking into it now

foxx commented 10 years ago

Same problem

Error trying to parse file: Trailing comma before closing bracket in ~/Library/Application Support/Sublime Text 2/Packages/Theme - Phoenix/Phoenix Dark.sublime-theme:2180:1

Removing the comma worked, thanks @Piderman. Sending a PR fix now

silverwind commented 10 years ago

Managed to fix this on Windows with removing that comma. No luck on OS X thought. The files in ~/Library aren't actually read it seems.

Merg1255 commented 10 years ago

Also confirmed on Windows with the solution provided above. Might need a general push for all systems.

netatoo commented 9 years ago

Thanks all and sorry for the delay, pull request from foxx was merged on master.