1511387 - UAWidget variant of datetime needs similar click handling as XBL version

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: uawidget_datetime_click.diff

The patch in bug 1089326 has the following change for the XBL


-          <svg xmlns="http://www.w3.org/2000/svg" width="12" height="12">
+          <svg xmlns="http://www.w3.org/2000/svg" width="12" height="12" style="pointer-events: none;">
             <path d="M 3.9,3 3,3.9 5.1,6 3,8.1 3.9,9 6,6.9 8.1,9 9,8.1 6.9,6 9,3.9 8.1,3 6,5.1 Z M 12,6 A 6,6 0 0 1 6,12 6,6 0 0 1 0,6 6,6 0 0 1 6,0 6,6 0 0 1 12,6 Z"/>

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 9028954 [details] [diff] [review]
uawidget_datetime_click.diff

You would need to do this in datetimebox.css because inline style will be blocked by CSP.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Whaat, CSP affecting to internal elements. That is a bug. But ok.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: uawidget_datetime_click_2.diff

Something like this then

Comment 5

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 9029035 [details] [diff] [review]
uawidget_datetime_click_2.diff

I wonder if bug 1505887 actually have made CSP not applicable to UA Widget DOM already, but this should work regardless.

Comment 6

[Guilherme Lima]

The patch landed with number of the duplicated bug.
https://hg.mozilla.org/mozilla-central/rev/eb4344bead3e