1661776 - Print dialog's validated fields should prevent pasting invalid input

Status: VERIFIED FIXED

(Toolkit :: Printing, defect, P2)

Description

[Micah [:mtigley] (she/her)]

Bug 1660060 introduces a keypress listener for the scale input UI component, which blocks invalid keys (non-numeric characters, "%", etc...) from being entered. As a result copy/paste/ctrl+a commands don't work. This is a problem since chrome supports this on Windows.

This issue should add a "paste" event handler that prevents invalid input from being entered.

Comment 1

[Micah [:mtigley] (she/her)]

Attachment: Bug 1661776 - Prevent pasting invalid input into ScaleInput UI. r?emalysz

Comment 2

[Emma Malysz]

Marking as "leave-open" because a second patch is required to handle pasting for start and end page range

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b7b3e0312fe4
Prevent pasting invalid input into ScaleInput UI. r=emalysz

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/b7b3e0312fe4

Comment 5

[Micah [:mtigley] (she/her)]

Attachment: Bug 1661776 - Add a paste event handler to PrintUIControlMixin that prevents invalid inputs. r?emalysz

Comment 6

[Micah [:mtigley] (she/her)]

Attachment: Bug 1661776 - Prevent pasting invalid input into PageRangeInput element. r?emalysz

Depends on D89009

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/94205e4e0130
Add a paste event handler to PrintUIControlMixin that prevents invalid inputs. r=emalysz

Comment 8

[Micah [:mtigley] (she/her)]

Comment on attachment 9172790 [details]
Bug 1661776 - Prevent pasting invalid input into ScaleInput UI. r?emalysz

Beta/Release Uplift Approval Request

• User impact if declined: Users can paste invalid input into the percent scale, copies, and page range fields. This experience is inconvenient if invalid input was accidentally entered and can result in a confusing experience for the user trying to change the percent scale, copies, or page range values.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: STR:

1. Copy "Invalid input" (or some non-numeric value) into your clipboard.
2. Try pasting into either the percent scale, copies, or page range input fields.

Expected

Pasting invalid input should not be allowed.

Actual

The invalid input is pasted into the field.

• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Small change to how numeric input fields handle invalid paste input. Only the percent scale, copies, and page range fields are affected.
• String changes made/needed:

Comment 9

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/94205e4e0130

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9172790 [details]
Bug 1661776 - Prevent pasting invalid input into ScaleInput UI. r?emalysz

Approved for 81.0b6.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/f0d2254abb8e
https://hg.mozilla.org/releases/mozilla-beta/rev/2bcd5f644fb5

Comment 12

[Emil Ghitta, QA [:emilghitta]]

This issue is verified fixed using Firefox 81.0b7 (BuildId:20200906164749) and Firefox 82.0a1 (BuildId:20200907094115) on Windows 10 64bit, macOS 10.14 and Ubuntu 18.04 64bit.