Skip to content

Commit 28142e6

Browse files
author
Eric Olkowski
committed
Updated based on Matts PR
1 parent 1108771 commit 28142e6

3 files changed

Lines changed: 3 additions & 7 deletions

File tree

packages/react-core/src/components/Toolbar/Toolbar.tsx

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@ export interface ToolbarProps extends React.HTMLProps<HTMLDivElement>, OUIAProps
3838
isFullHeight?: boolean;
3939
/** Flag indicating the toolbar is static */
4040
isStatic?: boolean;
41-
/** Flag indicating the toolbar should use the Page insets */
42-
usePageInsets?: boolean;
4341
/** Flag indicating the toolbar should stick to the top of its container */
4442
isSticky?: boolean;
4543
/** Insets at various breakpoints. */
@@ -140,7 +138,6 @@ class Toolbar extends React.Component<ToolbarProps, ToolbarState> {
140138
isFullHeight,
141139
isStatic,
142140
inset,
143-
usePageInsets,
144141
isSticky,
145142
ouiaId,
146143
numberOfFiltersText,
@@ -164,7 +161,6 @@ class Toolbar extends React.Component<ToolbarProps, ToolbarState> {
164161
styles.toolbar,
165162
isFullHeight && styles.modifiers.fullHeight,
166163
isStatic && styles.modifiers.static,
167-
usePageInsets && styles.modifiers.pageInsets,
168164
isSticky && styles.modifiers.sticky,
169165
formatBreakpointMods(inset, styles, '', getBreakpoint(width)),
170166
colorVariant === 'primary' && styles.modifiers.primary,

packages/react-core/src/components/Toolbar/ToolbarChipGroupContent.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ class ToolbarChipGroupContent extends React.Component<ToolbarChipGroupContentPro
8585
{...(collapseListedFilters && { 'aria-hidden': true })}
8686
/>
8787
{(showNumberOfFilters || showDefaultClearFilter || customChipGroupContent) && (
88-
<ToolbarGroup>
88+
<ToolbarGroup variant="action-group-inline">
89+
{showNumberOfFilters && <ToolbarItem>{numberOfFiltersText(numberOfFilters)}</ToolbarItem>}
8990
{showDefaultClearFilter && (
9091
<ToolbarItem>
9192
<Button variant="link" onClick={clearChipGroups} isInline>
@@ -94,7 +95,6 @@ class ToolbarChipGroupContent extends React.Component<ToolbarChipGroupContentPro
9495
</ToolbarItem>
9596
)}
9697
{customChipGroupContent && customChipGroupContent}
97-
{showNumberOfFilters && <ToolbarItem>{numberOfFiltersText(numberOfFilters)}</ToolbarItem>}
9898
</ToolbarGroup>
9999
)}
100100
</div>

packages/react-core/src/components/Toolbar/examples/ToolbarGroups.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ export const ToolbarGroups: React.FunctionComponent = () => {
189189
<React.Fragment>
190190
<ToolbarGroup variant="filter-group">{filterGroupItems}</ToolbarGroup>
191191
<ToolbarGroup variant="action-group-plain">{iconButtonGroupItems}</ToolbarGroup>
192-
<ToolbarGroup>{buttonGroupItems}</ToolbarGroup>
192+
<ToolbarGroup variant="action-group">{buttonGroupItems}</ToolbarGroup>
193193
</React.Fragment>
194194
);
195195

0 commit comments

Comments
 (0)