Skip to content

fix: restore PR and commit enhancements after GitHub DOM changes - #61

Merged
rrbe merged 2 commits into
mainfrom
fix/commits-content-layout
Oct 13, 2026
Merged

rrbe merged 2 commits into
mainfrom
fix/commits-content-layout

Conversation

@rrbe

@rrbe rrbe commented Oct 13, 2026 •

Copy link
Copy Markdown
Owner

GitHub dropped CSS-module prefixes from its list and Files changed DOM. Commit diff badges were appended directly to grid rows before their titles, PR-list height reservation stopped matching, and file/folder folding controls were no longer injected.

Use a shared selector for the current main-content wrapper, distinguished from the status icon's separate wrapper, to keep commit stats and PR branch/diff/review badges below metadata. Apply the same structure to loading-space reservation and retain compact-row behavior.

Update Files changed sidebar, header, and toolbar selectors. Expanding all folders now proceeds one depth per animation frame because React unmounts descendants when their parent closes; collapsing still runs deepest first.

Validation:

  • pnpm test: 306 tests passed; pnpm typecheck, pnpm build, and git diff --check passed.
  • Run the PR dashboard integration suite against both prefixed and current DOM, covering branch/diff/review ordering, label filtering, compact density, hydration, row replacement, and lazy conflict indicators.
  • Add regressions for commit placement and placeholders, status-icon exclusion, file/folder control placement and deduplication, and descendants mounted after parent expansion. The descendant regression fails with the previous expansion logic and passes with this fix.
  • Live browser checks confirmed 35 commit badges below metadata and inspected branch/diff/review badges on all 14 repository PR rows. Temporarily applied the updated folding module and verified all four files could be collapsed and expanded. Reproduced the nested-folder expansion issue in the live React tree; the mounted-descendant regression covers its fix.

Also inspected PR number-copy, label filtering, review/conflict, and Approve now DOM hooks; no additional selector failure was found in those paths. The installed extension was not replaced during the browser checks.

@rrbe rrbe changed the title fix: restore commit stats layout after GitHub DOM changes fix: restore PR and commit enhancements after GitHub DOM changes Oct 13, 2026
@rrbe
rrbe merged commit fd76887 into main Oct 13, 2026
1 check passed
@rrbe
rrbe deleted the fix/commits-content-layout branch October 13, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant