saadq / bliss-ui

Ultra minimalist theme optimized for usage with base16-ocean.
https://atom.io/themes/bliss-ui
MIT License
17 stars 3 forks source link

Git color in tree view does not work #1

Closed wyudong closed 6 years ago

wyudong commented 6 years ago

Git color won't show if using bliss-ui, other default ui are fine. Don't know if it's a bug or not.

Thanks for creating bliss-ui. Cheers!

saadq commented 6 years ago

Hey @wyudong, thanks for raising this issue! This was actually an intentional choice when this theme had just been for my own personal use and wanted to keep the look very minimal. The styles for this are here:

https://github.com/saadq/bliss-ui/blob/86871f01c937fe705e90d1abfcc498c543c44f45/styles/tree-view.less#L6-L23

However, I understand that people may not like this choice, so it may make sense to remove these styles from the theme, and maybe just put Recommended Styles in the README and move the styles there.

Would you like to send a PR? Just let me know, otherwise I can make the change myself within a couple of days.

wyudong commented 6 years ago

Hi @saadq I find I need to comment these lines as well in order to enable git color:

https://github.com/saadq/bliss-ui/blob/86871f01c937fe705e90d1abfcc498c543c44f45/styles/tree-view.less#L54-L56

PR #2

saadq commented 6 years ago

Ah, thanks for finding that, and just gave a quick comment to your PR

wyudong commented 6 years ago

To remove code about .icon-file-text, .icon occurs a side effect, that is when selecting any item in a folder which already has files being modified, all the other items will turn yellow (the modified color). I don't know the reason. Will have a look at it later.

git

saadq commented 6 years ago

Oh very strange, I'll try to take a look later as well in case you don't see any immediate cause.

wyudong commented 6 years ago

@saadq any progress from your side?

wyudong commented 6 years ago

@saadq I think I've fixed it. Testing it now and will make a pr if everything's good.

saadq commented 6 years ago

Sorry for the delay! I had some things going on so wasn't able to work on it. Checking out your PR right now!