Skip to content

Commit

Permalink
Clean up further
Browse files Browse the repository at this point in the history
  • Loading branch information
TylerJDev committed Nov 13, 2023
1 parent 8dbe325 commit cda5fc7
Show file tree
Hide file tree
Showing 6 changed files with 3 additions and 22 deletions.
2 changes: 1 addition & 1 deletion .changeset/lucky-gifts-rule.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
'@primer/react': minor
---

Added `PageLayout.PaneHeading` slot to `PageLayout` which utilizes `<Heading>` as its base.
Adjusts position of children within `PageLayout.Pane` to live above adjustable resize form.
4 changes: 2 additions & 2 deletions docs/content/SplitPageLayout.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -237,13 +237,13 @@ If you need a more flexible layout component, consider using the [PageLayout](/P
</Box>
```

### With pane heading
### With heading

```jsx live drafts
<Box sx={{height: 320, overflowY: 'auto', border: '1px solid', borderColor: 'border.default'}}>
<SplitPageLayout>
<SplitPageLayout.Pane>
<SplitPageLayout.PaneHeading as="h2">Pane Heading</SplitPageLayout.PaneHeading>
<Heading as="h2">Pane Heading</Heading>
<Placeholder label="Pane" height={120} />
</SplitPageLayout.Pane>
<SplitPageLayout.Content>
Expand Down
2 changes: 0 additions & 2 deletions src/PageLayout/PageLayout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -517,8 +517,6 @@ export type PageLayoutPaneProps = {
resizable?: boolean
widthStorageKey?: string
padding?: keyof typeof SPACING_MAP
heading?: string
headingLevel?: 'h1' | 'h2' | 'h3' | 'h4' | 'h5' | 'h6'
divider?: 'none' | 'line' | ResponsiveValue<'none' | 'line', 'none' | 'line' | 'filled'>
/**
* @deprecated Use the `divider` prop with a responsive value instead.
Expand Down
15 changes: 0 additions & 15 deletions src/SplitPageLayout/SplitPageLayout.docs.json
Original file line number Diff line number Diff line change
Expand Up @@ -154,21 +154,6 @@
"type": "SystemStyleObject"
}
]
},
{
"name": "SplitPageLayout.PaneHeading",
"props": [
{
"name": "as",
"type": "'h1' | 'h2' | 'h3' | 'h4' | 'h5' | 'h6'",
"required": true,
"defaultValue": ""
},
{
"name": "sx",
"type": "SystemStyleObject"
}
]
}
]
}
1 change: 0 additions & 1 deletion src/SplitPageLayout/SplitPageLayout.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,6 @@ const Template: Story = args => (
</SplitPageLayout.Content>
{args['Render pane?'] ? (
<SplitPageLayout.Pane
heading={args['Pane.heading']}
resizable={args['Pane.resizable']}
position={{
narrow: args['Pane.position.narrow'],
Expand Down
1 change: 0 additions & 1 deletion src/SplitPageLayout/SplitPageLayout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import {
PageLayoutFooterProps,
PageLayoutHeaderProps,
PageLayoutPaneProps,
PageLayoutPaneHeadingProps,
} from '../PageLayout'

// ----------------------------------------------------------------------------
Expand Down

0 comments on commit cda5fc7

Please sign in to comment.