From 4a446793212b97dc16f87c99f99def6fb5fcd1d2 Mon Sep 17 00:00:00 2001
From: chocolateboy <chocolate@cpan.org>
Date: Fri, 17 Nov 2017 19:02:13 +0000
Subject: improve #linkPrev and #linkNext:

- add support for <link> elements
- match if there's more than one rel e.g. <a rel="nofollow next">...</a>
- more tests
---
 src/content/navigates.js | 31 +++++++++++++++++--------------
 1 file changed, 17 insertions(+), 14 deletions(-)

(limited to 'src/content/navigates.js')

diff --git a/src/content/navigates.js b/src/content/navigates.js
index 64e5fc0..3e12a6f 100644
--- a/src/content/navigates.js
+++ b/src/content/navigates.js
@@ -2,13 +2,14 @@ const PREV_LINK_PATTERNS = [
   /\bprev\b/i, /\bprevious\b/i, /\bback\b/i,
   /</, /\u2039/, /\u2190/, /\xab/, /\u226a/, /<</
 ];
+
 const NEXT_LINK_PATTERNS = [
   /\bnext\b/i,
   />/, /\u203a/, /\u2192/, /\xbb/, /\u226b/, />>/
 ];
 
 const findLinkByPatterns = (win, patterns) => {
-  let links = win.document.getElementsByTagName('a');
+  const links = win.document.getElementsByTagName('a');
   return Array.prototype.find.call(links, (link) => {
     return patterns.some(ptn => ptn.test(link.textContent));
   });
@@ -22,30 +23,32 @@ const historyNext = (win) => {
   win.history.forward();
 };
 
-const linkPrev = (win) => {
-  let link = win.document.querySelector('a[rel=prev]');
+const linkCommon = (win, rel, patterns) => {
+  let link = win.document.querySelector(`link[rel~=${rel}][href]`);
+
   if (link) {
-    return link.click();
+    win.location = link.getAttribute('href');
+    return;
   }
-  link = findLinkByPatterns(win, PREV_LINK_PATTERNS);
+
+  link = win.document.querySelector(`a[rel~=${rel}]`) ||
+    findLinkByPatterns(win, patterns);
+
   if (link) {
     link.click();
   }
 };
 
+const linkPrev = (win) => {
+  linkCommon(win, 'prev', PREV_LINK_PATTERNS);
+};
+
 const linkNext = (win) => {
-  let link = win.document.querySelector('a[rel=next]');
-  if (link) {
-    return link.click();
-  }
-  link = findLinkByPatterns(win, NEXT_LINK_PATTERNS);
-  if (link) {
-    link.click();
-  }
+  linkCommon(win, 'next', NEXT_LINK_PATTERNS);
 };
 
 const parent = (win) => {
-  let loc = win.location;
+  const loc = win.location;
   if (loc.hash !== '') {
     loc.hash = '';
     return;
-- 
cgit v1.2.3


From ee3cd3faabbe64265746ef39d419f63db24d11ba Mon Sep 17 00:00:00 2001
From: chocolateboy <chocolate@cpan.org>
Date: Thu, 23 Nov 2017 14:10:36 +0000
Subject: pagination tweaks and fixes:

- fallback links:
  - select the last matching link rather than the first
  - use `innerText` rather than `textContent`
- use a single regex for each pattern rather than an array
- fix markup typo in test
---
 src/content/navigates.js       | 49 +++++++++++++++++++++++-------------------
 test/content/navigates.test.js |  6 +++---
 2 files changed, 30 insertions(+), 25 deletions(-)

(limited to 'src/content/navigates.js')

diff --git a/src/content/navigates.js b/src/content/navigates.js
index 3e12a6f..c9baa30 100644
--- a/src/content/navigates.js
+++ b/src/content/navigates.js
@@ -1,18 +1,18 @@
-const PREV_LINK_PATTERNS = [
-  /\bprev\b/i, /\bprevious\b/i, /\bback\b/i,
-  /</, /\u2039/, /\u2190/, /\xab/, /\u226a/, /<</
-];
-
-const NEXT_LINK_PATTERNS = [
-  /\bnext\b/i,
-  />/, /\u203a/, /\u2192/, /\xbb/, /\u226b/, />>/
-];
-
-const findLinkByPatterns = (win, patterns) => {
-  const links = win.document.getElementsByTagName('a');
-  return Array.prototype.find.call(links, (link) => {
-    return patterns.some(ptn => ptn.test(link.textContent));
-  });
+const REL_PATTERN = {
+  prev: /^(?:prev(?:ious)?|older)\b|\u2039|\u2190|\xab|\u226a|<</i,
+  next: /^(?:next|newer)\b|\u203a|\u2192|\xbb|\u226b|>>/i,
+};
+
+// Return the last element in the document matching the supplied selector
+// and the optional filter, or null if there are no matches.
+const selectLast = (win, selector, filter) => {
+  let nodes = win.document.querySelectorAll(selector);
+
+  if (filter) {
+    nodes = Array.from(nodes).filter(filter);
+  }
+
+  return nodes.length ? nodes[nodes.length - 1] : null;
 };
 
 const historyPrev = (win) => {
@@ -23,16 +23,21 @@ const historyNext = (win) => {
   win.history.forward();
 };
 
-const linkCommon = (win, rel, patterns) => {
-  let link = win.document.querySelector(`link[rel~=${rel}][href]`);
+// Code common to linkPrev and linkNext which navigates to the specified page.
+const linkRel = (win, rel) => {
+  let link = selectLast(win, `link[rel~=${rel}][href]`);
 
   if (link) {
-    win.location = link.getAttribute('href');
+    win.location = link.href;
     return;
   }
 
-  link = win.document.querySelector(`a[rel~=${rel}]`) ||
-    findLinkByPatterns(win, patterns);
+  const pattern = REL_PATTERN[rel];
+
+  link = selectLast(win, `a[rel~=${rel}][href]`) ||
+    // `innerText` is much slower than `textContent`, but produces much better
+    // (i.e. less unexpected) results
+    selectLast(win, 'a[href]', lnk => pattern.test(lnk.innerText));
 
   if (link) {
     link.click();
@@ -40,11 +45,11 @@ const linkCommon = (win, rel, patterns) => {
 };
 
 const linkPrev = (win) => {
-  linkCommon(win, 'prev', PREV_LINK_PATTERNS);
+  linkRel(win, 'prev');
 };
 
 const linkNext = (win) => {
-  linkCommon(win, 'next', NEXT_LINK_PATTERNS);
+  linkRel(win, 'next');
 };
 
 const parent = (win) => {
diff --git a/test/content/navigates.test.js b/test/content/navigates.test.js
index d8a3316..f1f0741 100644
--- a/test/content/navigates.test.js
+++ b/test/content/navigates.test.js
@@ -53,7 +53,7 @@ describe('navigates module', () => {
     ));
 
     it('navigates to <a> elements whose text matches "previous"', testPrev(
-      '<a href="#dummy">preview</a><a href="#prev">go to previous</a>'
+      '<a href="#dummy">previously</a><a href="#prev">previous page</a>'
     ));
 
     it('navigates to <a> elements whose decoded text matches "<<"', testPrev(
@@ -119,11 +119,11 @@ describe('navigates module', () => {
     ));
 
     it('prefers link[rel~=next] to a[rel~=next]', testNext(
-      '<a rel="next" href="#dummy">click me<><link rel="next" href="#next" />'
+      '<a rel="next" href="#dummy">click me</a><link rel="next" href="#next" />'
     ));
 
     it('prefers a[rel~=next] to a::text(pattern)', testNext(
-      '<a href="#dummy">go to next</a><a rel="next" href="#next">click me</a>'
+      '<a href="#dummy">next page</a><a rel="next" href="#next">click me</a>'
     ));
   });
 
-- 
cgit v1.2.3