Tricky because of the embedded widget for save options.
Reused the solution found in kuickshow.
Since QFileDialog implements overwrite checking, kill all
the custom code for doing the same.
Details
Details
- Reviewers
broulik andreagenor tcanabrava - Maniphest Tasks
- T11576: Port kolourpaint away from kdelibs4support
- Commits
- R374:ea2d7440b0a8: Port from KFileDialog to QFileDialog.
Save as (new file), save as (existing file)
Diff Detail
Diff Detail
- Branch
- my_fixes
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 19083 Build 19101: arc lint + arc unit
| mainWindow/kpMainWindow_File.cpp | ||
|---|---|---|
| 887 | Is there hope that Qt could be tricked into supporting native file dialogs that allow Qt widgets as extensions? I am not really happy with the pure Qt save dialog. It lacks all the bookmarks from the Places panel. | |
| mainWindow/kpMainWindow_File.cpp | ||
|---|---|---|
| 887 | Ouch, this "not use native dialog" is a *severe* usability regression. At first I thought it was a bug that it was using the Qt dialog. There's no editable address bar, there's no places panel, the mime type detection based on input filename (e.g. if I input ".jpg" it should switch to save as JPEG). | |
Comment Actions
I took this solution from kuickshow (using lxr), but clearly that must be less often used than kolourpaint :)
From previous discussions I think the best thing to do is to split this out of the filedialog. Either asking in a separate dialog afterwards, or turning this into global settings in the app config dialog. What's preferred?
Comment Actions
I fiddle with the quality selector for JPEG regularly. However, given it prompts "saving in this format might lose information" afterwards usually, having a dialog with that warning and the controls coming after the save dialog wouldn't be too bad.
However for PNGs I am not sure I want it asking about color depth every time - maybe we could represent that using different mine types? Have PNG (24 bit), PNG (8 bit) etc as additional fake mime types if that's possible?
Comment Actions
I just again accidentally saved something as JPG instead of PNG because typing "foo.png" doesn't automatically switch file type in Qt's dialog.. can we please get this thing back? :(
Comment Actions
It doesn't automatically append extension either.
If we cannot upstream our features to Qt, we need to keep our classes. We can move them back from kdelibs4support into supported frameworks.
Comment Actions
I'd rather move the settings out of the filedialog. It's just not a portable solution.
I'm not sure about fake mimetypes though, that would mean 5 (not 3) PNG subtypes, too easy to select the wrong one (especially if you don't know much about color depths). At least currently you can ignore the combo and keep the sensible default.
How about we just have a separate dialog afterwards? Much like OpenOffice has a whole PDF settings dialog (which I completely ignore every time).
We *could* have a dontshow checkbox, not sure.