From afc21e6aa769897b790e3c1705f1907ab2925e86 Mon Sep 17 00:00:00 2001 From: charles <30816317+charles7668@users.noreply.github.com> Date: Thu, 29 Feb 2024 04:23:49 +0800 Subject: [PATCH] Fix counter display number incorrectly displayed on the page (#29448) issue : #28239 The counter number script uses the 'checkbox' attribute to determine whether an item is selected or not. However, the input event only increments the counter value, and when more items are displayed, it does not update all previously loaded items. As a result, the display becomes incorrect because it triggers the update counter script, but checkboxes that are selected without the 'checked' attribute are not counted (cherry picked from commit 252047ed2e09e3f1f1ab394cd62995cf4cabe506) --- web_src/js/features/pull-view-file.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/web_src/js/features/pull-view-file.js b/web_src/js/features/pull-view-file.js index 90881ee989..2472e5a0bd 100644 --- a/web_src/js/features/pull-view-file.js +++ b/web_src/js/features/pull-view-file.js @@ -43,9 +43,11 @@ export function initViewedCheckboxListenerFor() { // Mark the file as viewed visually - will especially change the background if (this.checked) { form.classList.add(viewedStyleClass); + checkbox.setAttribute('checked', ''); prReview.numberOfViewedFiles++; } else { form.classList.remove(viewedStyleClass); + checkbox.removeAttribute('checked'); prReview.numberOfViewedFiles--; }