Closed Bug 1701230 Opened 3 years ago Closed 4 months ago

Focus the dialog element instead of the first focusable item as the default behaviour

Categories

(Core :: DOM: Core & HTML, task, P2)

task

Tracking

()

RESOLVED WONTFIX

People

(Reporter: sefeng, Assigned: sefeng)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxp])

Attachments

(1 file)

We believe having the default focus to be the dialog itself is an improvement to accessibility to provide consistent behaviours for our users.

There's a already a spec PR for it https://github.com/whatwg/html/pull/2197/files https://github.com/whatwg/html/pull/4184. Let's see if we can make it happen.

There is also https://github.com/whatwg/html/pull/4184 which would make this the default behaviour without changing the showModal API.

Oh yeah, Thanks! I linked to the wrong PR in the description. Let's try to make https://github.com/whatwg/html/pull/4184 happens, then we don't need https://github.com/whatwg/html/pull/2197/.

Instead of focusing the first focusable element, which may be any
element, having the focus to be the dialog itself is an improvement to
accessibility to provide consistent behavior for our uses.

See https://github.com/whatwg/html/pull/4184 for details.

Assignee: nobody → sefeng
Attachment #9211864 - Attachment description: WIP: Bug 1701230 - Focus the dialog element instead of the first focusable item as the default behavior r=smaug → Bug 1701230 - Focus the dialog element instead of the first focusable item as the default behavior r=smaug
Status: NEW → ASSIGNED
Severity: -- → N/A
Priority: -- → P2

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:sefeng, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(sefeng)
Flags: needinfo?(bugs)
Flags: needinfo?(bugs)

(I don't think this needs my attention)

I don't think I need to keep this needinfo..just need to resolve https://github.com/whatwg/html/pull/4184

Flags: needinfo?(sefeng)
Blocks: 1733536
No longer blocks: 1733536

:sefeng this is currently blocking https://mozilla-hub.atlassian.net/browse/FXP-839 from being closed, is there anything more to do? Should we remove it as a subtask?

Flags: needinfo?(sefeng)

Ah okay, so we need to get bug 1811129 implemented (I'll do that), then this one should be resolved.

Depends on: 1811129
Flags: needinfo?(sefeng)

Should be closed now probably?

Flags: needinfo?(sefeng)

Yeah!

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Flags: needinfo?(sefeng)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: