laCour / slack-night-mode

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

Thread text field is white #180

Closed keith closed 6 years ago

keith commented 6 years ago

The text field in the thread view no longer matches the rest of the theme:

image

I believe this started happening after Slack updated to allow images in threads.

roko-p commented 6 years ago

Adding background-color: #545454 to the CSS rule which is currently:

#message_edit_container .message_input, .inline_message_input_container .message_input { color: #e6e6e6;  }

fixed it for me.

So I edited this for myself, and am using the updated built css from this gist. (Feel free to link to that if this quick fix is good enough for you)


Didn't make a pull request because I'm not sure where exactly and how this should be done in the .scss files. Also the white background of the button should probably be styled as well, but it's not a problem to me so I didn't bother

nmielnik commented 6 years ago

Adding this selector fixed both the textbox and the button color:

#reply_container div.inline_message_input_container.with_file_upload {
  background: padding-box #545454;
  border-color: #424242;
  color: #e6e6e6;
}

there definitely could be another way to go about this, but targeting the .inline_message_input_contaner element itself controls the background of both the textbox and the button with the +

CrajyB commented 6 years ago

Also when you hover over a message in the thread view the whole message hovers white. Will this fix address this issue also?

I tried taking a screenshot but once I hover and push the screenshot keys, I lose my hover! Hopefully you guys will see the same issue in your env's.