Repository navigation
fix repo panel rendering on top of repo content again - #18
Conversation
findRepoSidebar fell back to main, which always exists, so waitFor resolved before React rendered the About pane. Only use content fallbacks after the wait times out, and make the borderGrid selector hash-agnostic. Fixes HappyHackingSpace#17
|
Caution Review failedThe pull request is closed. WalkthroughThe repository panel lookup now matches hashed ChangesRepository panel target selection
Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix · Severity of issue fixed: Medium Merge Risk:
|
Fixes #17 (regression of #4).
findRepoSidebarfell back tomain, which always exists, sowaitForresolved immediately, before React rendered the About pane. The content fallbacks now live infindRepoContentFallback()and are used only after the wait times out.CodeViewSidebar-module__borderGridno longer matched (class is now hashed); the selector is now hash-agnostic.Tested locally: the panel renders at the top of the About sidebar on repo pages.
Summary by CodeRabbit