[bug] Mention auto-suggest is stuk op Chrome mobile

[AW_Bos]

Ligt het aan mij of is het mentionen door de @ te typen nu stuk? Er gebeurt hier op de mobiel niks meer?

Stuk na de iteratie van dinsdag?

[DennusB]

Op desktop (MacOS met Chrome) werkt t nog wel!

[crisp]

@AW_Bos werkt hier prima, chrome op Android...

[AW_Bos]

Hmm.... raar dat hij bij mij niet meer werkt. Misschien is Chrome over de zeik?
Toch de boel weer eens rebooten na 3 weken

[_David_]

Bij mij doet ie het ook niet..

Chrome for Android 59.0.3071.125

[crisp]

Welk keyboard gebruiken jullie?

[Migrator]

Ik gebruik Gboard en kan me niet herinneren dat hij het ooit heeft gedaan
(Chrome for Android 59.0.3071.125; Android 7.0.0; Moto G 5)

[_David_]

crisp schreef op vrijdag 28 juli 2017 @ 18:46:
Welk keyboard gebruiken jullie?

"Gboard - The Google keyboard"

Migrator schreef op vrijdag 28 juli 2017 @ 19:08:
Ik gebruik Gboard en kan me niet herinneren dat hij het ooit heeft gedaan
(Chrome for Android 59.0.3071.125; Android 7.0.0; Moto G 5)

Hij heeft het hiervoor toch echt wel gedaan

[AW_Bos]

Ik toen ten tijde van de bug ook. Na een switch naar het standaard keyboard bleef het probleem. Ik heb mijn telefoon even gerebooted, en nu werkt beide prima.
Lijkt niet aan Gboard te liggen, een dat zou raar zijn anders... een @ is een @ lijkt me.

Edit:
Hm..... alsof de duivel ermee speelt
Nu werkt hij weer niet.

Lijkt iets mis in Chrome die de trigger niet activeert?

[ Voor 21% gewijzigd door AW_Bos op 28-07-2017 22:31 ]

[_David_]

Dit probleem is bij mij nog steeds aanwezig met de laatste versie van Chome for Android (v60.0.3112.107)

Heb het getest met twee verschillende keyboards:

• De standaard Google keyboard (GBoard, v6.4.16.162469584)
• De laatste swiftkey versie

Het probleem treed alleen op met Google Keyboard, via remote debugging met Chrome kwam ik erachter dat er een Javascript error optreed:


Kan helaas niet veel verder debuggen aangezien lokaal aanpassen van javascript niet werkt met remote debugging, en ik kan het ook niet reproduceren op de desktop versie van chrome...

[crisp]

_David_ schreef op zaterdag 26 augustus 2017 @ 16:30:
Dit probleem is bij mij nog steeds aanwezig met de laatste versie van Chome for Android (v60.0.3112.107)

Heb het getest met twee verschillende keyboards:

• De standaard Google keyboard (GBoard, v6.4.16.162469584)
• De laatste swiftkey versie

Het probleem treed alleen op met Google Keyboard, via remote debugging met Chrome kwam ik erachter dat er een Javascript error optreed:
[afbeelding]

Kan helaas niet veel verder debuggen aangezien lokaal aanpassen van javascript niet werkt met remote debugging, en ik kan het ook niet reproduceren op de desktop versie van chrome...

Die JS error ziet er uit als een race-conditie. Ik heb daar zojuist een fix voor online gezet. Werkt het nu wel (weer)?

[_David_]

crisp schreef op zaterdag 26 augustus 2017 @ 19:58:
[...]

Die JS error ziet er uit als een race-conditie. Ik heb daar zojuist een fix voor online gezet. Werkt het nu wel (weer)?

Heb het even snel getest op m'n mobiel, geen verschil. (Ook cache leeggegooid)

[crisp]

_David_ schreef op zaterdag 26 augustus 2017 @ 20:05:
[...]

Heb het even snel getest op m'n mobiel, geen verschil. (Ook cache leeggegooid)

Krijg je nog steeds dezelfde JS error?

edit: ah crap; onze deploytool heeft nog steeds moeite met gewijzigde statics... momentje

[ Voor 16% gewijzigd door crisp op 26-08-2017 20:23 ]

[_David_]

crisp schreef op zaterdag 26 augustus 2017 @ 20:17:
[...]

Krijg je nog steeds dezelfde JS error?

edit: ah crap; onze deploytool heeft nog steeds moeite met gewijzigde statics... momentje

Wou net melden dat ik nog steeds dezelfde error kreeg Ik wacht wel even

[crisp]

_David_ schreef op zaterdag 26 augustus 2017 @ 20:25:
[...]

Wou net melden dat ik nog steeds dezelfde error kreeg Ik wacht wel even

How about nu?

[AW_Bos]

@crisp Werkt hier!

[_David_]

crisp schreef op zaterdag 26 augustus 2017 @ 20:33:
[...]

How about nu?

"@" typen en dat een naam werkt, maar alleen "@" typen en dan de "topposters" krijgen werkt helaas nog niet. (lijkt erop alsof gboard die keypress niet doorgeeft ofzo..)

Ideegeval een verbetering

Edit: als ik vervolgens de naam die ik intypte weer weghaal, en dat dus alleen de "@" overblijft, dan krijg ik wel de topposters.

[ Voor 16% gewijzigd door _David_ op 26-08-2017 20:40 ]

[crisp]

Het lijkt er inderdaad op dat de '@' niet doorkomt via de JS keyhandler. Dat zou dan nog een bug in GBoard kunnen zijn, maar aangezien ik vakantie heb ga ik daar nu niet verder in duiken

[_David_]

crisp schreef op zaterdag 26 augustus 2017 @ 20:42:
Het lijkt er inderdaad op dat de '@' niet doorkomt via de JS keyhandler. Dat zou dan nog een bug in GBoard kunnen zijn, maar aangezien ik vakantie heb ga ik daar nu niet verder in duiken

Een paar console.log's bij de verschillende returns in keypress handler zou misschien helpen (Local javascript editing werkt niet met remote debugging in chrome...)

Maar genieten van je vakantie is ook prima

Edit: In Firefox werkt het wel gewoon, inclusief de topposters. Zal wel een bug report aanmaken.

[ Voor 8% gewijzigd door _David_ op 26-08-2017 20:49 ]

[Xander]

Dit heeft bij mij nog nooit gewerkt via Chrome Mobile? Wist niet eens dat dit hoorde te werken.

Maar nu werkt het wel

[_David_]

Bug report voor chrome: https://bugs.chromium.org/p/chromium/issues/detail?id=759304

[crisp]

Voordeel is wel dat het om een combinatie van Google producten gaat. Voor GBoard kon ik iig geen bugtracker vinden.

[AW_Bos]

Mag ik je vragen @crisp wat de fix was vanwege de vermoedelijke race-conditie die de bug triggerde? Nu werkt het eindelijk wel prima via Gboard, terwijl het vaak juist niet werkte.

[crisp]

AW_Bos schreef op zondag 27 augustus 2017 @ 12:30:
Mag ik je vragen @crisp wat de fix was vanwege de vermoedelijke race-conditie die de bug triggerde? Nu werkt het eindelijk wel prima via Gboard, terwijl het vaak juist niet werkte.

Bij het intypen van de '@' worden eerst de deelnemers van het topic opgehaald. Vervolgens wordt bij het intypen van een volgend karakter deze lijst eerst gefiltert. Als er echter nog geen lijst was dan gaf dat een foutmelding. Nu check ik eerst of die lijst wel gevuld is.

[_David_]

@crisp zie: https://bugs.chromium.org...ssues/detail?id=759304#c4

If the keyboard is sending key event directly, then @ produces keypress with the correct key value. But if keyboard composes @ (therefore underlines the text), then @ produces unidentified key (or 229 keyCode value).

SwiftKey presumably sends key event directly, so avoids this issue.

Chrome's behavior of sending unidentified key / 229 for composition is explained at issue 118639 .

The webpage should get text value directly or listen for input/composition events.

[crisp]

_David_ schreef op donderdag 31 augustus 2017 @ 19:52:
@crisp zie: https://bugs.chromium.org...ssues/detail?id=759304#c4


[...]

Hmm, dat moet ik dan maar eens rustig door gaan lezen. Er was echter een reden om keycode te gebruiken voor de detectie van @ (crossbrowser compat ws). Het lijkt er meer op dat ze het gewoon niet willen fixen terwijl het wel een hoop problemen oplevert..

[ Voor 3% gewijzigd door crisp op 31-08-2017 20:03 ]

[crisp]

@_David_: ik heb onze code nog eens bekeken maar we gebruiken helemaal geen keycode om de '@' te detecteren. De hele logica voor de mention-suggest wordt met een oninput handler gedaan die op basis van de caret positie de laatste input bekijkt. Ik zal zelf eens GBoard installeren op een van mijn tablets om te kijken wat er nou precies gebeurt daar...