Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the ability to not perform dialog focusing steps #2197

Closed
wants to merge 3 commits into from
+38 −19
Closed

Add the ability to not perform dialog focusing steps #2197

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0f019fe
Add the ability to not perform dialog focusing steps
domenic Dec 20, 2016
cb3eaf6
Rename to skipInitialFocus
domenic Dec 21, 2016
16d19cd
Fix naming typos
domenic Dec 22, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 19 additions & 17 deletions source
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56657,7 +56657,7 @@ interface <dfn>HTMLDialogElement</dfn> : <span>HTMLElement</span> {
};

dictionary <dfn>DialogShowOptions</dfn> {
boolean skipFocusing = false;
boolean skipInitialFocus = false;
};</pre>
</dd>
</dl><!--TOPIC:HTML-->
Expand Down Expand Up @@ -56708,29 +56708,31 @@ dictionary <dfn>DialogShowOptions</dfn> {

<dl class="domintro">

<dt><var>dialog</var> . <code subdfn data-x="dom-dialog-show">show</code>( [ { skipFocusing: true } ] )</dt>
<dt><var>dialog</var> . <code subdfn data-x="dom-dialog-show">show</code>( [ { skipInitialFocus: true } ] )</dt>

<dd>

<p>Displays the <code>dialog</code> element.</p>

<p>If the <code data-x="">skipFocusing</code> option is provided and set to true, then no <span
data-x="dialog focusing steps">additional automatic focusing logic</span> will be performed.
Otherwise, the user agent will attempt to focus the contents of the <code>dialog</code>, taking
into account the <code data-x="attr-fe-autofocus">autofocus</code> attribute if relevant.</p>
<p>If the <code data-x="">skipInitialFocus</code> option is provided and set to true, then no
<span data-x="dialog focusing steps">additional automatic focusing logic</span> will be
performed. Otherwise, the user agent will attempt to focus the contents of the
<code>dialog</code>, taking into account the <code data-x="attr-fe-autofocus">autofocus</code>
attribute if relevant.</p>

</dd>

<dt><var>dialog</var> . <code subdfn data-x="dom-dialog-showModal">showModal</code>( [ { skipFocusing: true } ] )</dt>
<dt><var>dialog</var> . <code subdfn data-x="dom-dialog-showModal">showModal</code>( [ { skipInitialFocus: true } ] )</dt>

<dd>

<p>Displays the <code>dialog</code> element and makes it the top-most modal dialog.</p>

<p>If the <code data-x="">skipFocusing</code> option is provided and set to true, then no <span
data-x="dialog focusing steps">additional automatic focusing logic</span> will be performed.
Otherwise, the user agent will attempt to focus the contents of the <code>dialog</code>, taking
into account the <code data-x="attr-fe-autofocus">autofocus</code> attribute if relevant.</p>
<p>If the <code data-x="">skipInitialFocus</code> option is provided and set to true, then no
<span data-x="dialog focusing steps">additional automatic focusing logic</span> will be
performed. Otherwise, the user agent will attempt to focus the contents of the
<code>dialog</code>, taking into account the <code data-x="attr-fe-autofocus">autofocus</code>
attribute if relevant.</p>

</dd>

Expand Down Expand Up @@ -56773,9 +56775,9 @@ dictionary <dfn>DialogShowOptions</dfn> {

<li><p>Set this <code>dialog</code> to the <span>normal alignment</span> mode.</p></li>

<li><p>If the <code data-x="">skipFocusing</code> member of <var>options</var> is not present, or
if it is present but its value is false, run the <span>dialog focusing steps</span> for this
<code>dialog</code> element.</p></li>
<li><p>If the <code data-x="">skipInitialFocusing</code> member of <var>options</var> is not
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be skipInitialFocus right ?

present, or if it is present but its value is false, run the <span>dialog focusing steps</span>
for this <code>dialog</code> element.</p></li>

</ol>

Expand Down Expand Up @@ -56816,9 +56818,9 @@ dictionary <dfn>DialogShowOptions</dfn> {
<li><p>Push <var>subject</var> onto <var>subject</var>'s
<span>node document</span>'s <span>pending dialog stack</span>.</p></li>

<li><p>If the <code data-x="">skipFocusing</code> member of <var>options</var> is not present, or
if it is present but its value is false, run the <span>dialog focusing steps</span> for
<var>subject</var>.</p></li>
<li><p>If the <code data-x="">skipInitialFocusing</code> member of <var>options</var> is not
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

present, or if it is present but its value is false, run the <span>dialog focusing steps</span>
for <var>subject</var>.</p></li>

</ol>

Expand Down