Page MenuHomePhabricator

Safari issue with Other amount option in banner: mask currency code letters
Closed, ResolvedPublic


We've replicated an error encountered by a Safari-using donor from Zendesk #357265: if you type currency code USD in addition to the amount on the Other field, you get a minimum amount error message:

PastedGraphic-3.png (216×532 px, 32 KB)

The banner is

Event Timeline

Fixed with this edit tightening up the existing blocking.

Re-opening, that change broke using backspace and arrow keys in Firefox so I've reverted it while I look for another fix.

Wow, for something so basic KeyboardEvents are a total mess of supposedly deprecated/unsupported properties. This change should work though, it explicitly excludes Backspace/Delete/cursor key events. Tested in Chrome, Firefox, IE, Safari.