Skip to content

Commit 1522c41

Browse files
LEANDERANTONYclaude
andcommitted
fix(ui): resume-builder export picker — replace dead segmented-toggle with the dropdown
The resume-builder's export theme picker still referenced `.b-artifact-style-option` — the CSS class deleted in 2b5a7ba when the main ArtifactViewer picker became a <select> dropdown. The two buttons in ResumeIntake therefore rendered as unstyled defaults with no visible selected state (operator: "both look like white buttons and i dont know what i am selecting"). Swap the radiogroup + buttons for the same `.b-artifact-style-select` dropdown the main picker uses (consistent UX; scales beyond 2 themes; maximally accessible). Expand from 2 to all 5 themes (parity with the backend export Literal that already supports them). Frontend-only. tsc + eslint clean. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 2b5a7ba commit 1522c41

1 file changed

Lines changed: 31 additions & 37 deletions

File tree

frontend/src/components/workspace/ResumeIntake.tsx

Lines changed: 31 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -686,43 +686,37 @@ export function ResumeIntake({
686686
Download your resume now, or continue to tailor it
687687
for a specific role below.
688688
</p>
689-
<div
690-
role="radiogroup"
691-
aria-label="Resume theme"
692-
style={{
693-
display: "flex",
694-
gap: 6,
695-
flexWrap: "wrap",
696-
}}
697-
>
698-
{(
699-
[
700-
{
701-
value: "professional_neutral",
702-
label: "Professional",
703-
},
704-
{ value: "classic_ats", label: "Classic ATS" },
705-
] as const
706-
).map((option) => (
707-
<button
708-
aria-checked={
709-
builderExportTheme === option.value
710-
}
711-
className="b-artifact-style-option"
712-
data-active={
713-
builderExportTheme === option.value
714-
}
715-
disabled={builderExporting !== null}
716-
key={option.value}
717-
onClick={() =>
718-
onBuilderExportThemeChange(option.value)
719-
}
720-
role="radio"
721-
type="button"
722-
>
723-
{option.label}
724-
</button>
725-
))}
689+
{/* Same dropdown pattern as the main ArtifactViewer
690+
picker (consistent UX; scales beyond 2 themes).
691+
The earlier segmented-button + `.b-artifact-style-
692+
option` class is dead — that CSS was removed when
693+
the main picker switched to a <select>; this
694+
picker still referenced it, which is why the
695+
buttons rendered with no visible selected state.
696+
Now matches main viewer; the 5 themes here mirror
697+
the workspace_models export Literal. */}
698+
<div className="b-artifact-style-select-wrap">
699+
<select
700+
aria-label="Resume theme"
701+
className="b-artifact-style-select"
702+
disabled={builderExporting !== null}
703+
onChange={(event) =>
704+
onBuilderExportThemeChange(
705+
event.target.value as ArtifactTheme,
706+
)
707+
}
708+
value={builderExportTheme}
709+
>
710+
<option value="professional_neutral">
711+
Professional
712+
</option>
713+
<option value="classic_ats">Classic ATS</option>
714+
<option value="modern_blue">Modern Blue</option>
715+
<option value="creative_warm">Creative Warm</option>
716+
<option value="architect_mono">
717+
Architect Mono
718+
</option>
719+
</select>
726720
</div>
727721
<div style={{ display: "flex", gap: 8 }}>
728722
<button

0 commit comments

Comments
 (0)