Skip to content

fix: mobile sidebar, complete version diff, and sidebar reopen#8

Merged
Sathvik-Rao merged 1 commit into
mainfrom
feat-ui
Jun 9, 2026
Merged

fix: mobile sidebar, complete version diff, and sidebar reopen#8
Sathvik-Rao merged 1 commit into
mainfrom
feat-ui

Conversation

@Sathvik-Rao

@Sathvik-Rao Sathvik-Rao commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Summary

Three UI fixes:

  • Mobile: the drawer was sized from --sw, which the desktop resize logic
    clamps to its 200px minimum on phone widths; combined with row action
    buttons that reserve ~90px even while hidden, the session-title column
    collapsed to 27px ("Se..."). Use a fixed min(86vw,330px) drawer width and
    reveal row actions on touch so titles are readable and favorite/rename/
    delete are reachable.
  • Diff: _buildDiffHtml only compared summary, competencies, experience,
    skills, and scores, so newly added sections (projects, education, certs,
    open source, publications, patents, awards, volunteer, languages,
    interests, contact) never appeared. Add serializers for every section;
    added entries render as additions, removed as deletions, edits keep
    word-level highlighting. Experience now walks the longer job list so
    added/removed jobs show too.
  • Sidebar: the expand button lived inside #sb, which is width:0;
    overflow:hidden when collapsed, so it was clipped and unclickable — no way
    to reopen. Move it into the toolbar and restyle it.

Type of change

  • fix: ... — bug fix → patch bump (0.0.x)
  • feat: ... — new feature → minor bump (0.x.0)
  • feat!: ... / BREAKING CHANGE — breaking change → major bump (x.0.0)
  • chore: / docs: / ci: / refactor: — no release created

PR title format

Your PR title becomes the commit message on merge and determines the version bump:

feat(resume): add cover letter section
fix: pdf export crash on empty work history
feat!: replace /generate endpoint with /build (breaking)
chore: update dev dependencies

Pattern: type(optional-scope): short description in lowercase

Checklist

  • Tests added / updated under backend/tests/ for any changed backend/app/ code
  • pytest passes locally (cd backend && pytest)
  • PR title follows the format above

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Suggested tests for this PR

PR title: fix(frontend): mobile sidebar, complete version diff, and sidebar reopen

PR description:

Summary

Three UI fixes:

  • Mobile: the drawer was sized from --sw, which the desktop resize logic
    clamps to its 200px minimum on phone widths; combined with row action
    buttons that reserve ~90px even while hidden, the session-title column
    collapsed to 27px ("Se..."). Use a fixed min(86vw,330px) drawer width and
    reveal row actions on touch so titles are readable and favorite/rename/
    delete are reachable.
  • Diff: _buildDiffHtml only compared summary, competencies, experience,
    skills, and scores, so newly added sections (projects, education, certs,
    open source, publications, patents, awards, volunteer, languages,
    interests, contact) never appeared. Add serializers for every section;
    added entries render as additions, removed as deletions, edits keep
    word-level highlighting. Experience now walks the longer job list so
    added/removed jobs show too.
  • Sidebar: the expand button lived inside #sb, which is width:0;
    overflow:hidden when collapsed, so it was clipped and unclickable — no way
    to reopen. Move it into the toolbar and restyle it.

Type of change

  • fix: ... — bug fix → patch bump (0.0.x)
  • feat: ... — new feature → minor bump (0.x.0)
  • feat!: ... / BREAKING CHANGE — breaking change → major bump (x.0.0)
  • chore: / docs: / ci: / refactor: — no release created

PR title format

Your PR title becomes the commit message on merge and determines the version bump:

feat(resume): add cover letter section
fix: pdf export crash on empty work history
feat!: replace /generate endpoint with /build (breaking)
chore: update dev dependencies

Pattern: type(optional-scope): short description in lowercase

Checklist

  • Tests added / updated under backend/tests/ for any changed backend/app/ code
  • pytest passes locally (cd backend && pytest)
  • PR title follows the format above

No backend/app/**.py files changed — no test suggestions.

@Sathvik-Rao Sathvik-Rao changed the title fix(frontend): mobile sidebar, complete version diff, and sidebar reopen fix: mobile sidebar, complete version diff, and sidebar reopen Jun 9, 2026
@Sathvik-Rao Sathvik-Rao merged commit 8e9d9c4 into main Jun 9, 2026
4 checks passed
@Sathvik-Rao Sathvik-Rao deleted the feat-ui branch June 9, 2026 18:13
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