Merge pull request #7 from berzniz/only_affect_styles_on_diff_pages

Author: berzniz

src/js/index.jsx

@@ -14,11 +14,15 @@ const observe = () => {
   observer.observe(pjaxContainer, { childList: true })
 }
 
-const renderTree = (rootElement) => {
+const renderTree = () => {
   const fileCount = parseInt((document.getElementById('files_tab_counter') || { innerText: 0 }).innerText, 10)
-  if (fileCount === 0) {
+  const rootElement = createRootElement()
+  const enabled = Boolean(rootElement && fileCount > 0)
+  document.body.classList.toggle('enable_better_github_pr', enabled)
+  if (!enabled) {
     return
   }
+
   const { tree, count } = createFileTree()
   render(<Tree root={tree} />, rootElement)
   if (fileCount !== count) {
@@ -28,11 +32,7 @@ const renderTree = (rootElement) => {
 
 const start = () => {
   observe()
-  const rootElement = createRootElement()
-  if (!rootElement) {
-    return
-  }
-  renderTree(rootElement)
+  renderTree()
 }
 
 observe()

src/js/style.css

@@ -21,11 +21,12 @@
 
 /* GitHub page changes */
 
-.diff-view {
+.enable_better_github_pr .diff-view,
+.enable_better_github_pr .commit.full-commit.prh-commit {
     padding-left: 250px;
 }
 
-.diff-view .file-header {
+.enable_better_github_pr .diff-view .file-header {
     position: sticky;
     top: 60px;
     z-index: 1;