Skip to content

fix(blog): copy buttons on prompts hub + numbering fix#552

Open
aafre wants to merge 3 commits into
mainfrom
fix/blog-numbering-copy-buttons
Open

fix(blog): copy buttons on prompts hub + numbering fix#552
aafre wants to merge 3 commits into
mainfrom
fix/blog-numbering-copy-buttons

Conversation

@aafre
Copy link
Copy Markdown
Owner

@aafre aafre commented May 26, 2026

Summary

  • Prompts hub (/blog/ai-resume-prompts-hub): Replace bare <pre><code> blocks with the shared CopyablePrompt component so users can copy workflow prompts with one click — consistent with all other prompt blog pages
  • Best builders (/blog/best-free-resume-builders-2026): Change <h3><p> for recommendation labels in the "Our Honest Recommendation" section — prose-lg was adding ~26px top margin to h3 elements, misaligning the green numbered circles

Test plan

  • All 1445 unit tests pass
  • Visit /blog/ai-resume-prompts-hub → "How to Build an AI Resume Workflow" — each of the 5 steps shows a Copy button, clicking copies prompt text, "Copied!" feedback appears for 2s
  • Visit /blog/best-free-resume-builders-2026 → "Our Honest Recommendation" — green numbered circles are tightly aligned with bold text, no large gaps

aafre added 2 commits May 26, 2026 08:50
…ring

Prompts hub: add a Copy button to each dark code block in the agentic
workflow section so users can copy prompt text with one click.

Best builders: change <h3> to <p> for recommendation labels — prose-lg
was adding large top margins to h3 elements, misaligning the green
numbered circles.
Replace custom dark pre/code blocks with the shared CopyablePrompt
component used by all other prompt pages. Consistent copy-to-clipboard
UX across the blog. Update test assertion to match new markup.
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request replaces static code blocks with a reusable CopyablePrompt component in the AI Resume Prompts Hub and updates the corresponding test assertions. Additionally, it improves heading semantics in the Best Free Resume Builders component. The reviewer suggested using a semantically correct <code> tag instead of a <span> within the new CopyablePrompt wrapper to improve accessibility and SEO.

Comment on lines +574 to +576
<CopyablePrompt title="Prompt" copyText={step.code}>
<span className="whitespace-pre-wrap">{step.code}</span>
</CopyablePrompt>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Using a tag instead of a is more semantically correct for representing code snippets or prompts, which improves accessibility for screen readers and SEO.

Suggested change
<CopyablePrompt title="Prompt" copyText={step.code}>
<span className="whitespace-pre-wrap">{step.code}</span>
</CopyablePrompt>
<CopyablePrompt title="Prompt" copyText={step.code}>
<code className="whitespace-pre-wrap">{step.code}</code>
</CopyablePrompt>

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gemini-code-assist Good catch — applied in 991b6c5. <code> is more semantic for copy-paste prompt snippets.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understood. Since the change has already been applied and verified, no further action is required on this thread.

@aafre aafre self-assigned this May 26, 2026
More semantically correct for copy-paste prompt snippets.
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