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

kimg_psd.dll should prob not be included by default #672

Open
brian6932 opened this issue Mar 7, 2024 · 7 comments
Open

kimg_psd.dll should prob not be included by default #672

brian6932 opened this issue Mar 7, 2024 · 7 comments
Labels
bug Something isn't working
Milestone

Comments

@brian6932
Copy link

brian6932 commented Mar 7, 2024

Environment:

  • OS: Windows 10
  • qView version: nightly 2023-12-27.1

Information:
I often go through my folders with arrow keys, and most of the time psd files don't actually open within qView and throw an error. Ofc I can just delete it within qView_dir/imageformats, but maybe including it by default's questionable. At the very least I feel like certain formats should just be excluded/excludable from folder seeking, so maybe this is also a feature request in disguise.

@brian6932 brian6932 added the bug Something isn't working label Mar 7, 2024
@jdpurcell
Copy link
Contributor

Similar issues: #638, #603

@brian6932
Copy link
Author

Ah I wasn't aware of your fork, nice

@brian6932
Copy link
Author

brian6932 commented Mar 8, 2024

A little off topic, but any idea why your fork (and #666) doesn't theme the titlebar anymore?
I have

$env:QT_QPA_PLATFORM
windows:darkmode=1

and it works on the nightly version used here which was built on Qt 6.2.2.
It may be specifically related to the Qt version you're using, as I've built programs with Qt >= 6.6.0 and they still theme the titlebar.

I tried this artifact for a different Qt version https://github.com/jdpurcell/qView/actions/runs/8040054619 and it also exhibits it.

@jdpurcell
Copy link
Contributor

I don't know, I'll have to see if I can reproduce it sometime. I'm mostly using macOS these days honestly. I know Qt has been working on dark mode on Windows and maybe they're forcing use of Fusion style to get the dark titlebar. Fusion style is the "Non-native theme" option in my fork by the way.

@brian6932
Copy link
Author

brian6932 commented Mar 8, 2024

they're forcing use of Fusion style to get the dark titlebar

Nah, Windows can have native dark titlebars, within DWM itself (DWMWA_USE_IMMERSIVE_DARK_MODE), Qt's supported this feature for a few years now.

Fusion style is the "Non-native theme" option in my fork

Yea I noticed fusion mode on your fork, and tried it, but wasn't able to get dark mode to work, it was just light mode, is there some separate toggle or something?


Side note, I'm down to build qView if it helps for debugging this issue, but need to refresh on qmake and .pro again, has been a few years since I last used that, Qt >= 6.0.0 uses cmake as default nowadays.

@jdpurcell
Copy link
Contributor

jdpurcell commented Mar 8, 2024

... any idea why your fork (and #666) doesn't theme the titlebar anymore?

I can reproduce this - I was testing by passing the command line argument -platform windows:darkmode=1 and got the dark title bar with Qt 6.2 but light title bar with Qt 6.5. But as I understand from Qt's blog entry and the commit here, that was an intentional change - Qt will now only use the dark title bar if the window background color is darker than the text color in the "default palette" (whatever that means*).

Yea I noticed fusion mode on your fork, and tried it, but wasn't able to get dark mode to work, it was just light mode, is there some separate toggle or something?

It's because of that darkmode=1 setting apparently, 1 means just for the window frame (when allowed by the previously mentioned logic), and 2 enables full dark mode support.

*EDIT: Default palette is just a QPalette created w/ its empty constructor, the values of which can be influenced by calling qApp->setPalette for example. To demonstrate Qt's newer logic for dark titlebars, one could run:

QPalette darkPalette;
darkPalette.setColor(QPalette::Window, QColorConstants::DarkGray);
darkPalette.setColor(QPalette::WindowText, QColorConstants::LightGray);
qApp->setPalette(darkPalette);

To force a dark titlebar even with the default style on Windows 10, but this of course is nothing more than a demonstration due to the obvious side effects.

@brian6932
Copy link
Author

2 enables full dark mode support.

Ah yes that works, nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants