Skip to content

fix: sidebar tree ui - #2739

Merged
deadlyjack merged 1 commit into
mainfrom
fix/sidebar-tree-ui
Aug 14, 2026
Merged

fix: sidebar tree ui#2739
deadlyjack merged 1 commit into
mainfrom
fix/sidebar-tree-ui

Conversation

@deadlyjack

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds root-folder-specific overflow styling to truncate long labels while retaining the existing behavior for nested rows.

  • Adds shrinking and ellipsis declarations for root-folder text.
  • Adds a unit test for the root and nested computed-style declarations.

Confidence Score: 4/5

The PR appears safe to merge, although its regression test does not prove that long root labels are actually truncated in production layout.

The selector matches the production root-folder structure and no blocking failure was established, but happy-dom only confirms the CSS declarations and cannot validate the intended ellipsis rendering.

Files Needing Attention: tests/unit/fileSidebarStyles.test.js

Important Files Changed

Filename Overview
src/sidebarApps/files/style.scss Adds a production-DOM-matching selector that changes root-folder labels from max-content overflow to shrinkable ellipsis styling.
tests/unit/fileSidebarStyles.test.js Verifies the declared computed styles but does not exercise the width-dependent truncation behavior in a layout-capable browser.

Reviews (1): Last reviewed commit: "fix: sidebar tree ui" | Re-trigger Greptile

Comment thread tests/unit/fileSidebarStyles.test.js
@deadlyjack
deadlyjack added this pull request to the merge queue Aug 14, 2026
Merged via the queue into main with commit bd4a788 Aug 14, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Aug 14, 2026
@deadlyjack
deadlyjack deleted the fix/sidebar-tree-ui branch August 14, 2026 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant