Fix button event handling bugs on mobile. Add button to clear directory filters.

This commit is contained in:
Sj-Si 2024-05-10 12:24:19 -04:00
parent c674dcfd64
commit 6bb5613ed4
3 changed files with 235 additions and 116 deletions

View File

@ -134,26 +134,37 @@
<div class="extra-network-content extra-network-content--dirs-view styled-scrollbar">
{dirs_html}
</div>
<div class="extra-network-content extra-network-content--cards resize-handle-row">
<div class="extra-network-content resize-handle-col" style="{tree_view_style}">
<div id='{tabname}_{extra_networks_tabname}_tree_list_loading_splash'
class='extra-network-tree extra-network-list-splash'>{tree_list_loading_splash_content}</div>
<div class="extra-network-content--tree-and-cards-view resize-handle-row">
<div class="extra-network-content extra-network-tree resize-handle-col" style="{tree_view_style}">
<div id='{tabname}_{extra_networks_tabname}_tree_list_loading_splash' class='extra-network-list-splash'>
{tree_list_loading_splash_content}</div>
<div id='{tabname}_{extra_networks_tabname}_tree_list_no_data_splash'
class='extra-network-tree extra-network-list-splash hidden'>{tree_list_no_data_splash_content}</div>
class='extra-network-list-splash hidden'>{tree_list_no_data_splash_content}</div>
<div id='{tabname}_{extra_networks_tabname}_tree_list_scroll_area'
class='extra-network-tree styled-scrollbar clusterize-scroll'>
class='styled-scrollbar clusterize-scroll'>
<div id='{tabname}_{extra_networks_tabname}_tree_list_content_area'
class='extra-network-tree-content clusterize-content'></div>
</div>
</div>
<div class="extra-network-content resize-handle-col" style="{card_view_style}">
<div id='{tabname}_{extra_networks_tabname}_card_list_loading_splash'
class='extra-network-cards extra-network-list-splash'>{card_list_loading_splash_content}</div>
<div class="extra-network-content extra-network-cards resize-handle-col" style="{card_view_style}">
<div class="extra-network-controls--buttons">
<button class="extra-network-control extra-network-control--clear-filters"
title="Clear all directory filters">
<svg class="extra-network-control--icon" viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg"
fill="none" stroke="currentColor" stroke-linecap="round" stroke-linejoin="round"
stroke-width="1">
<path d="M 2 2 V 4 L 6 8 V 13 L 9 14 V 12 M 12 5 L 13 4 V 2 H 2" />
<path d="M 9 10 L 12 7 M 9 7 L 12 10" />
</svg>
</button>
</div>
<div id='{tabname}_{extra_networks_tabname}_card_list_loading_splash' class='extra-network-list-splash'>
{card_list_loading_splash_content}</div>
<div id='{tabname}_{extra_networks_tabname}_card_list_no_data_splash'
class='extra-network-cards extra-network-list-splash hidden'>{card_list_no_data_splash_content}
class='extra-network-list-splash hidden'>{card_list_no_data_splash_content}
</div>
<div id='{tabname}_{extra_networks_tabname}_card_list_scroll_area'
class='extra-network-cards styled-scrollbar clusterize-scroll'>
class='styled-scrollbar clusterize-scroll'>
<div id='{tabname}_{extra_networks_tabname}_card_list_content_area'
class='extra-network-card-content clusterize-content'></div>
</div>

View File

@ -454,14 +454,19 @@ class ExtraNetworksTab {
addDirectoryFilter(div_id, filter_str, recurse) {
filter_str = isString(filter_str) ? filter_str : "";
recurse = recurse === true || recurse === false ? recurse : false;
if (this.card_list.addDirectoryFilter(div_id, filter_str, recurse)) {
this.card_list.addDirectoryFilter(div_id, filter_str, recurse);
this.directory_filters[div_id] = {filter_str: filter_str, recurse: recurse};
}
const clear_btn = this.container_elem.querySelector(".extra-network-control--clear-filters");
clear_btn.classList.toggle("hidden", !Object.keys(this.directory_filters).length);
}
removeDirectoryFilter(div_id) {
this.card_list.removeDirectoryFilter(div_id);
delete this.directory_filters[div_id];
const clear_btn = this.container_elem.querySelector(".extra-network-control--clear-filters");
clear_btn.classList.toggle("hidden", !Object.keys(this.directory_filters).length);
}
clearDirectoryFilters({excluded_div_ids} = {}) {
@ -480,6 +485,9 @@ class ExtraNetworksTab {
}
delete this.directory_filters[div_id];
}
const clear_btn = this.container_elem.querySelector(".extra-network-control--clear-filters");
clear_btn.classList.toggle("hidden", !Object.keys(this.directory_filters).length);
}
setDirectoryFilters() {
@ -853,10 +861,6 @@ class ExtraNetworksTab {
this.deselectDirsViewButton(dirs_btn);
}
}
if (!tree_btn_exists && !dirs_btn_exists) {
this.removeDirectoryFilter(div_id);
}
}
this.applyDirectoryFilters();
}
@ -1113,6 +1117,17 @@ function extraNetworksControlSearchClearOnClick(event) {
extra_networks_tabs[controls.dataset.tabnameFull].updateSearch("");
}
function extraNetworksControlClearFiltersOnClick(event) {
/** Clears all directory filters applied to the cards list. */
const btn = event.target.closest(".extra-network-control--clear-filters");
const pane = btn.closest(".extra-network-pane");
const tab = extra_networks_tabs[pane.dataset.tabnameFull];
tab.clearSelectedButtons();
tab.clearDirectoryFilters();
tab.applyDirectoryFilters();
}
function extraNetworksControlSortModeOnClick(event) {
/** Handles `onclick` events for Sort Mode buttons. */
const btn = event.target.closest(".extra-network-control--sort-mode");
@ -1390,7 +1405,11 @@ async function extraNetworksTreeDirectoryOnDblClick(event) {
const pane = btn.closest(".extra-network-pane");
const tab = extra_networks_tabs[pane.dataset.tabnameFull];
// Remove recursion on collapse
tab.setTreeListRecursionDepth(btn.dataset.divId, !("expanded" in btn.dataset));
await tab.tree_list.toggleRowExpanded(btn.dataset.divId);
// Add recursion on expand
tab.setTreeListRecursionDepth(btn.dataset.divId, ("expanded" in btn.dataset));
await tab.clearSelectedButtons();
}
@ -1401,8 +1420,11 @@ async function extraNetworksBtnTreeViewChevronOnClick(event) {
const pane = btn.closest(".extra-network-pane");
const tab = extra_networks_tabs[pane.dataset.tabnameFull];
// Remove recursion on collapse
tab.setTreeListRecursionDepth(btn.dataset.divId, !("expanded" in btn.dataset));
await tab.tree_list.toggleRowExpanded(btn.dataset.divId);
// Add recursion on expand
tab.setTreeListRecursionDepth(btn.dataset.divId, ("expanded" in btn.dataset));
tab.applyListButtonStates();
}
@ -1584,6 +1606,7 @@ function extraNetworksSetupEventDelegators() {
".edit-button": extraNetworksBtnEditMetadataOnClick,
".metadata-button": extraNetworksBtnShowMetadataOnClick,
".extra-network-control--search-clear": extraNetworksControlSearchClearOnClick,
".extra-network-control--clear-filters": extraNetworksControlClearFiltersOnClick,
".extra-network-control--sort-mode": extraNetworksControlSortModeOnClick,
".extra-network-control--sort-dir": extraNetworksControlSortDirOnClick,
".extra-network-control--dirs-view": extraNetworksControlDirsViewOnClick,
@ -1730,13 +1753,23 @@ function extraNetworksSetupEventDelegators() {
let press_timer;
let press_time_ms = 800;
let dbl_tap_timer;
const dbl_tap_time_ms = 500;
let curr_top;
let curr_left;
let curr_elem;
window.addEventListener("mousedown", event => {
["mousedown", "touchstart"].forEach(event_type => {
window.addEventListener(event_type, event => {
if (event_type.startsWith("mouse")) {
if (event.button !== 0) {
return;
}
} else if (event_type.startsWith("touch")) {
if (event.changedTouches.length !== 1) {
return;
}
}
let event_map = short_press_event_map;
if (event.ctrlKey && event.shiftKey) {
event_map = short_ctrl_shift_press_event_map;
@ -1749,10 +1782,15 @@ function extraNetworksSetupEventDelegators() {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
if (event_type.startsWith("mouse")) {
event.preventDefault();
}
event.stopPropagation();
elem.classList.add("pressed");
curr_elem = elem;
const curr_rect = elem.getBoundingClientRect();
curr_top = curr_rect.top;
curr_left = curr_rect.left;
}
}
@ -1769,11 +1807,17 @@ function extraNetworksSetupEventDelegators() {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
if (event_type.startsWith("mouse")) {
event.preventDefault();
}
event.stopPropagation();
elem.classList.add("pressed");
press_timer = setTimeout(() => {
elem.classList.remove("pressed");
const rect = elem.getBoundingClientRect();
if (curr_top !== rect.top || curr_left !== rect.left) {
return;
}
on_long_press(event, elem, obj.handler);
}, press_time_ms);
}
@ -1783,18 +1827,48 @@ function extraNetworksSetupEventDelegators() {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
if (event_type.startsWith("mouse")) {
event.preventDefault();
}
event.stopPropagation();
elem.classList.add("pressed");
if (event_type.startsWith("touch")) {
if (isNullOrUndefined(dbl_tap_timer)) {
dbl_tap_timer = setTimeout(() => {
dbl_tap_timer = null;
elem.classList.remove("pressed");
}, dbl_tap_time_ms);
} else {
clearTimeout(dbl_tap_timer);
dbl_tap_timer = null;
elem.classList.remove("pressed");
// Mimic the mouse events by adding detail=2 to the event.
elem.dispatchEvent(
new CustomEvent("dbltouchend", {bubbles: true, detail: 2}),
);
}
}
}
}
});
});
window.addEventListener("mouseup", event => {
["mouseup", "touchend", "dbltouchend"].forEach(event_type => {
window.addEventListener(event_type, event => {
// NOTE: long_press_event_map is handled by the timer setup in "mousedown" handlers.
if (event_type.startsWith("mouse")) {
if (event.button !== 0) {
clearTimeout(press_timer);
return;
}
} else if (event_type.startsWith("touch")) {
if (event.changedTouches.length !== 1) {
clearTimeout(press_timer);
return;
}
}
let event_map = short_press_event_map;
if (event.ctrlKey && event.shiftKey) {
event_map = short_ctrl_shift_press_event_map;
@ -1807,7 +1881,9 @@ function extraNetworksSetupEventDelegators() {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
if (event_type.startsWith("mouse")) {
event.preventDefault();
}
event.stopPropagation();
clearTimeout(press_timer);
if (isElement(curr_elem) && curr_elem !== elem) {
@ -1816,17 +1892,23 @@ function extraNetworksSetupEventDelegators() {
}
if (elem.classList.contains("pressed")) {
elem.classList.remove("pressed");
const rect = elem.getBoundingClientRect();
if (curr_top !== rect.top || curr_left !== rect.left) {
return;
}
on_short_press(event, elem, obj.handler);
}
}
}
if (event.detail % 2 === 0) {
if (event.detail !== 0 && event.detail % 2 === 0) {
for (const obj of dbl_press_event_map) {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
if (event_type.startsWith("mouse")) {
event.preventDefault();
}
event.stopPropagation();
clearTimeout(press_timer);
if (isElement(curr_elem) && curr_elem !== elem) {
@ -1844,8 +1926,29 @@ function extraNetworksSetupEventDelegators() {
curr_elem.classList.remove("pressed");
return;
}
// NOTE: long_press_event_map is handled by the timer setup in "mousedown" handlers.
});
});
// Disable the context menu for long press items. On mobile, long press causes
// context menu to appear which we don't want.
window.oncontextmenu = event => {
event_map = long_press_event_map;
if (event.ctrlKey && event.shiftKey) {
event_map = long_ctrl_shift_press_event_map;
} else if (event.ctrlKey) {
event_map = long_ctrl_press_event_map;
} else if (event.shiftKey) {
event_map = long_shift_press_event_map;
}
for (const obj of event_map) {
const elem = event.target.closest(obj.selector);
const neg = obj.negative ? event.target.closest(obj.negative) : null;
if (elem && !neg) {
return false;
}
}
};
}
async function extraNetworksSetupTab(tabname) {

View File

@ -1255,7 +1255,7 @@ body.resizing .resize-handle {
padding: var(--spacing-md);
}
.extra-network-content--cards {
.extra-network-content--tree-and-cards-view {
flex: 1;
flex-direction: row;
min-height: 0; /* prevent children from oversizing this container */
@ -1462,6 +1462,11 @@ body.resizing .resize-handle {
stroke: var(--input-placeholder-color);
}
.extra-network-control--clear-filters {
margin: 0 !important;
padding: 0 !important;
}
/* ==== ICONS ==== */
.extra-network-control--refresh:hover .extra-network-control--icon {
animation: extra-network-control--refresh-keyframe-rotate 3s infinite;