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

UI: some elements in settings don't react to appearance change #1951

Closed
HeyMyian opened this issue Jun 30, 2024 · 2 comments · Fixed by #1954
Closed

UI: some elements in settings don't react to appearance change #1951

HeyMyian opened this issue Jun 30, 2024 · 2 comments · Fixed by #1954
Labels
minor issue Issue: Mostly harmless user interface Component: General user interface
Milestone

Comments

@HeyMyian
Copy link
Contributor

Here I switched from "Dracula" color theme to "Default Dark Theme" and some pink color from Dracula remains. Closing and re-opening the settings will show the correct colors, but considering that all other colors update immediately on theme change, this looks like an oversight?

The pink text also doesn't seem to immediately react to a change in application font size.

Screenshot_20240630_114434

@vkbo
Copy link
Owner

vkbo commented Jun 30, 2024

Only the main UI is updated when you change theme, not the dialog boxes. Most of them still do change due to the underlying Qt framework having its settings updated, but some text elements have novelWriter specific colour settings, like the help label text. These do not change without a restart. Neither does the window border and a few other things. It will never be a complete update without a full restart of the app. Changing font is also incomplete and needs a restart. There is a dialog box that pops up about this, and it says right there in the Preferences that it "Requires restart to take effect".

It has always been like this, so it is nothing new. There just haven't been any UI themes where the difference is this obvious before.

The only way around it is to write a ton of code that goes and updates every single custom item, which will still not fix it all. The only clean and complete update comes with a restart.

@vkbo vkbo closed this as completed Jun 30, 2024
@vkbo vkbo closed this as not planned Won't fix, can't repro, duplicate, stale Jun 30, 2024
@vkbo
Copy link
Owner

vkbo commented Jun 30, 2024

I'm re-opening this. I changed the header labels of the editor and viewer to use this same coloured help text label, and those do need to be updated on theme change. They don't have this feature, so I need to add it. The editor and viewer already have dedicated theme update methods, so in this case it's a straight forward fix.

@vkbo vkbo reopened this Jun 30, 2024
@vkbo vkbo added user interface Component: General user interface minor issue Issue: Mostly harmless labels Jun 30, 2024
@vkbo vkbo added this to the Release 2.5 milestone Jun 30, 2024
@vkbo vkbo mentioned this issue Jul 3, 2024
6 tasks
@vkbo vkbo linked a pull request Jul 3, 2024 that will close this issue
6 tasks
@vkbo vkbo closed this as completed Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor issue Issue: Mostly harmless user interface Component: General user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants