diff options
author | Richard Moe Gustavsen <richard.gustavsen@theqtcompany.com> | 2016-05-09 12:36:37 +0200 |
---|---|---|
committer | Richard Moe Gustavsen <richard.gustavsen@theqtcompany.com> | 2016-05-09 11:38:44 +0000 |
commit | c906d7abb84e60d2d4b66edce712cb32b3ce4470 (patch) | |
tree | 14a24b7ac92443946d7aa5d96083dd0c57c80db5 /src/widgets/widgets/qwidgettextcontrol_p_p.h | |
parent | f643f6504bf2233c7a5c937be620daf944bb0f61 (diff) |
QWidgetTextControl: ensure we listen for changes to cursorFlashTimeChanged
Change 3cdc02d actually "reverted" the behavior of listening for
cursorFlashTimeChanged. The reason is that we sat "blinkingEnabled"
directly to true in the constructor instead of calling
"setCursorBlinking", which was responsible for setting up the
connection.
And as it turns out, after 3cdc02d, nobody is actually calling
"setBlinkingCursorEnabled" anymore. From the widgets
point of view, it should always blink when visible
(unless QPA sets cursorFlashTime to zero). So we can remove the whole
function, and set up the connection in "setVisible" instead.
Change-Id: I577a5fbbbd9c56331ac7f8bb38567a684ca8c1df
Reviewed-by: Jan Arve Sæther <jan-arve.saether@theqtcompany.com>
Diffstat (limited to 'src/widgets/widgets/qwidgettextcontrol_p_p.h')
-rw-r--r-- | src/widgets/widgets/qwidgettextcontrol_p_p.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/widgets/widgets/qwidgettextcontrol_p_p.h b/src/widgets/widgets/qwidgettextcontrol_p_p.h index e9b3589c5c..fea72d98ff 100644 --- a/src/widgets/widgets/qwidgettextcontrol_p_p.h +++ b/src/widgets/widgets/qwidgettextcontrol_p_p.h @@ -177,7 +177,6 @@ public: QTextDocument *doc; bool cursorOn; - bool blinkingEnabled; bool cursorVisible; QTextCursor cursor; bool cursorIsFocusIndicator; |