slack-black-theme icon indicating copy to clipboard operation
slack-black-theme copied to clipboard

changes do not persist after reboot

Open gonzobrains opened this issue 6 years ago • 6 comments

This mod works great...until I reboot! I have to update the index.js file after every reboot in order for the theme to continue working.

gonzobrains avatar Nov 27 '17 14:11 gonzobrains

Confirming this on Windows 7 local instance.

jrausch12 avatar Nov 27 '17 17:11 jrausch12

Reporting not necessarily, in every case. I'm using Slack 2.9.0 64-bit and it worked just fine after multiple reboots (Tested 3 times; Third time is the charm)

Possible problem due to new slack update 3.0.0 beta (as helpfully pointed out by others)

There are several possibilities as to why it failed to work.

  • Maybe it was a network-related issue. The stylesheet is served from a CDN https://cdn.rawgit.com/widget-/slack-black-theme/master/custom.css which may have failed to cache in previous use, or load in current session. Try using locally served?
  • The setup might have been all wrong or maybe Slack app added something else to the file. This has happened before and it causes a blank white screen to appear
  • Maybe it's just the update. Stick with stable?

A possible workaround: Should you find it cumbersome to have to reapply every single time on Windows, you could add this script to the startup.

image

siddhantrimal avatar Nov 28 '17 04:11 siddhantrimal

Updated script for Slack 3.0.0 beta /w support for Slack 2.9.0. Just keep it in your startup if nothing else works. Cheers!

Tested, with multiple restarts, on Slack 3.0.0beta in Windows 10 Pro 64-bit and Slack 2.9.0 in Windows 7 Ultimate 32-bit

Possible case of fail after update, rather than fail after reboot.

siddhantrimal avatar Nov 28 '17 10:11 siddhantrimal

In my experience, this happens when slack updates a new version upon reboot (which is frequently). Even a minor version update will break the theme as it creates a new folder with a different index.js file and you have to go replace it again with the edited index file. Using the automatic applicator script from siddhantrimal could be beneficial if you find this to be annoying.

Siddhant, note that your edit for 3.0.0 only applies the default black theme and does not work with any custom color choices made to the index file. You might want to include some information on how to adjust it so that custom choices can continue to be applied.

Bterrell721 avatar Nov 28 '17 16:11 Bterrell721

i did automation for mac and then took @siddhantrimal's idea and made a shortcut https://github.com/widget-/slack-black-theme/pull/37

The "reboot" issue is actually just slack doing updates ever time it is restarted (if available). This script can replace your slack app icon and do a bit of work with starting slack, checking if it updated the theme (removed it) and then re-updating it if it is not already there.

Artistan avatar Dec 22 '17 20:12 Artistan

@Artistan nicely done! Could you take a look at this and add an automatic UAC Elevated Access towards the end(at the :ErrorHandler segment?). I mean, when you go, why not go all the way :D

There's also a powershell version but I believe making it available to cmd makes more sense because a lot of computer still don't have it.

Looking forward. Cheers!

...and yes, this definitely averts the RARE(Reset-After-REboot) issue

siddhantrimal avatar Dec 27 '17 08:12 siddhantrimal