1238092 - Update Mobile Activation snippet template to capture keypress events

Status: RESOLVED FIXED

(Snippets Graveyard :: General, defect)

Description

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

This bug is based on the information in bug1237350comment3.

Snippet template to update: https://snippets.allizom.org/admin/base/snippettemplate/50/

Bug 1237350 is updates to the product code to remove the keypress takeovers, but in the meantime let's see if we can make this modification to the snippet to get out of the door.

For reference:

(In reply to :Gijs Kruitbosch from comment #3)
> Finally, I believe snippets can work around this by adding an event listener
> for the "keypress" event to the textbox and calling event.stopPropagation()
> on the event. That should probably be a separate bug against the snippets,
> though.

:giorgos - would you be available to take care of this in the upcoming week?

Comment 1

[Giorgos Logiotatidis [:giorgos]]

Pull request https://github.com/mozilla/snippets/pull/77

Comment 2

[Giorgos Logiotatidis [:giorgos]]

Fix: https://github.com/mozilla/snippets/commit/9f3d68bda0eb2a9c1c023cab6dcaff3b4ef6fbd7

This is now live on production. I'll push the same fix to the snippets service so all future snippets with input boxes get the fix and then I'll revert the changes made here. 

I'll keep this bug open to track the removal as well.

Comment 3

[Giorgos Logiotatidis [:giorgos]]

Generic fix https://github.com/mozilla/snippets-service/pull/144

Comment 4

[[github robot]]

Commit pushed to master at https://github.com/mozilla/snippets-service

https://github.com/mozilla/snippets-service/commit/cdc13f87a847ca3869a37beb0569e1b2c164572c
Merge pull request #150 from glogiotatidis/bug1238092-sugardough

[bug 1241124] Prevent search box from stealing keypresses.

Comment 5

[Giorgos Logiotatidis [:giorgos]]

Template was updated and we pushed a generic fix as well.