Fix: Prevent duplicate percentage values when switching between mobile and desktop in Tech Report (#1140)

* Initial plan

* Fix: Remove existing monthchange element before appending new one in updateAppSummary

Co-authored-by: max-ostapenko <1611259+max-ostapenko@users.noreply.github.com>

* Better implementation

* Pin super linter

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: max-ostapenko <1611259+max-ostapenko@users.noreply.github.com>
Co-authored-by: Barry Pollard <barrypollard@google.com>

Author: 3 people

.github/workflows/test-website.yml

@@ -35,7 +35,7 @@ jobs:
     - name: Use more complete checks for generated HTML linting
       run: cp -f .github/linters/.htmlhintrc_morechecks .github/linters/.htmlhintrc
     - name: Lint Generated HTML
-      uses: super-linter/super-linter/slim@v8
+      uses: super-linter/super-linter/slim@v8.1.0
       env:
         DEFAULT_BRANCH: main
         FILTER_REGEX_INCLUDE: static/html/.*

src/js/techreport/timeseries.js

@@ -249,10 +249,16 @@ class Timeseries {
           const latestMoM = latestClient?.momPerc;
           const latestMoMStr = latestClient?.momString;
           const styling = UIUtils.getChangeStatus(latestMoM, changeMeaning);
-          const monthChange = document.createElement('span');
+
+          /* Add month change element if not already present */
+          let monthChange = card.querySelector('.monthchange');
+          if(!monthChange) {
+            monthChange = document.createElement('span');
+            card.appendChild(monthChange);
+          }
+
           monthChange.textContent = latestMoMStr;
           monthChange.className = `monthchange ${styling?.color} ${styling?.direction}`;
-          card.appendChild(monthChange);
         }
       }
     });