fix(button,badge): make cursor consistent — pointer on Button, default on Badge (#309)

* fix(button): restore pointer cursor on Button

Tailwind v4 dropped the Preflight rule that gave button/[role="button"] a
cursor: pointer, so the base Button showed the default arrow cursor while
components that hardcode astw:cursor-pointer (e.g. ActionPanel) did not.
Add astw:cursor-pointer to the buttonVariants base classes so all Button
instances are consistent. Disabled buttons keep the default cursor via the
existing astw:disabled:pointer-events-none.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* fix(badge): pin cursor: default so badges don't inherit pointer

A Badge is non-interactive but renders a <div> with no cursor, so it inherited
cursor: pointer from clickable ancestors (e.g. a DataTable row with onClickRow).
Add astw:cursor-default to badgeVariants so a badge never signals clickability.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* docs(badge): note badges are non-interactive (default cursor)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: erickteowarang

.changeset/button-cursor-pointer.md

@@ -0,0 +1,17 @@
+---
+"@tailor-platform/app-shell": patch
+---
+
+Fix cursor consistency across `Button` and `Badge`:
+
+- **`Button`**: restore the pointer cursor. Tailwind v4 dropped the Preflight
+  rule that gave `button`/`[role="button"]` a `cursor: pointer`, so the base
+  `Button` showed the default arrow cursor while components that hardcode
+  `astw:cursor-pointer` (e.g. `ActionPanel`) did not. Added `astw:cursor-pointer`
+  to the `buttonVariants` base classes so all `Button` instances are consistent.
+  Disabled buttons keep the default cursor via the existing
+  `astw:disabled:pointer-events-none`.
+- **`Badge`**: pin `astw:cursor-default`. A `Badge` is non-interactive but
+  renders a `<div>` with no cursor of its own, so it inherited `cursor: pointer`
+  from clickable ancestors (e.g. a `DataTable` row with `onClickRow`). Explicitly
+  setting the default cursor stops a badge from signalling clickability.

docs/components/badge.md

@@ -223,7 +223,7 @@ function ProductBadge({ product }: { product: Product }) {
 - Use semantic color meanings consistently (green = success, red = error, etc.)
 - Don't rely solely on color to convey information
 - Consider adding icons or text labels for clarity
-- Badges receive focus when interactive (clickable)
+- Badges are non-interactive: they render with the default cursor (not a pointer) and won't suggest clickability, even when placed inside a clickable container such as a `DataTable` row with `onClickRow`
 
 ## Usage in DescriptionCard