chore(comment): fix copy-tab-data class comment

micmro · Oct 7, 2019 · 7671aef · 7671aef
1 parent 1823a56
commit 7671aef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/ts/transformers/har-tabs.ts b/src/ts/transformers/har-tabs.ts
@@ -132,6 +132,7 @@ function makeContentTab(entry: Entry) {
   const lineCount = newLines ? newLines.length : 1;
   return makeLazyWaterfallEntryTab(
     `Content (${lineCount} Line${lineCount > 1 ? "s" : ""})`,
+    // class `copy-tab-data` needed to catch bubbled up click event in `details-overlay/html-details-body.ts`
     () => `
     <button class="copy-tab-data">Copy Content to Clipboard</button>
     <pre><code>${escapeHtml(unescapedText)}</code></pre>
@@ -144,7 +145,7 @@ function makeRawData(entry: Entry) {
   return makeLazyWaterfallEntryTab(
     "Raw Data",
     () => {
-      // class `copy` needed to catch bubbled up click event in `details-overlay/html-details-body.ts`
+      // class `copy-tab-data` needed to catch bubbled up click event in `details-overlay/html-details-body.ts`
       return `
       <button class="copy-tab-data">Copy Raw Data to Clipboard</button>
       <pre><code>${escapeHtml(JSON.stringify(entry, null, 2))}</code></pre>

diff --git a/src/ts/waterfall/details-overlay/svg-details-overlay.ts b/src/ts/waterfall/details-overlay/svg-details-overlay.ts
@@ -48,7 +48,7 @@ const onTabDataCopyClick = (event: MouseEvent) => {
     el.value = btn.nextElementSibling ? (btn.nextElementSibling as HTMLElement).innerText : "";
     document.body.appendChild(el);
     el.select();
-    el.setSelectionRange(0, 99999);
+    el.setSelectionRange(0, 99999); // for mobile
     document.execCommand("copy");
     document.body.removeChild(el);
   }