mirror of
https://github.com/miniflux/v2.git
synced 2025-06-27 16:36:00 +00:00
Golf a bit internal/ui/static/js/app.js
- Use `….classList.toggle` instead of `….classList.add`/`….classList.remove` in a condition - Replace a `function()` with a `() =>` - Use `Math.min` instead of a handwritten condition
This commit is contained in:
parent
79ec6ef81f
commit
f67d2e230b
1 changed files with 3 additions and 4 deletions
|
@ -71,12 +71,11 @@ function toggleMainMenu(event) {
|
|||
const menu = document.querySelector(".header nav ul");
|
||||
const menuToggleButton = document.querySelector(".logo");
|
||||
if (menu.classList.contains("js-menu-show")) {
|
||||
menu.classList.remove("js-menu-show");
|
||||
menuToggleButton.setAttribute("aria-expanded", false);
|
||||
} else {
|
||||
menu.classList.add("js-menu-show");
|
||||
menuToggleButton.setAttribute("aria-expanded", true);
|
||||
}
|
||||
menu.classList.toggle("js-menu-show");
|
||||
}
|
||||
|
||||
// Handle click events for the main menu (<li> and <a>).
|
||||
|
@ -662,7 +661,7 @@ function showToast(label, iconElement) {
|
|||
const toastElementWrapper = document.getElementById("toast-wrapper");
|
||||
if (toastElementWrapper) {
|
||||
toastElementWrapper.classList.remove('toast-animate');
|
||||
setTimeout(function () {
|
||||
setTimeout(() => {
|
||||
toastElementWrapper.classList.add('toast-animate');
|
||||
}, 100);
|
||||
}
|
||||
|
@ -782,7 +781,7 @@ function handleMediaControl(button) {
|
|||
enclosures.forEach((enclosure) => {
|
||||
switch (action) {
|
||||
case "seek":
|
||||
enclosure.currentTime = enclosure.currentTime + value > 0 ? enclosure.currentTime + value : 0;
|
||||
enclosure.currentTime = Math.min(enclosure.currentTime + value, 0);
|
||||
break;
|
||||
case "speed":
|
||||
// I set a floor speed of 0.25 to avoid too slow speed where it gives the impression it stopped.
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue