[#181] remove calcControlsHeight as the component is always 70px in height now, and it caused issues on mobile UI. It's become obsolete

This commit is contained in:
Robin Scholtes 2023-04-20 21:05:44 +12:00
parent a547935f0a
commit 1fd72363bd
3 changed files with 0 additions and 16 deletions

View file

@ -668,7 +668,6 @@ class App {
this.manager.recipe.adjustWidth();
this.manager.input.calcMaxTabs();
this.manager.output.calcMaxTabs();
this.manager.controls.calcControlsHeight(); //@TODO
}

View file

@ -12,9 +12,6 @@
- test drag and drop etc. Regular mobile events / UX
### Desktop UI:
#### Bug
- `calcControlsHeight` and `adjustComponentSizes` cause trouble in `recipe` `rec-list` when resizing from desktop to mobile
### General UI:
- fix up key / tab events so UI can be navigated comfortably with keys ( inc. visual focus feedback ). Probably a lot of work though

View file

@ -418,18 +418,6 @@ ${navigator.userAgent}
bakeButton.classList.add("btn-success");
}
}
/**
* Calculates the height of the controls area and adjusts the recipe
* height accordingly.
*/
calcControlsHeight() {
const controls = document.getElementById("controls"),
recList = document.getElementById("rec-list");
recList.style.bottom = controls.clientHeight + "px";
}
}
export default ControlsWaiter;