Steps to reproduce:

Currently replicated on Android 6.0.1 and 7.1.1.

  1. Find an autocomplete field e.g. Authoring information > Authored by: /node/add/page
  2. Type in a partial, valid username

Expected behaviour:

A username should appear.

What is happening:

It appears no callback is triggered and therefore no usernames appear. However once a whole word is typed and space entered, the callback is triggered.

The correct behaviour is experienced on the jQuery UI autocomplete page at http://jqueryui.com/autocomplete/

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

oliverpolden created an issue. See original summary.

oliverpolden’s picture

Issue summary: View changes
droplet’s picture

Version: 8.3.7 » 8.3.x-dev
Related issues: +#2823589: Improve IME handling on Autocomplete

Hmm. It seems Android's keyboard working in IME mode and `compositionend` event is not fired.

droplet’s picture

Assigned: Unassigned » droplet

Assigned to me. I will work out a patch in next 1 hour when I back to my desktop :)

We should exclude Android in this case :(

droplet’s picture

Assigned: droplet » Unassigned
Status: Active » Needs review
Issue tags: +Needs manual testing, +Android issue
FileSize
3.23 KB

Try it. Simple reload in Android Chrome might not clear cache properly. I highly recommended removing cache on browser setting or Enable "Aggregate JavaScript files" and "Clear all caches" in Drupal ( admin/config/development/performance )

EDIT: It's cross-browsing issue, no bot test.

Status: Needs review » Needs work

The last submitted patch, 5: 2909128-5.patch, failed testing. View results

droplet’s picture

Version: 8.3.x-dev » 8.5.x-dev
oliverpolden’s picture

Thanks Kay,

I can confirm that is working for me. Thank you for such a quick fix!

The patch included changes to an es6 file that I don't believe exists in core.

Oliver

droplet’s picture

Status: Needs work » Needs review

@oliverpolden it's a new age :)

@see: https://www.drupal.org/node/2815083

oliverpolden’s picture

Ahh, thanks Kay, was unaware of that. I am however getting an "isAndroid is not defined" error.

droplet’s picture

Ahh. What's your browser version? Did you patch your D8.3 with ES6 version?

It seems impossible. The indexOf used almost everywhere in D8.

(Even said if you use ES6, the Const also well-supported in very very older Chrome, only could be a problem back to around Android browser 4.4)

oliverpolden’s picture

Kinda my bad. I just checked the patch file and found that it didn't apply the addition of the variables at the top of the file so I've now applied them manually and all is good. The patch not applying cleanly could be to do with me using Pantheon.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.