Skip to content

Commit

Permalink
Merge pull request #217 from crashbell/master
Browse files Browse the repository at this point in the history
Fix finding a wrong branch in pull request page
  • Loading branch information
buunguyen committed Dec 7, 2015
2 parents 55192e4 + 26de85b commit 124c2d8
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion src/adapters/github.js
Original file line number Diff line number Diff line change
Expand Up @@ -147,15 +147,21 @@ class GitHub extends Adapter {
const GH_BRANCH_SEL_1 = '[aria-label="Switch branches or tags"]'
const GH_BRANCH_SEL_2 = '.repo-root a[data-branch]'
const GH_BRANCH_SEL_3 = '.repository-sidebar a[aria-label="Code"]'
const GH_BRANCH_SEL_4 = 'link[title*="Recent Commits to"]'
const GH_BRANCH_SEL_4 = '.current-branch'
const GH_BRANCH_SEL_5 = 'link[title*="Recent Commits to"]'

const branch =
// Detect branch in code page (don't care about non-code pages, let them use the next fallback)
$(GH_BRANCH_SEL_1).attr('title') || $(GH_BRANCH_SEL_2).data('branch') ||
// Non-code page
($(GH_BRANCH_SEL_3).attr('href') || ' ').match(/([^\/]+)/g)[3] ||
// Non-code page (new design)
// Specific handle /commit page
($(GH_BRANCH_SEL_4).attr('title') || ' ').match(/([^\:]+)/g)[1] ||
// Ignore if Github expands one more <link> - use last selected one instead
($(GH_BRANCH_SEL_5).length === 1
&& ($(GH_BRANCH_SEL_5).attr('title') || ' ').match(/([^\:]+)/g)[1]) ||

// Reuse last selected branch if exist
(currentRepo.username === username && currentRepo.reponame === reponame && currentRepo.branch)
// Get default branch from cache
Expand Down

0 comments on commit 124c2d8

Please sign in to comment.