Merge pull request #805 from paulwarren-wk/footnote-fixes

Arelle · Jan 27, 2025 · f135d5b · f135d5b
2 parents ab399df + 66b5daa
commit f135d5b
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 15 deletions.
diff --git a/iXBRLViewerPlugin/viewer/src/html/footnote-details.html b/iXBRLViewerPlugin/viewer/src/html/footnote-details.html
@@ -3,7 +3,7 @@
 -->
 
 <div>
-  <table class="fact-properties">
+  <table class="property-table fact-properties">
     <tr class="value">
       <th data-i18n="footnotes.footnote">Footnote</th>
       <td><span class="value"></span> <button class="show-all inline-button">[...]</button></td>

diff --git a/iXBRLViewerPlugin/viewer/src/js/footnote.js b/iXBRLViewerPlugin/viewer/src/js/footnote.js
@@ -21,6 +21,12 @@ export class Footnote {
         return this.textContent();
     }
 
+    readableValueHTML() {
+        const span = document.createElement("span");
+        span.append(document.createTextNode(this.textContent()));
+        return span;
+    }
+
     isTextBlock() {
         return false;
     }

diff --git a/iXBRLViewerPlugin/viewer/src/js/inspector.js b/iXBRLViewerPlugin/viewer/src/js/inspector.js
@@ -1208,7 +1208,7 @@ export class Inspector {
         const valueHTML = item.readableValueHTML();
         const tr = $('tr.value', context);
         const valueSpan = tr.find('td .value').empty();
-        if (!item.isNumeric() && !showAll) {
+        if (!(item instanceof Fact && item.isNumeric()) && !showAll) {
             const vv = wrapLabel(valueHTML.textContent, 120);
             if (vv.length > 1) {
                 tr.addClass("truncated");
@@ -1463,6 +1463,7 @@ export class Inspector {
         } 
         else { 
             $('#inspector').removeClass('no-fact-selected').removeClass("hidden-fact").removeClass("html-hidden-fact");
+            $('#inspector .tags').show();
 
             $('#inspector .fact-inspector')
                 .empty()
@@ -1512,6 +1513,7 @@ export class Inspector {
             else if (cf instanceof Footnote) {
                 $('#inspector').addClass('footnote-mode');
                 $('#inspector .footnote-details .footnote-facts').empty().append(this._footnoteFactsHTML(cf));
+                $('#inspector .tags').hide();
             }
             $('.fact-details').localize();
         }

diff --git a/iXBRLViewerPlugin/viewer/src/less/inspector.less b/iXBRLViewerPlugin/viewer/src/less/inspector.less
@@ -689,19 +689,6 @@
             }
           }
 
-          .select-icon {
-            float: right;
-            .square-button();
-            .clickable();
-
-            &:first-of-type {
-              margin-left: 0.5rem;
-            }
-
-            &::before {
-              .icon-select();
-            }
-          }
 
           .more-results {
             .clickable();
@@ -1297,6 +1284,20 @@
     .fact-list-item {
       .block-list-item();
 
+      .select-icon {
+        float: right;
+        .square-button();
+        .clickable();
+
+        &:first-of-type {
+          margin-left: 0.5rem;
+        }
+
+        &::before {
+          .icon-select();
+        }
+      }
+
       .title {
         color: var(--colour-text-title);
         font-weight: bold;