Review Request 120934: Fix Krita Gemini/Sketch window titles for filenames with %1 or %2 inside
Friedrich W. H. Kossebau
kossebau at kde.org
Thu Nov 13 20:23:08 GMT 2014
> On Nov. 13, 2014, 8:03 p.m., Boudewijn Rempt wrote:
> > Hm, I'm sure I haven't thought through the entire problem but one thing stood out: with KF5's deprecation of kdialog, should we really add code that uses it?
Yes, we should :) Because in Qt4 world KDialog is doing what only starting in Qt5 world Qt is doing itself, i.e. adding the app name to the window title in a consistent way.
See the extra comment at the end on http://qt-project.org/doc/qt-5/qwidget.html#windowTitle-prop which is missing from http://qt-project.org/doc/qt-4.8/qwidget.html#windowTitle-prop
So in the port to Qt5 the call to KDialog::makeStandardCaption() can be simply removed. So no real extra burden for the port, and consistent (bugfree) titles for 2.9.
- Friedrich W. H.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120934/#review70344
-----------------------------------------------------------
On Nov. 1, 2014, 3:28 a.m., Friedrich W. H. Kossebau wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120934/
> -----------------------------------------------------------
>
> (Updated Nov. 1, 2014, 3:28 a.m.)
>
>
> Review request for Calligra, Arjen Hiemstra, Dan Leinir Turthra Jensen, and Boudewijn Rempt.
>
>
> Repository: calligra
>
>
> Description
> -------
>
> When testing Calligra Gemini with the Calligra test document set, I was wondering why "CalligraGemini" suddenly was inside the filenames displayed in the title. Reason is that the window title composing is custom made, using "%1" and "%2", strings which can be also found in the filenames of the test document set, as they are created from encoded origin urls.
>
> And the code of Calligra Gemini is the same as in Krita Gemini and Krita Sketch.
>
> Attached patch proposes to just create the window title by falling back to the code KMainWindow also uses:
> http://api.kde.org/4.x-api/kdelibs-apidocs/kdeui/html/kmainwindow_8cpp_source.html#l00552
>
> There is one issue though, which I need input for. I suspected this line actually will be called if the current document is modified:
> ```
> connect(DocumentManager::instance(), SIGNAL(documentChanged()), SLOT(resetWindowTitle()));
> ```
> But seems that the signal `documentChanged()` is only about that there was a switch which document is loaded.
>
> So even after this patch the window title will never show the "[modified]" flag, which is different from how Desktop Krita behaves.
> Intended (so the respective lines could be removed from this patch), or a bug?
> If not intended, how and where should the modified signal be registered to and unregistered from?
>
>
> Diffs
> -----
>
> krita/gemini/MainWindow.cpp d74c12c
> krita/sketch/MainWindow.cpp 98a6c02
>
> Diff: https://git.reviewboard.kde.org/r/120934/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Friedrich W. H. Kossebau
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141113/6e6e5d91/attachment.htm>
More information about the calligra-devel
mailing list