Skip to content
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 Support #15

Merged
merged 4 commits into from May 18, 2019
Merged

Theme Support #15

merged 4 commits into from May 18, 2019

Conversation

vkbo
Copy link
Owner

@vkbo vkbo commented May 18, 2019

Syntax colours are now saved in a config file, that together with the main style sheet, lives in the folder nw/themes/default. Other folders can be added with other sets of files, and the folder name used as the theme setting in the config file.

The folder must contain the files styles.css and theme.conf.

This PR also moves the editor and viewer into its own splitter, and sets the headers in the viewer to the same colour as in the editor.

@codecov-io
Copy link

codecov-io commented May 18, 2019

Codecov Report

Merging #15 into master will increase coverage by 0.35%.
The diff coverage is 91.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   76.67%   77.03%   +0.35%     
==========================================
  Files          22       23       +1     
  Lines        2847     2939      +92     
==========================================
+ Hits         2183     2264      +81     
- Misses        664      675      +11
Impacted Files Coverage Δ
nw/gui/docviewer.py 100% <100%> (ø) ⬆️
nw/gui/dochighlight.py 100% <100%> (ø) ⬆️
nw/gui/winmain.py 70% <100%> (ø) ⬆️
nw/config.py 95.28% <100%> (+0.3%) ⬆️
nw/gui/doceditor.py 64.94% <100%> (ø) ⬆️
nw/theme.py 85.52% <85.52%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a90864...251614e. Read the comment docs.

@vkbo vkbo merged commit d89d96f into master May 18, 2019
@vkbo vkbo deleted the themes branch May 18, 2019 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants