While restoring a saved session the last URL is selected but
View Mode is not triggered. This is caused by wrong order of
switching mode and setting the current URL.
BUG: 393176
FIXED-IN: 18.04.1
rkflx |
Gwenview |
While restoring a saved session the last URL is selected but
View Mode is not triggered. This is caused by wrong order of
switching mode and setting the current URL.
BUG: 393176
FIXED-IN: 18.04.1
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Tested in a VM with Kubuntu 17.10 and with xsm.
app/mainwindow.cpp | ||
---|---|---|
1646 | Not needed since it's set in showStartMainPage(). |
While I first thought the problem has been there forever, via git blame it turns out that this is actually yet another sideeffect of D8196: Try to keep ContextManager in sync with viewed files in MainWindow, which on purpose prevented opening View without a valid URL. Your patch makes sense when you think about it in this light…
Thanks for fixing it on such short notice! LGTM
app/mainwindow.cpp | ||
---|---|---|
1644 | If you are touching this, you might as well add a const ;) | |
1646 | Agreed, but please split this out into a separate commit on master (no need for a Diff). |