Skip to content

fix: show pointer cursor on report button#1340

Open
kevinw-openai wants to merge 1 commit intoopenclaw:mainfrom
kevinw-openai:codex/fix-report-button-pointer-cursor
Open

fix: show pointer cursor on report button#1340
kevinw-openai wants to merge 1 commit intoopenclaw:mainfrom
kevinw-openai:codex/fix-report-button-pointer-cursor

Conversation

@kevinw-openai
Copy link
Copy Markdown

Why

image

The skill detail page renders the authenticated Report action as a shared .btn.btn-ghost button, but the base .btn styles did not set a pointer cursor.

That made the Report button show the default arrow on hover instead of the expected hand cursor, which felt broken/inconsistent compared with other interactive controls.

The same issue applies to other places whenever .btn is used

What Changed

  • Added cursor: pointer to the shared .btn base style in src/styles.css

Before

  • Hovering the Report button on the skill detail page showed the default arrow cursor

After

  • Hovering the Report button shows the pointer cursor
  • Other non-disabled buttons that use the shared .btn base style now also consistently show a pointer cursor
  • Disabled buttons still use the existing cursor: not-allowed rule

Testing

  • bun run test -- src/__tests__/skill-detail-page.test.tsx
  • bun run build

Verification

  • Ran the app locally and checked the skill page in the browser
  • Verified via Playwright that the computed cursor for a shared .btn button resolves to "pointer"
  • Confirmed the Report button uses the same shared .btn styling path on the skill detail page

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 28, 2026

@kevinw-openai 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
Copy Markdown
Contributor

greptile-apps bot commented Mar 28, 2026

Greptile Summary

This PR adds cursor: pointer to the shared .btn base style in src/styles.css, fixing the inconsistency where .btn-based buttons (including the authenticated Report button on the skill detail page) displayed the default arrow cursor instead of the expected pointer on hover.

The change is minimal, correctly scoped, and integrates cleanly with the rest of the button styles:

  • The new cursor: pointer rule sits in .btn (the base class)
  • The existing .btn:disabled { cursor: not-allowed; } rule takes precedence for disabled states via CSS specificity, so disabled buttons are unaffected
  • The PR follows Conventional Commits format, includes a screenshot, and documents the test commands run (bun run test, bun run build)

Confidence Score: 5/5

This PR is safe to merge — it is a single, correctly implemented CSS property addition with no regressions.

The change is a one-line CSS fix that adds the missing cursor: pointer to .btn. The existing .btn:disabled rule already correctly overrides it for disabled buttons. No logic, functionality, or other styles are affected. The PR is well-tested, documented, and follows all repo guidelines from AGENTS.md.

No files require special attention.

Important Files Changed

Filename Overview
src/styles.css Adds cursor: pointer to the .btn base style; disabled button override (cursor: not-allowed) remains intact and unaffected.

Reviews (1): Last reviewed commit: "fix: show pointer cursor on report butto..." | Re-trigger Greptile

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