Skip to content

refactor: skill card with tag#981

Open
geoffrey-xiao wants to merge 1 commit intoopenclaw:mainfrom
geoffrey-xiao:feat/refactor_card_tag
Open

refactor: skill card with tag#981
geoffrey-xiao wants to merge 1 commit intoopenclaw:mainfrom
geoffrey-xiao:feat/refactor_card_tag

Conversation

@geoffrey-xiao
Copy link
Contributor

Summary

Moves skill card tags below the title to improve card hierarchy and readability in the skills grid.

What Changed

  • Updated src/components/SkillCard.tsx so the skill title renders before the tag row.
  • Kept the existing tag styling and behavior unchanged; this is a layout-only refactor.

before
image

after
image

Why

The previous layout put badges/platform tags above the title, which made card scanning less natural. Rendering the title first gives the card a clearer reading order:

  1. title
  2. tags
  3. summary
  4. owner/stats

This works better for both promotional badges like Highlighted and technical tags like OS labels.

@vercel
Copy link
Contributor

vercel bot commented Mar 17, 2026

@geoffrey-xiao is attempting to deploy a commit to the 0xBuns Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 17, 2026

Greptile Summary

This is a minimal layout-only refactor of SkillCard.tsx that reorders the card's DOM elements so the skill title (<h3>) renders before the tags row rather than after it. No logic, styling, or behavior changes were made.

  • The <h3 className="skill-card-title"> element was moved from below the hasTags block to above it, establishing a clearer visual hierarchy: title → tags → summary → footer.
  • All tag rendering logic (badges, chip, platformLabels) and conditional rendering (hasTags) remain identical.
  • No props, types, or CSS class names were changed.

No issues were found in this PR.

Confidence Score: 5/5

  • This PR is safe to merge — it is a trivial one-line DOM reorder with no behavioral, logical, or styling impact.
  • The entire change is moving a single JSX element one position up in the render tree. There are no conditional branches, no new dependencies, no prop or type changes, and no CSS modifications. The risk surface is essentially zero.
  • No files require special attention.

Last reviewed commit: a9e6059

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