Intent to Implement: <dialog>: add DialogShowParams to show()/showModal()

27 views
Skip to first unread message

Dan Beam

unread,
Dec 29, 2016, 5:39:18 PM12/29/16
to blink-dev
db...@chromium.org https://github.com/whatwg/html/pull/2197 Adds parameters to change how show() or showModal() is called. Currently, this allows bypassing initial focus, which is set when a dialog is shown. Some folks just don't want initial focus to be set. See: https://github.com/whatwg/html/issues/1929 for more context. Firefox: Public support Edge: No public signals Safari: No public signals Web developers: Positive Basically none. This code (and its tests) need to be maintained. Yes https://bugs.chromium.org/p/chromium/issues/detail?id=677562 https://www.chromestatus.com/features/5121613552943104 Yes

Chris Harrelson

unread,
Jan 1, 2017, 7:39:02 PM1/1/17
to Dan Beam, blink-dev
Hi,

Since this is an intent to implement *and* ship, could you resend with that changed subject line?

While the change seems reasonable, I don't see much engagement outside of Chromium team members/Googlers. Has it seen sufficient exposure for people to comment?

Chris

On Thu, Dec 29, 2016 at 2:38 PM, Dan Beam <db...@chromium.org> wrote:
db...@chromium.org https://github.com/whatwg/html/pull/2197 Adds parameters to change how show() or showModal() is called. Currently, this allows bypassing initial focus, which is set when a dialog is shown. Some folks just don't want initial focus to be set. See: https://github.com/whatwg/html/issues/1929 for more context. Firefox: Public support Edge: No public signals Safari: No public signals Web developers: Positive Basically none. This code (and its tests) need to be maintained. Yes https://bugs.chromium.org/p/chromium/issues/detail?id=677562 https://www.chromestatus.com/features/5121613552943104 Yes

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.

Reply all
Reply to author
Forward
0 new messages