aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/inputlogic (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Don't resume suggestion on links.Jean Chalard2014-02-171-0/+4
| | | | | Bug: 11447084 Change-Id: I5bd558b9dd85d1505aa918f44e8ac3e52ec42d97
* Add PunctuationSuggestions classTadashi G. Takaoka2014-02-141-2/+2
| | | | | | | This change must be checked in together with I366c477516. Bug: 12640318 Change-Id: I8d3864b9d9649c9e3306e82b01a6327b11281b1f
* Backward compatibility hack : send delete when empty.Jean Chalard2014-02-131-1/+7
| | | | | | | | | | | | | | | | | | | | Some apps depend on the keyboard sending something to them when the text is empty. This is BROKEN. Your app must not lie to the keyboard about what is before the cursor. If there is something to delete, you must not pretend there is not and try to catch some delete event. This will result in a bad user experience. This will not work with all IMEs. If your app needs this broken behavior, you're doing it wrong. Seriously guys, we're not in the era of typewriters any more, there are touch screens, there are gestures, there is accessibility, there are many innovative IMEs that don't have keys. Do *NOT* rely on key events. This change implements an ugly hack so that these broken apps may continue half-working with LatinIME. We are very unhappy about this. Bug: 12998568 Change-Id: Ia62ae2fbee4fee65b463acf3a79aafcfd0defa1d
* Fix a test.Jean Chalard2014-02-131-17/+29
| | | | | | | | | | This fixes two separate problems: - The word finds itself with two separate suggestion spans. This is fine for LatinIME, but it's hard to predict whether it's fine for other interested parties (other keyboards). - The test for the blue underline was incorrect. Change-Id: I3ecc849676851bf25a25238d694adaa956521a26
* Revert "Display predictions when the cursor is moved after a word."Jean Chalard2014-02-101-8/+1
| | | | | | | | This change is lacking some comments and break some unit tests. It needs more work. This reverts commit 38d31a5e79c64e706e1698d2d24adb51c4ac53aa. Change-Id: I675854fd0729f2d01b7751e35c6d0117f4f88993
* Display predictions when the cursor is moved after a word.Jean Chalard2014-02-101-1/+8
| | | | | Bug: 12951777 Change-Id: I744b98df6679edd34ca3c1392e38597325935932
* Conditionally store the raw suggestions.Jean Chalard2014-02-061-1/+2
| | | | | Bug: 12913520 Change-Id: Ie15f97d90c142c51089280252b35b5ad1d226e19
* Merge "Fix a flash in the wrong place"Jean Chalard2014-02-051-3/+2
|\
| * Fix a flash in the wrong placeJean Chalard2014-02-051-3/+2
| | | | | | | | | | Bug: 7226099 Change-Id: I8f74802b3ae24018fc4176fb2fff2c777ae170b0
* | [IL109] Move more code to InputLogicJean Chalard2014-02-051-1/+113
| | | | | | | | | | Bug: 8636060 Change-Id: Ib12eacc836e8d39c4165c97a2a287f32f6bf5405
* | [IL108] Remove some uses of LatinIME in input logic.Jean Chalard2014-02-051-7/+11
|/ | | | | | | Access the suggestion strip through the interface instead. Bug: 8636060 Change-Id: Ia8bd705c230f943e309d677ef683b1a2cbe68036
* Check isEligibleForAutoCommit when batch input is ended.Keisuke Kuroyanagi2014-02-051-1/+4
| | | | | Bug: 11951198 Change-Id: Ibbe09fa38eaa3ebf8f9b69f744a29db7ac598975
* Don't resume suggestions if suggestions are not requested.Jean Chalard2014-02-031-0/+2
| | | | | Bug: 12832614 Change-Id: I6a9b6d128d7e0324ba59a43cf153a3b54e1d35bf
* [IL106] Inline a now useless method.Jean Chalard2014-01-291-3/+4
| | | | | Bug: 8636060 Change-Id: Ic8e0225206358680aa0de421f17ab443c888036d
* [IL105] Remove more useless argumentsJean Chalard2014-01-291-3/+2
| | | | | Bug: 8636060 Change-Id: I9e817bb9622c84544a5ca8708e5c1fccad1ed447
* [IL104] Remove useless argumentsJean Chalard2014-01-291-5/+2
| | | | | Bug: 8636060 Change-Id: I97a371fe0c6227eb0f03d3e391a33726e6280322
* [IL103] Save the typed word separately in SuggestedWords.Jean Chalard2014-01-291-3/+2
| | | | | | | | This helps managing the cases where the typed word is not in the suggestions. This happens during recorrection. Bug: 8636060 Change-Id: I6784feb793cae96272a7f1d123a0e3bbb8f03143
* Merge "Fix inconsistent behavior with the back-to-the-main-keyboard key"Yohei Yukawa2014-01-281-0/+4
|\
| * Fix inconsistent behavior with the back-to-the-main-keyboard keyYohei Yukawa2014-01-281-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | With this patch, the back-to-the-main-keyboard in the Emoji palette will be registered as a key-release action instead of a key-press action, like switch-to-the-emoji-palette in the main layout. This provides mroe consistent UX when the layout is switched from the main layout to the Emoji palette then switched back to the main layout. BUG: 12464067 Change-Id: Ia0d0185db43234dfcfb7cee2677f3d199fe6ed96
* | [IL102] Rename a methodJean Chalard2014-01-281-3/+2
| | | | | | | | | | Bug: 8636060 Change-Id: Ia54afdfab9124aa5c35699e357e63eae66a7557f
* | [IL101] Remove a methodJean Chalard2014-01-281-3/+6
| | | | | | | | | | Bug: 8636060 Change-Id: Ic9c12fc6970987b0e1912e8ae85bed55260d66b5
* | [IL99] Migrate some code to InputLogic.Jean Chalard2014-01-281-0/+19
|/ | | | | Bug: 8636060 Change-Id: Ibf1f9c7bdc3fdd872d6c15447fa555434485cc52
* Merge "[IL86] Simplification and correctness fix."Jean Chalard2014-01-271-5/+1
|\
| * [IL86] Simplification and correctness fix.Jean Chalard2014-01-271-5/+1
| | | | | | | | | | | | | | This should be what was intended. Bug: 8636060 Change-Id: I20a1cc4f0b1fb9444fb6f9a20ef727e3d1a3a880
* | Merge "[IL84] Remove a useless call and rename a method"Jean Chalard2014-01-271-2/+2
|\|
| * [IL84] Remove a useless call and rename a methodJean Chalard2014-01-271-2/+2
| | | | | | | | | | Bug: 8636060 Change-Id: I5cac787953a3c71b6402520c2f5c70619d4a40a8
* | Merge "[IL81] Remove a useless test."Jean Chalard2014-01-271-12/+3
|\|
| * [IL81] Remove a useless test.Jean Chalard2014-01-271-12/+3
| | | | | | | | | | | | | | | | It seems the comments were right all along. This seems to really be useless. Bug: 8636060 Change-Id: Ie030e9cb73007334380d023abeaa4ce52a863a34
* | Merge "Try to figure out whether d.quotes open or close."Jean Chalard2014-01-271-5/+21
|\ \ | |/ |/|
| * Try to figure out whether d.quotes open or close.Jean Chalard2014-01-271-5/+21
| | | | | | | | | | Bug: 8911672 Change-Id: I5d5635949530a67f95e5208986907251b7bce903
* | [IL80] Reverse a test for clarity.Jean Chalard2014-01-271-50/+49
| | | | | | | | | | Bug: 8636060 Change-Id: I5092942ab4fd6aaf37023083040cadbc18583fd1
* | [IL79] Some refactoringJean Chalard2014-01-271-18/+10
| | | | | | | | | | Bug: 8636060 Change-Id: I931c553f68a1f15d22711a661dbffd4e5d421979
* | [IL78] Move most of onUpdateSelection to InputLogic.Jean Chalard2014-01-271-0/+79
| | | | | | | | | | Bug: 8636060 Change-Id: I7290250637e37bf6964bab71d2141e19f20a5459
* | [IL77] Remove refs to Settings from InputLogicJean Chalard2014-01-271-9/+3
| | | | | | | | | | Bug: 8636060 Change-Id: I4ce5c9b62181ce8593a453ce222ae7d1fdc08f07
* | [IL75] Move the SubtypeSwitcher out of InputLogic.Jean Chalard2014-01-271-5/+4
|/ | | | | Bug: 8636060 Change-Id: I2a5da27e73cf378e55bd80ab2054c1ba8b13f6ec
* Merge "[IL76] Remove a SuggestionStripView dependency"Jean Chalard2014-01-231-3/+2
|\
| * [IL76] Remove a SuggestionStripView dependencyJean Chalard2014-01-231-3/+2
| | | | | | | | | | | | | | | | This constant is better located in SuggestedWords. Or it could be in Constants, that would be fine too. Bug: 8636060 Change-Id: I3e721bb1e2559f028dce8929eceadfe0478c5924
* | [IL66.1] Add a commentJean Chalard2014-01-231-1/+1
|/ | | | | | Followup to IL66 Change-Id: I067e72c36f483cc3e3b24736c01843531298b507
* [IL70] Remove unused argumentsJean Chalard2014-01-231-11/+7
| | | | | Bug: 8636060 Change-Id: Iec56bce362ebfc52fedab02b127978abb4f3c7f9
* [IL69] Oust Keyboard from WordComposer#setComposingWordJean Chalard2014-01-231-6/+9
| | | | | Bug: 8636060 Change-Id: I624a45500603dfad355ee3b8b794b2d895219c0b
* [IL67] Externalize a call to the keyboard switcherJean Chalard2014-01-231-9/+6
| | | | | Bug: 8636060 Change-Id: Ib13a54854e30afb79217a0a1f2aeca50a95ba155
* [IL66] Remove two accesses to KeyboardSwitcher.Jean Chalard2014-01-231-7/+7
| | | | | Bug: 8636060 Change-Id: I3b144993e2bcd552f9b292b8f057c9db4216125c
* [IL64] Pull up X,Y processing, step 3Jean Chalard2014-01-221-20/+4
| | | | | Bug: 8636060 Change-Id: Ic051e5d5514d270101b0571a2d30e2caa8f85bc1
* [IL63] Pull up X,Y processing, step 2Jean Chalard2014-01-221-20/+20
| | | | | | | | This also fixes the coordinates passed to logOnDelete and log onSeparator. Bug: 8636060 Change-Id: I4703d61ad7de76b501adca18df3a69e06450c9a1
* [IL62] Pull up X,Y processing, step 1Jean Chalard2014-01-221-8/+9
| | | | | | | | | | | | | | | | | | This is fine because getKey{X,Y} is idempotent for any non-keyboard coordinate value. However this makes a net change : the x,y values passed to LatinImeLoggerUtils.onNonSeparator are now different. The point is however that they used to be wrong. The logged values used not to account for the keyboard padding that is present on tablets, and in the simulating tools we don't know about that padding so we couldn't use the coordinates. The catch here is that other calls like LoggerUtils.onSeparator should follow suit, but this is too big a change to do it at once. Follow-up changes will fix them too. Bug: 8636060 Change-Id: If4b3d3cb1ed4b44c35f23e66aba3b5797236bba7
* Merge "[IL61] Remove an internal use of key coordinates"Jean Chalard2014-01-221-9/+9
|\
| * [IL61] Remove an internal use of key coordinatesJean Chalard2014-01-171-9/+9
| | | | | | | | | | Bug: 8636060 Change-Id: Id87b0ac4a99ec6a26db6e250d6cd22defc02a23d
* | Use sorted int[] to represent word separatorsTadashi G. Takaoka2014-01-201-2/+3
| | | | | | | | Change-Id: I4103541d99fe59bfcf12379a1298a0a690497846
* | Refactor logic to retrieve relevant suggestions a bitTadashi G. Takaoka2014-01-201-1/+2
| | | | | | | | Change-Id: Ic7d2cbb2c1b2deaa4e735484bdc7413c0b3b1939
* | Refactor RichInputConnection a bitTadashi G. Takaoka2014-01-171-4/+5
|/ | | | Change-Id: I866488a47ca04ca587e805663dfd597bb7b1ebce