From b2227ccd6f232e3aa0ab54528940eadeb3e97e3f Mon Sep 17 00:00:00 2001 From: paanSinghCoder Date: Fri, 5 Jun 2026 14:20:20 +0530 Subject: [PATCH 1/6] feat!: remove misused styling props Remove one-off styling props that should be handled via className/style, and reshape Callout's `outline` boolean into a `variant` axis. CSS defaults are preserved, so default rendering is unchanged. BREAKING CHANGE: - Callout: removed `width`; `outline` boolean -> `variant="outline"` (new `variant?: 'solid' | 'outline'`, default `solid`; orthogonal to `type`, so `type="success" variant="outline"` still combines) - Button: removed `width`, `maxWidth` - Input: removed `width` - TextArea: removed `width` - List: removed `maxWidth` (root) and `minWidth` (List.Label) - Flex: removed `width="full"` - Dialog.Content: removed `width` - AlertDialog.Content: removed `width` Consumers should migrate these to `style`/`className` (or a sized wrapper). Co-Authored-By: Claude Opus 4.8 (1M context) --- apps/www/src/app/docs/[[...slug]]/page.tsx | 9 ++++-- apps/www/src/app/examples/page.tsx | 8 ++---- apps/www/src/app/not-found.tsx | 3 +- apps/www/src/components/datatable-demo.tsx | 7 ++--- apps/www/src/components/dataview-demo.tsx | 24 ++++++++-------- apps/www/src/components/docs/footer.tsx | 2 +- apps/www/src/components/docs/search.tsx | 2 +- apps/www/src/components/docs/sidebar.tsx | 7 ++++- .../playground/alert-dialog-examples.tsx | 4 +-- .../playground/callout-examples.tsx | 2 +- .../components/playground/dialog-examples.tsx | 4 +-- .../components/playground/list-examples.tsx | 6 ++-- .../playground/text-area-examples.tsx | 5 +++- .../theme-customiser/theme-customiser.tsx | 2 +- .../docs/components/alert-dialog/props.ts | 3 -- .../content/docs/components/button/props.ts | 6 ---- .../content/docs/components/callout/demo.ts | 17 ++++------- .../content/docs/components/callout/index.mdx | 7 ----- .../content/docs/components/callout/props.ts | 9 ++---- .../content/docs/components/dialog/demo.ts | 14 +++++----- .../content/docs/components/dialog/props.ts | 3 -- .../src/content/docs/components/flex/props.ts | 5 ---- .../src/content/docs/components/list/demo.ts | 10 +++---- .../src/content/docs/components/list/props.ts | 6 ---- .../src/content/docs/components/radio/demo.ts | 2 +- .../content/docs/components/sidepanel/demo.ts | 18 ++++++------ .../content/docs/components/textarea/demo.ts | 12 -------- .../docs/components/textarea/index.mdx | 7 ----- .../content/docs/components/textarea/props.ts | 6 ---- .../alert-dialog/alert-dialog-content.tsx | 4 +-- .../raystack/components/button/button.tsx | 8 +----- .../callout/__tests__/callout.test.tsx | 18 +----------- .../raystack/components/callout/callout.tsx | 21 ++++++-------- .../components/dialog/dialog-content.tsx | 4 +-- .../components/flex/__tests__/flex.test.tsx | 14 ---------- .../raystack/components/flex/flex.module.css | 4 --- packages/raystack/components/flex/flex.tsx | 9 ++---- .../components/input/__tests__/input.test.tsx | 18 ------------ packages/raystack/components/input/input.tsx | 3 -- .../components/list/__tests__/list.test.tsx | 28 +------------------ packages/raystack/components/list/list.tsx | 26 +++-------------- .../text-area/__tests__/text-area.test.tsx | 12 -------- .../components/text-area/text-area.tsx | 4 +-- .../raystack/components/toast/toast-root.tsx | 2 +- packages/raystack/figma/callout.figma.tsx | 5 +++- 45 files changed, 102 insertions(+), 288 deletions(-) diff --git a/apps/www/src/app/docs/[[...slug]]/page.tsx b/apps/www/src/app/docs/[[...slug]]/page.tsx index 5a43853e4..9c9b5463a 100644 --- a/apps/www/src/app/docs/[[...slug]]/page.tsx +++ b/apps/www/src/app/docs/[[...slug]]/page.tsx @@ -34,8 +34,13 @@ export default async function Page(props: PageProps<'/docs/[[...slug]]'>) { pageTree={docs.pageTree} source={page.data.source} /> - - + + { + Some important message in the footer } @@ -1570,7 +1570,7 @@ const Page = () => { - + Dialog Title @@ -1657,7 +1657,6 @@ const Page = () => { } - width='100%' /> @@ -1731,7 +1730,7 @@ const Page = () => { open={nestedDialogOpen} onOpenChange={setNestedDialogOpen} > - + This is the nested dialog content. { } - width='100%' /> diff --git a/apps/www/src/app/not-found.tsx b/apps/www/src/app/not-found.tsx index fedbdd91e..2f531616d 100644 --- a/apps/www/src/app/not-found.tsx +++ b/apps/www/src/app/not-found.tsx @@ -6,8 +6,7 @@ export default function NotFound() { align='center' justify='center' gap={9} - width='full' - style={{ height: '100vh' }} + style={{ width: '100%', height: '100vh' }} > 404 diff --git a/apps/www/src/components/datatable-demo.tsx b/apps/www/src/components/datatable-demo.tsx index 1eccfd3c5..c2bcf7e9d 100644 --- a/apps/www/src/components/datatable-demo.tsx +++ b/apps/www/src/components/datatable-demo.tsx @@ -110,7 +110,7 @@ export const columns: DataTableColumnDef[] = [ const DataTableDemo = () => { const data = useMemo(() => generateData(100), []); return ( - +
{ const data = useMemo(() => generateData(1000), []); return ( - +
{ +
@@ -202,7 +202,7 @@ export function DataViewTableDemo() { export function DataViewListDemo() { return ( - +
@@ -217,7 +217,7 @@ export function DataViewListDemo() { export function DataViewSearchDemo() { return ( - +
@@ -242,7 +242,7 @@ export function DataViewMultiViewDemo() { [] ); return ( - +
+
+ @@ -386,7 +386,7 @@ function generatePeople(count: number): Person[] { export function DataViewVirtualizedDemo() { const data = useMemo(() => generatePeople(1000), []); return ( - +
@@ -414,7 +414,7 @@ export function DataViewGroupingDemo() { // so the sticky group header visibly swaps as the user moves between teams. const groupedPeople = useMemo(() => generatePeople(60), []); return ( - +
generatePeople(1500), []); return ( - +
+