unreleased-commits - Restore support for old repository overview

[134130]

Description

• Closes New unreleased-commits causes Switch branches to be hidden in <details> #6922
• Disabled via hotfix: refined-github/yolo@4581c81
• Reason is described here: New unreleased-commits causes Switch branches to be hidden in <details> #6922 (comment)

Test URLs

• https://github.com/refined-github/refined-github

Screenshot

• New repository overview disabled
  

• New repository overview enabled
  

[134130]

table[aria-labelledby="folders-and-files"] means New repository overview's file tree table; But I don't know is there any nice way to describe this with comment.

[134130]

Suggested change

	observe(`table[aria-labelledby="folders-and-files"] ${branchSelector}`, add, {signal});
	observe(`${branchSelector}:not(${branchSelectorParent} ${branchSelector})`, add, {signal});

• We can resolve this like this way too.

[fregante]

Either way, it should be part of the selector we have saved

[134130]

export const branchSelector = '[data-hotkey="w"]';
export const branchSelectorParent = 'details#branch-select-menu';

Since these 2 apis are available currently, It looks hard to me to adding new general selector.

• ${branchSelector}:not(${branchSelectorParent} ${branchSelector}) -> branchSelectorWithoutBranchSelectorParent 😶
  • It looks too long.. and something indescribable.
• table[aria-labelledby="folders-and-files"] ${branchSelector} -> branchSelectorOnNew 👎
  • Avoid new naming

Or merge add() and addLegacy(), and check is parent branchSelectorParent in function and append with that condition?

I don't know which is more refined way.

[fregante]

Let's merge it into a single selector and observer. Then add the condition inside

[134130]

Never same mistake; Both are checked.

• Disable New repository overview
  

• Enable New repository overview