Fix ariaLabel for Firefox ESR (#83)

Author: smashedr

.github/workflows/build.yaml

@@ -21,8 +21,8 @@ jobs:
           node-version: 20
 
       - name: "Update Manifest Version"
-        uses: cssnr/update-json-value-action@v1
         if: ${{ github.event_name == 'release' }}
+        uses: cssnr/update-json-value-action@v1
 
       - name: "Build All"
         run: |
@@ -36,6 +36,7 @@ jobs:
           path: web-ext-artifacts/
 
       - name: "Upload to Release"
+        if: ${{ github.event_name == 'release' }}
         uses: svenstaro/upload-release-action@v2
         with:
           repo_token: ${{ secrets.GITHUB_TOKEN }}

src/js/extract.js

@@ -77,12 +77,13 @@ function extractSelection() {
  * @param {HTMLAnchorElement} element
  */
 function pushElement(array, element) {
+    // console.debug('element:', element)
     try {
         const data = {
             href: decodeURI(element.href),
-            text: element.textContent,
+            text: element.textContent?.trim(),
             title: element.title,
-            label: element.ariaLabel,
+            label: element.ariaLabel || '',
             rel: element.rel,
             target: element.target,
             origin: element.origin,

src/js/popup.js

@@ -218,11 +218,11 @@ function extractURLs(text) {
         const match = urlmatcharr[0]
         const url = new URL(match)
         const data = {
-            text: null,
-            title: null,
-            label: null,
-            target: null,
-            rel: null,
+            text: '',
+            title: '',
+            label: '',
+            target: '',
+            rel: '',
             href: url.href,
             origin: url.origin,
         }