feat: Trigger ignore whitespace on PR load (#340)

Automatically trigger ignore whitespace setting when
loading pull request in the Old PR experience.

Fixes #306
Fixes #311

src/ignore-whitespace/ignore-whitespace.js

@@ -1,6 +1,7 @@
 // @flow
+import elementReady from 'element-ready'
 
-export default function init(prRow: Element) {
+export function ignoreWhitespaceSearchParam(prRow: Element) {
     const link: HTMLAnchorElement = (prRow.querySelector(
         'a[data-qa="pull-request-row-link"]'
     ): any)
@@ -10,3 +11,12 @@ export default function init(prRow: Element) {
     url.search = searchParams.toString()
     link.href = url.href
 }
+
+export async function ignoreWhitespaceInit() {
+    const toggle: HTMLElement = await elementReady(
+        '[data-key="ignoreWhitespace"]'
+    )
+    if (toggle && toggle.getAttribute('aria-checked') === 'false') {
+        toggle.click()
+    }
+}

src/ignore-whitespace/ignore-whitespace.spec.js

@@ -1,9 +1,11 @@
 import { h } from 'dom-chef'
 import test from 'ava'
 
-import ignoreWhitespace from '.'
+import { ignoreWhitespaceSearchParam } from '.'
 
 import '../../test/setup-jsdom'
+import { ignoreWhitespaceInit } from './ignore-whitespace'
+import { cleanDocumentBody } from '../../test/test-utils'
 
 test('should transform pull request link to add ignore whitespace query param to 1', t => {
     const actual = (
@@ -30,7 +32,7 @@ test('should transform pull request link to add ignore whitespace query param to
         </div>
     )
 
-    ignoreWhitespace(actual)
+    ignoreWhitespaceSearchParam(actual)
 
     t.is(actual.outerHTML, expected.outerHTML)
 })
@@ -60,7 +62,7 @@ test('should transform pull request link to toggle ignore whitespace query param
         </div>
     )
 
-    ignoreWhitespace(actual)
+    ignoreWhitespaceSearchParam(actual)
 
     t.is(actual.outerHTML, expected.outerHTML)
 })

src/ignore-whitespace/index.js

@@ -1,2 +1,5 @@
 // @flow
-export { default } from './ignore-whitespace'
+export {
+    ignoreWhitespaceSearchParam,
+    ignoreWhitespaceInit,
+} from './ignore-whitespace'

src/main.js

@@ -14,7 +14,10 @@ import collapseDiff from './collapse-diff'
 import defaultMergeStrategy from './default-merge-strategy'
 import diffIgnore from './diff-ignore'
 import removeDiffsPlusesAndMinuses from './diff-pluses-and-minuses'
-import ignoreWhitespace from './ignore-whitespace'
+import {
+    ignoreWhitespaceSearchParam,
+    ignoreWhitespaceInit,
+} from './ignore-whitespace'
 import insertCopyFilename from './insert-copy-filename'
 import insertCopyFilenameNew from './insert-copy-filename-new'
 import keymap from './keymap'
@@ -110,7 +113,7 @@ function pullrequestListRelatedFeatures(config) {
         'tr[data-qa="pull-request-row"]',
         function() {
             if (config.ignoreWhitespace) {
-                ignoreWhitespace(this)
+                ignoreWhitespaceSearchParam(this)
             }
 
             if (config.augmentPrEntry) {
@@ -205,6 +208,10 @@ function codeReviewFeatures(config) {
         setLineLengthLimit(config.lineLengthLimit, config.stickyHeader)
     }
 
+    if (config.ignoreWhitespace) {
+        ignoreWhitespaceInit()
+    }
+
     if (config.stickyHeader) {
         setStickyHeader()
     }