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.

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

Convert popup=popup to popup=auto or just popup #34217

Merged
merged 1 commit into from
Jun 1, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
</style>

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<option>bottom left</option>
<option>two</option>
<option>three</option>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
<script src="/resources/testdriver-vendor.js"></script>

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<selectmenu id="nested0">
<option id="child1">one</option>
<option id="child2">two</option>
Expand All @@ -18,7 +18,7 @@
</selectmenu>

<selectmenu id="selectMenu1">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<select>
<option>one</option>
<option>two</option>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<link rel="stylesheet" href="/fonts/ahem.css">

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option>
option with image displayed
<img src="/images/green-256x256.png">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
</style>

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option>
option with image displayed
<img src="/images/green-256x256.png">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<script src="/resources/testdriver-vendor.js"></script>

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option id="selectMenu0-child1">one</option>
<option id="selectMenu0-child2">two</option>
<div behavior="option" id="selectMenu0-child3">three</div>
Expand All @@ -18,7 +18,7 @@
</selectmenu>

<selectmenu id="selectMenu1">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<div behavior="button" id="selectMenu1-button">
Custom button
</div>
Expand All @@ -30,7 +30,7 @@
<selectmenu id="selectMenu2">
<div slot="button" behavior="button" id="selectMenu2-button">
Custom button
<div popup=popup behavior="listbox" id="selectMenu2-popup">
<div popup behavior="listbox" id="selectMenu2-popup">
<option>one</option>
<option id="selectMenu2-child2">two</option>
</div>
Expand All @@ -53,7 +53,7 @@
<selectmenu id="selectMenu4">
<div slot="button" behavior="button" id="selectMenu4-button0">button0</div>
<div slot="listbox" id="selectMenu4-listbox-slot">
<div popup=popup behavior="listbox" id="selectMenu4-listbox0">
<div popup behavior="listbox" id="selectMenu4-listbox0">
<option>one</option>
<option id="selectMenu4-option2">two</option>
</div>
Expand All @@ -72,7 +72,7 @@
<!-- No associated JS test -- just don't crash when parsing! -->
<selectmenu id="selectMenu6">
<div slot="button"></div>
<div popup=popup slot="listbox" behavior="listbox"></div>
<div popup slot="listbox" behavior="listbox"></div>
</selectmenu>

<!-- No associated JS test -- just don't crash when parsing! -->
Expand All @@ -89,7 +89,7 @@

<selectmenu id="selectMenu9">
<div slot="listbox" id="selectMenu9-listbox-slot">
<div popup=popup behavior="listbox" id="selectMenu9-originalListbox">
<div popup behavior="listbox" id="selectMenu9-originalListbox">
<option>one</option>
<option id="selectMenu9-option2">two</option>
</div>
Expand All @@ -103,7 +103,7 @@
</selectmenu>

<selectmenu id="selectMenu11">
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option>one</option>
</div>
<div slot="button" behavior="listbox" id="selectMenu11-button">Test</div>
Expand All @@ -114,7 +114,7 @@
<div behavior="button" id="selectMenu12-button0">button0</div>
</div>
<div slot="listbox" id="selectMenu12-listbox-slot">
<div popup=popup behavior="listbox" id="selectMenu12-originalListbox">
<div popup behavior="listbox" id="selectMenu12-originalListbox">
<option id="selectMenu12-option1">one</option>
<option>two</option>
</div>
Expand All @@ -131,12 +131,12 @@
</div>
<div slot="listbox" id="selectMenu13-listbox-slot">
<div id="selectMenu13-removeContent-listbox">
<div popup=popup behavior="listbox" id="selectMenu13-originalListbox">
<div popup behavior="listbox" id="selectMenu13-originalListbox">
<option id="selectMenu13-option1">one</option>
<option id="selectMenu13-option2">two</option>
</div>
</div>
<div popup=popup behavior="listbox" id="selectMenu13-newListbox">
<div popup behavior="listbox" id="selectMenu13-newListbox">
<option>three</option>
<option id="selectMenu13-option4">four</option>
</div>
Expand Down Expand Up @@ -313,7 +313,7 @@
assert_equals(selectMenu.value, "one", "Initial value should be the first option");

let newListbox = document.createElement("div");
newListbox.setAttribute("popup", "popup");
newListbox.setAttribute("popup", "auto");
newListbox.setAttribute("behavior", "listbox");
let newOption = document.createElement("option");
newOption.innerText = "three";
Expand Down Expand Up @@ -402,7 +402,7 @@
assert_false(selectMenu.open);

let newListbox = document.createElement("div");
newListbox.setAttribute("popup", "popup");
newListbox.setAttribute("popup", "auto");
newListbox.setAttribute("behavior", "listbox");
let newOption = document.createElement("option");
newOption.innerText = "three";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@

<selectmenu id="selectMenu0">
<div slot="button" behavior="button" id="selectMenu0-button">Custom bottom left</div>
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<option>bottom left</option>
<option>two</option>
<option>three</option>
Expand All @@ -58,7 +58,7 @@

<selectmenu id="selectMenu1">
<div slot="button" behavior="button" id="selectMenu1-button">Custom top left</div>
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<option>top left</option>
<option>two</option>
<option>three</option>
Expand All @@ -67,7 +67,7 @@

<selectmenu id="selectMenu2">
<div slot="button" behavior="button" id="selectMenu2-button">Custom bottom right</div>
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu2-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu2-popup">
<option>bottom right</option>
<option>two</option>
<option>three</option>
Expand All @@ -76,7 +76,7 @@

<selectmenu id="selectMenu3">
<div slot="button" behavior="button" id="selectMenu3-button">Custom top right</div>
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu3-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu3-popup">
<option>top right</option>
<option>two</option>
<option>three</option>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
</style>

<selectmenu id="selectMenu0">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu0-popup">
<option>bottom left</option>
<option>two</option>
<option>three</option>
Expand All @@ -44,23 +44,23 @@
<br>

<selectmenu id="selectMenu1">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu1-popup">
<option>top left</option>
<option>two</option>
<option>three</option>
</div>
</selectmenu>

<selectmenu id="selectMenu2">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu2-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu2-popup">
<option>bottom right</option>
<option>two</option>
<option>three</option>
</div>
</selectmenu>

<selectmenu id="selectMenu3">
<div popup=popup slot="listbox" behavior="listbox" id="selectMenu3-popup">
<div popup slot="listbox" behavior="listbox" id="selectMenu3-popup">
<option>top right</option>
<option>two</option>
<option>three</option>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
<div slot="button" behavior="button" class="button">
Custom button
</div>
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option>one</option>
<option class="child2">two</option>
<option class="child3">three</option>
Expand All @@ -32,7 +32,7 @@

<selectmenu id="selectMenu3">
<div slot="listbox">
<div popup=popup behavior="listbox" id="selectMenu3-listbox">
<div popup behavior="listbox" id="selectMenu3-listbox">
<option>one</option>
</div>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<selectmenu id="selectmenu1">
<template shadowroot=open>
<button behavior="button">Custom button</button>
<div popup=popup behavior="listbox">
<div popup behavior="listbox">
<slot></slot>
</div>
</template>
Expand Down Expand Up @@ -58,7 +58,7 @@
</style>
<button behavior="button">Open select!</button>
<div id="value" behavior="selected-value"></div>
<div popup=popup behavior="listbox">
<div popup behavior="listbox">
<input type="text" placeholder="Filter options">
<option>Thing 1</option>
<option>Thing 2</option>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
<div slot="button" behavior="button">
<div behavior="selected-value" id="selectMenu5-custom-selected-value">Default custom selected-value text</div>
</div>
<div popup=popup slot="listbox" behavior="listbox">
<div popup slot="listbox" behavior="listbox">
<option id="selectMenu5-option1">one</option>
<option id="selectMenu5-option2">two</option>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<script src="/resources/testharnessreport.js"></script>

<button id=b1>This is an anchor button</button>
<div popup=popup id=p1 anchor=b1>This is a popup</div>
<div popup id=p1 anchor=b1>This is a popup</div>
<button id=b2 popup=p1>This button invokes the popup but isn't an anchor</button>

<script>
Expand Down
4 changes: 2 additions & 2 deletions html/semantics/popups/popup-anchor-nesting.tentative.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
which contains a separate popup (p2) which is anchored
by a separate anchor (b2). -->
<button id=b1 onclick='p1.showPopup()'>Popup 1
<div popup=popup id=p2 anchor=b2>
<div popup id=p2 anchor=b2>
<span id=inside2>Inside popup 2</span>
</div>
</button>
<div popup=popup id=p1 anchor=b1>This is popup 1</div>
<div popup id=p1 anchor=b1>This is popup 1</div>
<button id=b2 onclick='p2.showPopup()'>Popup 2</button>

<style>
Expand Down
Loading