Juergen Spitzmueller wrote: > It came out that the problem is caused by refreshReadOnly(), called > (indirectly) by Dialog::checkStatus(). refreshReadOnly disables all > widgets (that are addReadOnly()'ed) in read only mode and enables > all these widgets in non-read-only mode. The latter, which is > necessary to switch modes, overrides the intiial disabling of > certain widgets by the dialogs. > > The attached patch just calls the dialog's update() after all > widgets have been enabled by refreshReadOnly, so that the widgets > that should be disabled actually get disabled. > > I have tested all scenarios I could think of (non-read-only, > read-only and switches between those two modes) and saw no problems. > > Comments? > > Jürgen
Hi, Jürgen. Sorry I didn't manage to get back to your bugzilla query earlier. Now all is explained, I can see why this patch is needed and why it will work. The solution looks reasonable to me. -- Angus
