netatoo / phoenix-theme

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

Duplicate folder icons with Sublime Build 3065 #36

Closed adamjohnson closed 9 years ago

adamjohnson commented 10 years ago

Sublime Build 3065 introduced the ability to have sidebar icons (changelog). This causes Phoenix to display duplicate folder icons:

https://s3.amazonaws.com/f.cl.ly/items/271g2O28061g2B1F182n/duplicate-folder-icons.png

Full size preview

Fix:

Phoenix should now use ST3's built in sidebar icon functionality.

zekesonxx commented 10 years ago

I'm also having this issue

CMCDragonkai commented 10 years ago

Same here. But this package hasn't been updated in a year?

daslicht commented 10 years ago

same here , any idea how to fix, is there a fixed version? please?

daslicht commented 10 years ago

Just found : http://www.reddit.com/r/SublimeText/comments/2ewbj5/sublime_text_3_beta_build_3065_released/ck3mso4 Any knows how to apply this ? I tried extracting the content of the package and to edit the theme file. But how to activate it again ? Its not shown up when i put it in my users folder :/

kristijanhusak commented 10 years ago

I created a PR for this. If you don't want to wait, just use my fork with manual installation.

daslicht commented 10 years ago

@kristijanhusak Thank You very much ! works like a charm

CMCDragonkai commented 10 years ago

@netatoo please merge this, so it can be downloaded from sublime package.

sheerun commented 10 years ago

I made pretty solid fix in my derivative: https://github.com/sheerun/sublime-wombat-theme/commit/fcc2fb84d617911843c2f6c2e454f94fa0bf23d0

CMCDragonkai commented 10 years ago

Cool, what's the main differences between wombat and phoenix?

sheerun commented 10 years ago

It's mix of Phoenix and Soda, crafted specifically for Wombat color scheme (but you can use either different scheme or theme, they are separate). Btw. you need to wait few hours for update to propagate.

netatoo commented 9 years ago

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