Skip to content

Commit 5ee15a8

Browse files
Copilothotlong
andcommitted
fix: address code review feedback - remove redundant sm: prefix, improve JSDoc
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent 893e11b commit 5ee15a8

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

packages/components/src/renderers/action/action-bar.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export interface ActionBarSchema {
4646
location?: ActionLocation;
4747
/** Maximum visible inline actions before overflow into "More" menu (default: 3) */
4848
maxVisible?: number;
49-
/** Maximum visible inline actions on mobile (default: 1) */
49+
/** Maximum visible inline actions on mobile devices (default: 1). Desktop uses maxVisible instead. */
5050
mobileMaxVisible?: number;
5151
/** Visibility condition expression */
5252
visible?: string;

packages/plugin-detail/src/DetailSection.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -195,9 +195,9 @@ export const DetailSection: React.FC<DetailSectionProps> = ({
195195
className={cn(
196196
"grid gap-3 sm:gap-4",
197197
effectiveColumns === 1 ? "grid-cols-1" :
198-
effectiveColumns === 2 ? "grid-cols-2 sm:grid-cols-2" :
199-
effectiveColumns === 3 ? "grid-cols-2 sm:grid-cols-2 lg:grid-cols-3" :
200-
"grid-cols-2 sm:grid-cols-2 lg:grid-cols-3"
198+
effectiveColumns === 2 ? "grid-cols-2" :
199+
effectiveColumns === 3 ? "grid-cols-2 lg:grid-cols-3" :
200+
"grid-cols-2 lg:grid-cols-3"
201201
)}
202202
>
203203
{layoutFields.map(renderField)}

0 commit comments

Comments
 (0)