AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

iOS: be more specific about IM callbacks to iOS

Browse Source 
No need to call textWillChange all the time if the text is
really not changing. And report that selectionWillChange
when Qt reports that it has changed.

Change-Id: I7bd9f540cd9302c37888926a6152b803cc871ccb
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@digia.com>
This commit is contained in:
Richard Moe Gustavsen 2014-01-31 15:28:07 +01:00 committed by The Qt Project
parent f9a994e707
commit 865ef95107
1 changed files with 16 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/plugins/platforms/ios/quiview_textinput.mm
View File

@ -154,27 +154,32 @@ Q_GLOBAL_STATIC(StaticVariables, staticVariables);
- (void)updateInputMethodWithQuery:(Qt::InputMethodQueries)query - (void)updateInputMethodWithQuery:(Qt::InputMethodQueries)query
{ {
// TODO: check what changed, and perhaps update delegate if the text was
// changed from somewhere other than this plugin....
// Note: This function is called both when as a result of the application changing the
// input, but also (and most commonly) as a response to us sending QInputMethodQueryEvents.
// Because of the latter, we cannot call textWill/DidChange here, as that will confuse
// iOS IM handling, and e.g stop spellchecking from working.
Q_UNUSED(query); Q_UNUSED(query);
QObject *focusObject = QGuiApplication::focusObject(); QObject *focusObject = QGuiApplication::focusObject();
if (!focusObject) if (!focusObject)
return; return;
if (!m_inSendEventToFocusObject) if (!m_inSendEventToFocusObject) {
[self.inputDelegate textWillChange:id<UITextInput>(self)]; if (query & (Qt::ImCursorPosition | Qt::ImAnchorPosition))
[self.inputDelegate selectionWillChange:id<UITextInput>(self)];
if (query & Qt::ImSurroundingText)
[self.inputDelegate textWillChange:id<UITextInput>(self)];
}
// Note that we ignore \a query, and instead update using Qt::ImQueryInput. This enables us to just
// store the event without copying out the result from the event each time. Besides, we seem to be
// called with Qt::ImQueryInput when only changing selection, and always if typing text. So there would
// not be any performance gain by only updating \a query.
staticVariables()->inputMethodQueryEvent = QInputMethodQueryEvent(Qt::ImQueryInput); staticVariables()->inputMethodQueryEvent = QInputMethodQueryEvent(Qt::ImQueryInput);
QCoreApplication::sendEvent(focusObject, &staticVariables()->inputMethodQueryEvent); QCoreApplication::sendEvent(focusObject, &staticVariables()->inputMethodQueryEvent);
if (!m_inSendEventToFocusObject) if (!m_inSendEventToFocusObject) {
[self.inputDelegate textDidChange:id<UITextInput>(self)]; if (query & (Qt::ImCursorPosition | Qt::ImAnchorPosition))
[self.inputDelegate selectionDidChange:id<UITextInput>(self)];
if (query & Qt::ImSurroundingText)
[self.inputDelegate textDidChange:id<UITextInput>(self)];
}
} }
- (void)sendEventToFocusObject:(QEvent &)e - (void)sendEventToFocusObject:(QEvent &)e