Skip to content

fix(desktop): align changelog title gutter#5435

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/changelog-collapsed-sidebar-title
Jun 2, 2026
Merged

fix(desktop): align changelog title gutter#5435
ComputelessComputer merged 1 commit into
mainfrom
fix/changelog-collapsed-sidebar-title

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

@ComputelessComputer ComputelessComputer commented Jun 1, 2026

Summary:

  • Apply the collapsed sidebar header gutter to the changelog title.
  • Add regression coverage for collapsed and expanded sidebar timeline states.

Test Plan:

  • pnpm exec dprint fmt
  • pnpm -F @hypr/desktop test src/changelog/index.test.tsx
  • pnpm -F @hypr/desktop typecheck

Note

Low Risk
UI-only padding and config/shell reads in the changelog tab, with tests and no auth or data changes.

Overview
When sidebar timeline mode is on and the left sidebar is collapsed, the changelog tab header now gets the same pl-[156px] gutter used elsewhere (e.g. session note headers) so the “What’s new” title lines up with the timeline layout.

The gutter is only applied when timeline mode is enabled and the sidebar is collapsed; an expanded sidebar leaves the header unchanged. Vitest coverage was added for both cases.

Reviewed by Cursor Bugbot for commit c304e82. Bugbot is set up for automated code reviews on this repo. Configure here.

Apply the collapsed sidebar header gutter to the changelog title and cover it with a regression test.
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 1, 2026

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit c304e82
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a1d8fb227f3b00008dcb05b

@ComputelessComputer ComputelessComputer merged commit f5d2e9c into main Jun 2, 2026
10 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/changelog-collapsed-sidebar-title branch June 2, 2026 01:22
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