Переглянути джерело

refactor(js): use `replace` instead of `remove+add`.

jvoisin 5 місяців тому
батько
коміт
4ace959667
1 змінених файлів з 2 додано та 8 видалено
  1. 2 8
      internal/ui/static/js/app.js

+ 2 - 8
internal/ui/static/js/app.js

@@ -648,10 +648,7 @@ function toggleEntryStatus(element, toasting) {
             showToastNotification(newStatus, currentStatus === "read" ? buttonElement.dataset.toastUnread : buttonElement.dataset.toastRead);
         }
 
-        if (element.classList.contains("item-status-" + currentStatus)) {
-            element.classList.remove("item-status-" + currentStatus);
-            element.classList.add("item-status-" + newStatus);
-        }
+        element.classList.replace("item-status-" + currentStatus, "item-status-" + newStatus);
 
         if (isListView() && getVisibleEntries().length === 0) {
             window.location.reload();
@@ -821,10 +818,7 @@ function openOriginalLinkFromListView() {
     goToListItem(1);
 
     // Mark as read if currently unread
-    if (currentItem.classList.contains("item-status-unread")) {
-        currentItem.classList.remove("item-status-unread");
-        currentItem.classList.add("item-status-read");
-
+    if (currentItem.classList.replace("item-status-unread", "item-status-read")) {
         const entryID = parseInt(currentItem.dataset.id, 10);
         updateEntriesStatus([entryID], "read");
     }