rscodeio icon indicating copy to clipboard operation
rscodeio copied to clipboard

Theme doesn't work with other editor themes

Open salim-b opened this issue 4 years ago • 7 comments

First of all, awesome work, thanks a lot!

Now, while I love your dark menu palettes (and especially the fact that you provide GNOME-specific style sheets!), I don't like the rscodeio editor theme and prefer to stick to Tomorrow Night Bright instead.

Although in #18 you state

Once installed, the menu theme is applied when any dark editor theme is applied;

this is not true for me. The menu theme (except the top menu bar) reverts back to Modern's default ugly blue colors as soon as I switch the editor theme to anything other than rscodeio.

Screenshot (note the top menu bar which is still dark):

Bildschirmfoto von 2020-06-20 22-53-21

Maybe this is because I'm using the latest development version of RStudio (~~1.3.1039~~ 1.3.1050 at the time of writing)?

Excerpt of xfun::session_info():

R version 4.0.2 (2020-06-22)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 20.04 LTS, RStudio 1.3.1050

Locale:
  LC_CTYPE=de_DE.UTF-8       LC_NUMERIC=C               LC_TIME=de_CH.UTF-8        LC_COLLATE=de_DE.UTF-8    
  LC_MONETARY=de_CH.UTF-8    LC_MESSAGES=de_DE.UTF-8    LC_PAPER=de_CH.UTF-8       LC_NAME=C                 
  LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=de_CH.UTF-8 LC_IDENTIFICATION=C

salim-b avatar Jun 20 '20 21:06 salim-b

The menu theme is the top menu bar; this is what is changed when rscodeio::activate_menu_theme() is executed. All of the remaining UI elements are styled with the editor theme.

anthonynorth avatar Jul 01 '20 10:07 anthonynorth

I see... so this means, there's currently no way to combine your UI element modifications besides the syntax highlighing theme with the default editor themes provided by RStudio, right?

I guess it would then be a simple copy-paste effort to create an editor theme with all of your awesome UI element mods but with Tomorrow Night Bright syntax higlighting? Would you accept a PR introducing such a second editor theme (with the name Tomorrow Night Bright (rscodeio) for example)?

salim-b avatar Jul 05 '20 19:07 salim-b

@salim-b Were you able to get around this, I am running into the same issue and want to run a different editor theme but keep all the styling.

Thanks

raahimSal avatar Sep 20 '20 19:09 raahimSal

@salim-b yes that will be ok & sorry for the (very) delayed response.

anthonynorth avatar Sep 21 '20 00:09 anthonynorth

If possible, could the file be changed in a way where the editor themes can just be copy pasted from other sources yet all the other styling remains the same? Haven't done anything like this before so just wondering, Thanks for such a nice theme btw love this alot!

raahimSal avatar Sep 21 '20 01:09 raahimSal

It is possible, but given the way I hacked the stylesheet together, it's probably going to be painful. There are some css variables defined at the top of the stylesheet which you could take a look at; I expect it's incomplete for what you want though. I had intended to refactor that stylesheet into a postcss pipeline, but I haven't had time to complete it, unfortunately.

anthonynorth avatar Sep 22 '20 08:09 anthonynorth

@salim-b Were you able to get around this, I am running into the same issue and want to run a different editor theme but keep all the styling.

I've just submitted #44 which adds a theme named Tomorrow Night Bright (rscodeio).

You can use one of RStudio's default themes and complement it by the necessary CSS from rscodeio if you like.

salim-b avatar Oct 09 '20 09:10 salim-b