-
-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Theme contribution] Cyberpunk Night (an AMOLED black theme) #1730
Comments
Looks great, I'm happy to take this as a contribution. Could you possibly make a pull request for it? There are a few review points to go through for new themes. |
I realise you may not be familiar with GitHub, since you just created the account. If so, I can of course do it. |
@vkbo You're right. I'm not that familiar with Github or with any special requirements for this project, so if you could do it that would be appreciated. 👍 I noticed that there's a new version upcoming, so if anything changed regarding the way themes work let me know and I'll update accordingly and test it with the AppImage of the new version. Also feel free to let me know if anything looks broken on systems other than Linux since I've only got that to test with. |
Updated the syntax theme because it was missing the "optional" tag as it isn't included on https://novelwriter.io/docs/int_customise.html - I only found it by inspecting one of the original themes. 😃 |
The "optional" entry is a 2.3 feature, which has not been released yet. That's why it is not in the documentation yet. |
Changed errorline to be different from spellcheckline (using the color from the night_owl theme). |
It would be better if the spell check line was consistent with the other colours rather than copied from a different theme. Also, you can just paste the text here instead of the tar.gz files. Just wrap them in three backticks like so: ```. I would also prefer if you didn't set multiple colour settings that are used to distinguish different elements to the same colour. It defeats some of the purpose of syntax highlighting.
|
@vkbo I'm editing the colors to create more separation between the tag types. Is there a sample text (that includes all available tag types) that I can use? There are some tags like "replacetag" that I haven't been able to test yet. |
You can create an example project from "new project" which will have everything demonstrated. In the example project, the document "Making a Scene" has most elements showcased. |
@vkbo Thanks! One tag I can't find is "modifier" - how can I make some text that uses it? Also, I'm not sure which tag that corresponds to non-breaking spaces. It seems to use a color that I haven't defined. |
"modifier" is the colour applied to the "%Synopsis:" part of your screenshot. It is referring to the special modifiers for comments. |
You can insert non-breaking spaces from the Insert menu. They should be shown with a different background colour than the regular background. It's the same thing that LibreOffice does. That's where the feature is from. |
@vkbo I figured out that non-breaking spaces take their color from the "emphasis" tag. Not sure if that's intended or something I should raise an issue for, but at least it allows me to complete the theme now. 😄
|
Yeah, that's probably right. The theme files are processed here in the code: novelWriter/novelwriter/gui/theme.py Line 276 in a3bd0d3
And applied to the document here:
You can track which colour is used for what. There are many colours that are used for multiple things. |
I've tried to create some more visual separation between the tag types here, and hopefully I managed to catch everything:
|
Looks good! I still think the comments need to be a bit easier to read. That is, the "hidden" colour. Also, for the GUI theme, you must set "window", "base" and "alternatebase" to different shades. In particular "alternatebase" is used for showing which headings are in the currently open document in the Novel view. If it's the same colour as the background, this feature doesn't work. It is less of a problem that "window" and "base" are the same shade. |
After trying a few variations I will need to stick with "0,0,0" for "window" and "base" to keep the AMOLED Black approach, but I can see what you mean regarding "alternatebase" and put together a new version of the GUI theme (some other tags were slightly tweaked for increased visibility as well):
|
Great! I'll test it out as well. In version 2.4 there will also be the need for a colour for highlighted text in the syntax theme. The implementation only exists in the dev branch for now, here's an example entry: When 2.4 is ready, a highlight colour needs to be added to your syntax theme too. In 2.4 it will be possible to define an alpha channel colour as a fourth value in the theme files too. |
@vkbo Once you need updates for 2.4, could you please create an issue and assign it to me? Thanks! |
Sure, no problem. Do you mind just creating a fork of this repo and contribute it as a pull request? There may be future changes as well. I believe this is simple enough that you can edit the theme files directly on your fork of the repo on the GitHub website and don't need the full git setup on your computer to do it. |
I'll try to figure that out. |
I've created an AMOLED black/Cyberpunk style theme that is easy on the eyes and perfect for late night writing sessions.
Feel free to include it in the default selection of themes since there's currently a lack of dark mode themes, especially AMOLED black ones.
cyberpunk_night.tar.gz
The text was updated successfully, but these errors were encountered: