Skip to content

Commit

Permalink
Remove support for popup invoking attributes on text inputs
Browse files Browse the repository at this point in the history
Per the [1] resolution, we will not support this use case for
now.

[1] openui/open-ui#420 (comment)

Bug: 1307772
Change-Id: Iecb548d80a50135aa08e81aa2aacf5b9d9755f36
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3764298
Reviewed-by: Joey Arhar <jarhar@chromium.org>
Auto-Submit: Mason Freed <masonf@chromium.org>
Commit-Queue: Mason Freed <masonf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1026329}
  • Loading branch information
mfreed7 authored and chromium-wpt-export-bot committed Jul 20, 2022
1 parent 5f0779f commit 7d1e56e
Showing 1 changed file with 1 addition and 30 deletions.
31 changes: 1 addition & 30 deletions html/semantics/popups/popup-invoking-attribute.tentative.html
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,6 @@
}
return {expectedBehavior, expectedId};
}
const textLogic = (t,s,h) => {
// This mimics the expected logic for text field invokers, which can
// only be shown via the down arrow, and never hidden.
return {expectedBehavior: (t || s) ? "show" : "none", expectedId: t || s || 1};
};
const noActivationLogic = (t,s,h) => {
// This does not activate any pop-ups.
return {expectedBehavior: "none", expectedId: 1};
Expand Down Expand Up @@ -69,16 +64,7 @@
supported: true,
};
});
const supportedTextTypes = ['text','email','password','search','tel','url'].map(type => {
return {
name: `<input type="${type}"">`,
makeElement: makeElementWithType('input',type),
invokeFn: activateTextInputFn('down'),
getExpectedLogic: textLogic, // Down arrow should work
supported: true,
};
});
const unsupportedTypes = ['checkbox','radio','range','file','color','date','datetime-local','month','time','week','number'].map(type => {
const unsupportedTypes = ['text','email','password','search','tel','url','checkbox','radio','range','file','color','date','datetime-local','month','time','week','number'].map(type => {
return {
name: `<input type="${type}"">`,
makeElement: makeElementWithType('input',type),
Expand All @@ -90,22 +76,7 @@
const invokers = [
...supportedButtonTypes,
...supportedInputButtonTypes,
...supportedTextTypes,
...unsupportedTypes,
{
name: '<input type=text> with right arrow invocation',
makeElement: makeElementWithType('input','text'),
invokeFn: activateTextInputFn('right'),
getExpectedLogic: noActivationLogic, // Right arrow should not work
supported: false,
},
{
name: '<input type=text> focus only',
makeElement: makeElementWithType('input','text'),
invokeFn: el => el.focus(),
getExpectedLogic: noActivationLogic, // Just focusing the control should not work
supported: false,
},
];
["auto","hint","manual"].forEach(type => {
invokers.forEach(testcase => {
Expand Down

0 comments on commit 7d1e56e

Please sign in to comment.