1871745 - Inputs styled with "-moz-user-focus: none" is no longer focusable

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[SUN Haitao]

Attachment: demo.html

Gecko 122 no longer allows inputs styled with "-moz-user-focus: none" to be focused. Since some websites (such as https://www.decathlon.com.cn/) still set this for unknown reasons, this change will create some compatible problems.

Such sites are functional on Gecko 120. The attached file named "demo.html" can be used to demonstrate the behavior difference.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1868552

:emilio, since you are the author of the regressor, bug 1868552, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 2

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

Comment 3

[Emilio Cobos Álvarez (:emilio)]

So... what's the right way of checking the new use counter data? https://mozilla.github.io/usecounters/ doesn't show newer use-counter data. (I'd ask :chutten but requests blocked :)).

In any case, depending on the use counter data, we can take a few approaches:

• If it's really low, CSS compat intervention would do.
• Otherwise, hiding this property from content is probably better.

For now I requested sheriffs to back out the regressing patch from beta, for now.

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1871745 - Hide -moz-user-focus from content. r=smaug

-moz-user-focus: none didn't do anything useful for non-XUL until
bug 1868552. It seems nonetheless some sites specify it, which can cause
compat issues.

Let's hide this property from content, to avoid breaking those sites.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/65525b63d6cf
Hide -moz-user-focus from content. r=smaug

Comment 6

[Norisz Fay [:noriszfay]]

Backed out for causing bc failures on browser_bug462289.js, browser_PanelMultiView_focus.js

Backout link

Push with failures

Failure log 1 // Failure log 2

Comment 7

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1871745 - Keep exposing chrome-only properties on CSS2Properties. r=edgar,smaug,peterv

So that chrome code can keep setting -moz-user-focus via script, for
example.

Comment 8

[Emilio Cobos Álvarez (:emilio)]

Attachment: Requested diff.

Comment 9

[Donal Meehan [:dmeehan]]

Setting 122 to fixed, the regressor was backed out of beta

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/619b6b3dfcfe
Hide -moz-user-focus from content. r=smaug
https://hg.mozilla.org/integration/autoland/rev/659e4e834582
Keep exposing chrome-only properties on CSS2Properties. r=peterv

Comment 11

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/619b6b3dfcfe
https://hg.mozilla.org/mozilla-central/rev/659e4e834582

Comment 12

[Alessio Placitelli [:Dexter]]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)

So... what's the right way of checking the new use counter data? https://mozilla.github.io/usecounters/ doesn't show newer use-counter data. (I'd ask :chutten but requests blocked :)).

To my knowledge the new data is still pending validation, so I'd hesitate to recommend using it until we get greenlight from Chris (who is now back). See bug 1866834 for the most recent updates on the topic.

Comment 13

[Hamish Willee]

MDN docs for this can be tracked in https://github.com/mdn/content/issues/31928.

My understanding is that from the website developer perspective, the change will appear to be that the -moz-user-focus (and associated property values) has been removed from Firefox in FF123. Is that correct, or is the existence of this property still useful to anyone outside of firefox development?

If so, then would the right change be to record this as removed in BCD and mark https://developer.mozilla.org/en-US/docs/Web/CSS/-moz-user-focus as deprecated?

Comment 14

[Emilio Cobos Álvarez (:emilio)]

That sounds about right.