Skip to content

fix(sidebar): truncate long item titles to prevent overflow - #7069

Merged
akshaydeo merged 3 commits into
devfrom
09-11-fix_ui_sidebar_menus_trucation
Sep 12, 2026
Merged

akshaydeo merged 3 commits into
devfrom
09-11-fix_ui_sidebar_menus_trucation

Conversation

@impoiler

Copy link
Copy Markdown
Member

Summary

Sidebar item titles were overflowing their containers when text was long, causing layout issues. This fix ensures text is properly truncated with ellipsis instead of breaking the sidebar layout.

Changes

  • Added min-w-0 to flex containers in sidebar items and sub-items to allow proper shrinking within flex layouts
  • Added truncate and min-w-0 to title <span> elements so long text is clipped with an ellipsis rather than overflowing
  • Added shrink-0 to sub-item icons to prevent them from being compressed when adjacent text truncates

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (React)
  • Docs

How to test

Navigate to a sidebar that contains items or sub-items with long titles and verify that the text truncates cleanly with an ellipsis rather than overflowing or wrapping outside the sidebar bounds.

cd ui
pnpm i || npm i
pnpm build || npm run build

Screenshots/Recordings

Add before/after screenshots showing the sidebar with long item titles to confirm truncation behavior.

Breaking changes

  • Yes
  • No

Related issues

Security considerations

None.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai

coderabbitai Bot commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: e7dfd08d-8b8a-4505-915e-0230c59bc445

📥 Commits

Reviewing files that changed from the base of the PR and between 4b7d9e2 and a5e04ed.

📒 Files selected for processing (1)
  • ui/components/sidebar.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Bug Fixes
    • Improved sidebar navigation layout to prevent horizontal overflow.
    • Long navigation labels now truncate cleanly in expanded and collapsed sidebar views.

Walkthrough

The sidebar updates item and submenu layouts to allow content shrinking, truncate long titles, and preserve icon sizing in expanded and collapsed views.

Changes

Sidebar overflow handling

Layer / File(s) Summary
Sidebar and submenu content sizing
ui/components/sidebar.tsx
Sidebar items and submenus now shrink within available space, truncate long titles, and prevent icons from shrinking.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: akshaydeo

Merge Risk: ⚪ Minimal · up to cd368

The sidebar truncation and sizing updates are ready to merge with no identified user-facing regression risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description clearly explains the sidebar overflow bug, lists the UI changes, identifies the change as a bug fix, and provides validation steps. Screenshots, related issues, and checklist confirmat…
Title check ✅ Passed The title clearly and concisely describes the primary change: truncating long sidebar item titles to prevent overflow.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 09-11-fix_ui_sidebar_menus_trucation

Comment @coderabbitai help to get the list of available commands.

impoiler commented Sep 11, 2026 •

Copy link
Copy Markdown
Member Author

coderabbitai[bot]
coderabbitai Bot previously approved these changes Sep 11, 2026
@impoiler
impoiler force-pushed the 09-09-tests_adds_e2e_test_covorage_for_regex_models_pattern branch from 4b7d9e2 to d5afbd6 Compare September 11, 2026 06:12
@impoiler
impoiler force-pushed the 09-11-fix_ui_sidebar_menus_trucation branch from a5e04ed to 924d198 Compare September 11, 2026 06:12
@impoiler
impoiler force-pushed the 09-09-tests_adds_e2e_test_covorage_for_regex_models_pattern branch from d5afbd6 to 1c987b3 Compare September 11, 2026 07:06
@impoiler
impoiler force-pushed the 09-11-fix_ui_sidebar_menus_trucation branch from 924d198 to 05e8450 Compare September 11, 2026 07:06
@impoiler
impoiler force-pushed the 09-09-tests_adds_e2e_test_covorage_for_regex_models_pattern branch from 1c987b3 to f89af37 Compare September 12, 2026 05:31
@impoiler
impoiler force-pushed the 09-11-fix_ui_sidebar_menus_trucation branch from 05e8450 to 713c109 Compare September 12, 2026 05:31
@impoiler
impoiler changed the base branch from 09-09-tests_adds_e2e_test_covorage_for_regex_models_pattern to graphite-base/7069 September 12, 2026 06:23
@impoiler
impoiler force-pushed the 09-11-fix_ui_sidebar_menus_trucation branch from 713c109 to cd36837 Compare September 12, 2026 06:32
@impoiler
impoiler changed the base branch from graphite-base/7069 to 09-12-feat_show_all_authenticaiton_method_in_mcp_usage_guide_sheet September 12, 2026 06:32

akshaydeo commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

Merge activity

  • Sep 12, 6:38 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Sep 12, 6:40 AM UTC: @akshaydeo merged this pull request with Graphite.

@akshaydeo
akshaydeo changed the base branch from 09-12-feat_show_all_authenticaiton_method_in_mcp_usage_guide_sheet to graphite-base/7069 September 12, 2026 06:39
@akshaydeo
akshaydeo changed the base branch from graphite-base/7069 to dev September 12, 2026 06:40
@akshaydeo
akshaydeo dismissed coderabbitai[bot]’s stale review September 12, 2026 06:40

The base branch was changed.

@akshaydeo
akshaydeo merged commit 3a6d793 into dev Sep 12, 2026
11 checks passed
@akshaydeo
akshaydeo deleted the 09-11-fix_ui_sidebar_menus_trucation branch September 12, 2026 06:40
@akshaydeo akshaydeo mentioned this pull request Sep 15, 2026
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.

2 participants