Merge lp:~michael-sheldon/ubuntu-keyboard/fix-1661283 into lp:ubuntu-keyboard

Proposed by Michael Sheldon
Status: Superseded
Proposed branch: lp:~michael-sheldon/ubuntu-keyboard/fix-1661283
Merge into: lp:ubuntu-keyboard
Diff against target: 30 lines (+4/-4)
2 files modified
src/plugin/inputmethod.cpp (+2/-2)
src/view/abstracttexteditor.cpp (+2/-2)
To merge this branch: bzr merge lp:~michael-sheldon/ubuntu-keyboard/fix-1661283
Reviewer Review Type Date Requested Status
system-apps-ci-bot continuous-integration Pending
Ubuntu Phablet Team Pending
Review via email: mp+316246@code.launchpad.net

This proposal has been superseded by a proposal from 2017-02-03.

Commit message

Allow predictive text in URL mode (but disable automatic space insertion)

Description of the change

Allow predictive text in URL mode (but disable automatic space insertion)

To post a comment you must log in.
538. By Michael Sheldon

Merge Mir deprecation updates

539. By Michael Sheldon

Disable spaces on word ribbon selection, but keep them when pressing the space bar

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/plugin/inputmethod.cpp'
--- src/plugin/inputmethod.cpp 2016-12-10 01:16:27 +0000
+++ src/plugin/inputmethod.cpp 2017-02-02 16:06:50 +0000
@@ -418,9 +418,9 @@
418{418{
419 Q_D(InputMethod);419 Q_D(InputMethod);
420420
421 if (d->contentType != FreeTextContentType421 if (d->contentType != FreeTextContentType && d->contentType != UrlContentType
422 && !(d->editor.wordEngine()->languageFeature()->alwaysShowSuggestions()422 && !(d->editor.wordEngine()->languageFeature()->alwaysShowSuggestions()
423 && (d->contentType == UrlContentType || d->contentType == EmailContentType))) {423 && d->contentType == EmailContentType)) {
424 d->wordEngineEnabled = false;424 d->wordEngineEnabled = false;
425 }425 }
426426
427427
=== modified file 'src/view/abstracttexteditor.cpp'
--- src/view/abstracttexteditor.cpp 2016-08-18 13:50:14 +0000
+++ src/view/abstracttexteditor.cpp 2017-02-02 16:06:50 +0000
@@ -501,8 +501,8 @@
501 }501 }
502502
503 if (replace_preedit) {503 if (replace_preedit) {
504 if (!textOnRight.isEmpty() && d->editing_middle_of_text) {504 if ((!textOnRight.isEmpty() && d->editing_middle_of_text) || d->word_engine->languageFeature()->contentType() == Maliit::UrlContentType) {
505 // Don't insert a space if we are correcting a word in the middle of a sentence505 // Don't insert a space if we are correcting a word in the middle of a sentence or if we're in URL mode
506 space = "";506 space = "";
507 d->look_for_a_double_space = false;507 d->look_for_a_double_space = false;
508 d->editing_middle_of_text = false;508 d->editing_middle_of_text = false;

Subscribers

People subscribed via source and target branches