Skip to content

fix(contest): style contest share as a proper harmony button#14528

Open
dylanjeffers wants to merge 1 commit into
mainfrom
fix/contest-share-button
Open

fix(contest): style contest share as a proper harmony button#14528
dylanjeffers wants to merge 1 commit into
mainfrom
fix/contest-share-button

Conversation

@dylanjeffers

Copy link
Copy Markdown
Contributor

Description

The share action on the contest page rendered as a bare IconButton — an unstyled arrow icon sitting awkwardly next to the properly styled Follow and Enter Contest buttons.

This replaces it with an icon-only harmony Button (variant='secondary', size='small', iconLeft={IconShare}), matching the Figma design and the height/border/hover treatment of the neighboring buttons. It's the same icon-only button pattern already used by the profile page StatBanner action row. The header loading skeleton is updated to match the new rectangular shape.

Before: bare arrow icon, no button chrome
After: bordered secondary button, 32px tall like Follow / Enter Contest, purple fill on hover

How Has This Been Tested?

  • Verified in the local web client on a live contest page (/soundsbymystic/contest/mstic-dfwm-1):
    • Button renders with the same height (32px), corner radius, and inset border as the adjacent Follow button
    • Hover fills purple per the secondary variant
    • Clicking still opens the Share Contest modal
    • aria-label="Share contest" preserved
  • eslint passes on the modified file

🤖 Generated with Claude Code

The contest page share action rendered as a bare IconButton — an
unstyled arrow icon sitting next to the properly styled Follow and
Enter Contest buttons. Replace it with an icon-only secondary Button
(size small) so it matches the height, border, and hover treatment of
the neighboring actions, per the Figma spec and the same pattern used
by the profile StatBanner action row. Update the loading skeleton to
match the new rectangular shape.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e2739ae

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant