[LyX/master] Stop dialogs from updating twice all the time.

Richard Kimberly Heck rikiheck at lyx.org
Fri Feb 28 04:51:43 UTC 2020


commit b7509419104cef75d2c4058d9e8d497e507391ff
Author: Richard Kimberly Heck <rikiheck at lyx.org>
Date:   Fri Feb 28 00:11:45 2020 -0500

    Stop dialogs from updating twice all the time.
    
    I've long wondered why dialogs seem to update twice when it seems
    they just need to update once. It seems it was this code. But it
    was only put there to deal with a problem with code in the button
    controllers that was disabled in 2007! So I'm going to try removing
    it. If we start seeing weird behavior with dialogs in master, then
    this will be why.
---
 src/frontends/qt/Dialog.cpp |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/src/frontends/qt/Dialog.cpp b/src/frontends/qt/Dialog.cpp
index 4dbc2d4..aed484b 100644
--- a/src/frontends/qt/Dialog.cpp
+++ b/src/frontends/qt/Dialog.cpp
@@ -259,10 +259,6 @@ void Dialog::checkStatus()
 	if (canApply()) {
 		bool const readonly = isBufferReadonly();
 		enableView(!readonly || canApplyToReadOnly());
-		// refreshReadOnly() is too generous in _enabling_ widgets
-		// update dialog to disable disabled widgets again
-
-		updateView();
 	} else
 		enableView(false);
 }


More information about the lyx-cvs mailing list