John Spray wrote: > The spellchecker dialog. > > In GViewBase it was necessary to move the update() to after show() > because with the spellchecker update()ing sometimes involves immediately > cancelling the dialog (if there are no spelling errors to prompt the > user about).
That sounds like a poor UI to me. I try and spell check and the dialog appears and then disappears? Why not, the dialog appears but the "start checking" button is disabled? Your change if (!window()) { build(); } - update(); window()->show(); + update(); } will presumably lead to an unnecessary flicker for most dialogs... -- Angus