disable tooltips on mobile, update todos

This commit is contained in:
Robin Scholtes 2023-08-15 20:44:11 +12:00
parent ab8b48c437
commit 6a375c62a8
3 changed files with 14 additions and 20 deletions

View file

@ -836,6 +836,7 @@ class App {
setMobileUI() { setMobileUI() {
this.setSplitter(false); this.setSplitter(false);
this.assignAvailableHeight(); this.assignAvailableHeight();
$("[data-toggle=tooltip]").tooltip("disable");
} }
/** /**

View file

@ -1,8 +1 @@
operations: - UI tests
- UI tests etc.
manual testing issues:
Recipe;
- some reordering in recipe is not super smooth ( when the recipe-li is very large for instance )
- double click deleting c-recipe-li not reliable

View file

@ -48,6 +48,18 @@ export class CRecipeLi extends HTMLElement {
this.removeEventListener("dblclick", this.handleDoubleClick.bind(this)); this.removeEventListener("dblclick", this.handleDoubleClick.bind(this));
} }
/**
* Handle double click
*
* @param {Event} e
*/
handleDoubleClick(e) {
// do not remove if icons or form elements are double-clicked
if (e.target === this.querySelector("li") || e.target === this.querySelector("div.op-title")) {
this.removeOperation();
}
}
/** /**
* Handle mousedown * Handle mousedown
* @fires Manager#statechange * @fires Manager#statechange
@ -87,18 +99,6 @@ export class CRecipeLi extends HTMLElement {
} }
} }
/**
* Handle double click
*
* @param {Event} e
*/
handleDoubleClick(e) {
// do not remove if icons or form elements are double-clicked
if (e.target === this.querySelector("li") || e.target === this.querySelector("div.op-title")) {
this.removeOperation();
}
}
/** /**
* Remove this operation from the recipe list * Remove this operation from the recipe list
* *