laCour / slack-night-mode

A Stylus style for easy Slack theming
Creative Commons Zero v1.0 Universal
568 stars 387 forks source link

Additional Style Updates #208

Closed zengshengliu closed 5 years ago

zengshengliu commented 5 years ago

Update App "About" page background, commands, and font color Update App Tabs color Update Thread "footer" color Update Button Attachment color Update Dialog textarea font color Update c-pillow_file font color Update Threads banner color Update Threads Flexpane reply box color Update Threads page color (I hate they have p-threads and p_threads class 😞) Update Threads app label and timestamp color Update Channel Preview footer color Update Channel Notification Banner color Update Flexpane Close button color Update Multi File Upload dialog color Update File Upload overlay color

zengshengliu commented 5 years ago

@mastermel Let me know if you have any more questions. I think I am good on this PR (unless I find more missing styling)

mastermel commented 5 years ago

@zengshengliu I wonder if you could help me out? I'm attempting to test your PR locally using Stylish and the Slack web client in Chrome. When I attempt to import the transpiled css I get this error. Any ideas?

zengshengliu commented 5 years ago

@mastermel It looks like Stylish doesn't like the class name --active. Let me see if there is another selector I can use instead, because it does looks like an invalid class name but somehow Slack uses it

zengshengliu commented 5 years ago

@mastermel That the exact selector Slack used by default. I don't think there is another selector we can use. Do you have any suggestion?

image

mastermel commented 5 years ago

@zengshengliu sorry for the late reply! I haven't been able to find another way to override that .channel_header__tab.--active selector such that it will pass the Mozilla Format 😞

Without being able to do that I'm not sure we'll be able to publish your changes to userstyles.org.

Do you think for the time being you could comment that override out, so that we can get the rest of your updates merged?

zengshengliu commented 5 years ago

@mastermel will do that

zengshengliu commented 5 years ago

@mastermel This PR should be good to merge now.

nysos3 commented 5 years ago

@zengshengliu @mastermel

I'll be switching to running this fork for some time for some pre-merge testing.