[PATCH 2/4] Make some dialogs modal to MainWindow
Lubomir I. Ivanov
neolit123 at gmail.com
Tue Mar 11 12:20:13 PDT 2014
On 11 March 2014 20:23, Dirk Hohndel <dirk at hohndel.org> wrote:
> On Tue, 2014-03-11 at 18:30 +0200, Lubomir I. Ivanov wrote:
>> From: "Lubomir I. Ivanov" <neolit123 at gmail.com>
>>
>> For some dialogs like Preferences and Renumber it would be
>> much safer to make them Qt::WindowModal to the parent (MainWindow).
>
> Discuss
some notes on all the changed files and modality:
- qt-ui/divecomputermanagementdialog.ui
this one can be a bit weird if the user has opened the dialog and then
inserted a new DC, the name will not update in the list.
possibly should be modal.
- qt-ui/preferences.ui
if the prefs. are safe and that was the original intention, we can
leave the dialog non-modal.
- qt-ui/renumber.ui
this one is weird, because the user can click back to edit the dive
and the renumber will not work during that.
- qt-ui/shiftimagetimes.ui
- qt-ui/shifttimes.ui
possibly both should be modal.
---
so perhaps the patch can be modified to exclude the preferences dialog change?
lubomir
--
More information about the subsurface
mailing list