Merge pull request #9317 from marxin/fix-left-arrow-hotkey

tk0miya · web-flow · commit 355937aef802 · 2021-06-13T20:26:24.000+09:00
Fix missing break statement for arrow shortcuts.
diff --git a/sphinx/themes/basic/static/doctools.js b/sphinx/themes/basic/static/doctools.js
@@ -301,12 +301,14 @@ var Documentation = {
               window.location.href = prevHref;
               return false;
             }
+            break;
           case 39: // right
             var nextHref = $('link[rel="next"]').prop('href');
             if (nextHref) {
               window.location.href = nextHref;
               return false;
             }
+            break;
         }
       }
     });

Original file line number	Diff line number	Diff line change
`@@ -301,12 +301,14 @@ var Documentation = {`
`301`	`301`	`window.location.href = prevHref;`
`302`	`302`	`return false;`
`303`	`303`	`}`
	`304`	`+ break;`
`304`	`305`	`case 39: // right`
`305`	`306`	`var nextHref = $('link[rel="next"]').prop('href');`
`306`	`307`	`if (nextHref) {`
`307`	`308`	`window.location.href = nextHref;`
`308`	`309`	`return false;`
`309`	`310`	`}`
	`311`	`+ break;`
`310`	`312`	`}`
`311`	`313`	`}`
`312`	`314`	`});`