Skip to content

feat: sidebar toggle fix + Portuguese (PT-BR) translation - #25922

Open
schummerdev wants to merge 1 commit into
NousResearch:mainfrom
schummerdev:feat/sidebar-toggle-ptbr
Open

feat: sidebar toggle fix + Portuguese (PT-BR) translation#25922
schummerdev wants to merge 1 commit into
NousResearch:mainfrom
schummerdev:feat/sidebar-toggle-ptbr

Conversation

@schummerdev

Copy link
Copy Markdown

Summary

  • Fix sidebar toggle on desktop: button now in header (always visible)
  • Fix margin-left bug: content now properly expands when sidebar is hidden
  • Add hover-to-expand behavior when sidebar is collapsed
  • Add all UI strings Portuguese (PT-BR) translation in i18n
  • Fix responsive behavior for mobile and desktop

Changes

  • web/src/App.tsx: Complete sidebar overhaul with toggle in header, hover expand, and resize functionality
  • i18n: Portuguese translation support for all UI components

Testing

  • Desktop: Toggle button in header works correctly
  • Mobile: Responsive behavior maintained
  • Sidebar: Hover expand when collapsed works
  • Content: Properly expands to fill space when sidebar hidden
  • Portuguese: All UI strings translated and working

- Fix sidebar toggle on desktop: button now in header (always visible)
- Fix margin-left bug: content now properly expands when sidebar is hidden
- Add hover-to-expand behavior when sidebar is collapsed
- Add all UI strings Portuguese (PT-BR) translation in i18n
- Fix responsive behavior for mobile and desktop
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery labels May 14, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate PT-BR translation component — prior PRs: #15737 (closed), #22063 (closed), #22182 (closed salvage of #15737). The sidebar toggle fix is new. Related issue: #15738.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform area/i18n Localization, locales, translations labels Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/i18n Localization, locales, translations comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants