ubuntu-keyboard

TextArea generates onTextChanged events when OSK is displayed or screen is rotated

Bug #1243164 reported by Ugo Riboni
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-keyboard
Fix Released
Undecided
Unassigned

Bug Description

In the attached test case the Label at the bottom counts the number of times the TextArea received an onTextChanged event.

Steps to reproduce:
* Tap on the TextArea to start editing: the count increases by one or more while the OSK shows up
* While the OSK is visible, rotate the device: the count increases further

Expected result:
None of the above should happen and the count should never change unless the user types in something with the OSK.

(tested on maguro on build 100)

Related branches

lp://staging/~schwann/ubuntu-keyboard/keyboard-rotate-update
Thomas Moenicke (community): Approve
Diff: 14 lines (+4/-0)
1 file modified
src/view/abstracttexteditor.cpp (+4/-0)
Revision history for this message
Ugo Riboni (uriboni) wrote :
Revision history for this message
Ugo Riboni (uriboni) wrote :

The same issue happens replacing TextArea with TextEdit

Revision history for this message
Günter Schwann (schwann) wrote :

Works for me as expected with a clean flashed galaxy nexus. Flashed via "phablet-flash ubuntu-system --channel trusty"

Revision history for this message
Olivier Tilloy (osomon) wrote :

Just ran the test on an a trusty-based image (r11), and I can confirm the problem.

By the way, I think entering just one character in the field should generate only one textChanged event, but apparently it triggers at least 4 of them, as evidenced by the counter increment.

Changed in ubuntu-ui-toolkit:
status: New → Confirmed
Changed in ubuntu-keyboard:
status: New → Confirmed
Revision history for this message
Thomas Moenicke (thomas-moenicke) wrote :

@olivier: how exactly did you run the example?

Revision history for this message
Olivier Tilloy (osomon) wrote :

cd /tmp
wget https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1243164/+attachment/3887315/+files/test2.qml
adb wait-for-device
adb push test2.qml /tmp
adb shell
sudo -u phablet -i
/usr/lib/arm-linux-gnueabihf/qt5/bin/qmlscene /tmp/test2.qml --desktop_file_hint=/usr/share/applications/webbrowser-app.desktop

Revision history for this message
Günter Schwann (schwann) wrote :

When having the predictive text enabled, this issue doesn't show up anymore:
adb shell sudo -iu phablet gsettings set com.canonical.keyboard.maliit predictive-text true

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Fixed back in Nov

Changed in ubuntu-keyboard:
status: Confirmed → Fix Released
no longer affects: ubuntu-ui-toolkit
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.