Skip to content

Commit

Permalink
fix(html): remove k-header class from tabstrip
Browse files Browse the repository at this point in the history
The k-header class does not add any styles and seems to be obsolete
  • Loading branch information
TeyaVes committed Dec 2, 2024
1 parent 40d95f3 commit 9642807
Show file tree
Hide file tree
Showing 13 changed files with 21 additions and 26 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ export const ColumnMenuTabbed = (props) => (
<ColumnMenu
view="tabbed"
children={
<TabStripNormal header={false} className="k-pos-relative"
<TabStripNormal className="k-pos-relative"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand Down
6 changes: 3 additions & 3 deletions packages/html/src/column-menu/tests/column-menu-tabbed.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export default () =>(

<section>
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem icon="filter" />
Expand Down Expand Up @@ -56,7 +56,7 @@ export default () =>(

<section>
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand All @@ -73,7 +73,7 @@ export default () =>(

<section>
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-pos-relative"
<TabStripNormal className="k-pos-relative"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand Down
4 changes: 2 additions & 2 deletions packages/html/src/editor/tests/editor-find-replace.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default () =>(

<section>
<WindowNormal title="Find and Replace" actions={[ 'x' ]}>
<TabStripNormal header={false} className="k-editor-find-replace"
<TabStripNormal className="k-editor-find-replace"
tabStripItems={
<>
<TabStripItem className="k-tab-on-top" value="Find" active />
Expand Down Expand Up @@ -76,7 +76,7 @@ export default () =>(

<section>
<WindowNormal className="k-editor-window" title="Find and Replace" actions={[ 'x' ]}>
<TabStripNormal header={false} className="k-editor-find-replace"
<TabStripNormal className="k-editor-find-replace"
tabStripItems={
<>
<TabStripItem value="Find" />
Expand Down
4 changes: 2 additions & 2 deletions packages/html/src/editor/tests/editor-table-wizard.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ export default () =>(
<Button>Cancel</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first active value="General"/>
Expand Down Expand Up @@ -138,7 +138,7 @@ export default () =>(
<Button>Cancel</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="General"/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export default () =>(
<Button icon="trash" fillMode="flat" themeColor="primary">Delete</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first active value="General" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ export default () =>(
<Button icon="trash" fillMode="flat" themeColor="primary">Delete</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="General" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export default () =>(
<Button icon="trash" fillMode="flat" themeColor="primary">Delete</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="General" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export default () =>(
<Button icon="trash" fillMode="flat" themeColor="primary">Delete</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="General" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export default () =>(
<Button icon="trash" fillMode="flat" themeColor="primary">Delete</Button>
</>
}>
<TabStripNormal header={false}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="General" />
Expand Down
10 changes: 5 additions & 5 deletions packages/html/src/grid/tests/grid-column-menu-tabbed.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export default () =>(
<section>
<Popup className="k-grid-columnmenu-popup">
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand All @@ -49,7 +49,7 @@ export default () =>(
<section>
<Popup className="k-grid-columnmenu-popup">
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem icon="filter" />
Expand Down Expand Up @@ -82,7 +82,7 @@ export default () =>(
<section>
<Popup className="k-grid-columnmenu-popup">
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem icon="filter" />
Expand Down Expand Up @@ -129,7 +129,7 @@ export default () =>(
<section>
<Popup className="k-grid-columnmenu-popup">
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-overflow-hidden"
<TabStripNormal className="k-overflow-hidden"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand All @@ -148,7 +148,7 @@ export default () =>(
<section>
<Popup className="k-grid-columnmenu-popup">
<ColumnMenuTabbed>
<TabStripNormal header={false} className="k-pos-relative"
<TabStripNormal className="k-pos-relative"
tabStripItems={
<>
<TabStripItem active icon="filter" />
Expand Down
3 changes: 1 addition & 2 deletions packages/html/src/spreadsheet/spreadsheet-sheets-bar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,7 @@ export const SpreadsheetSheetsBar = (
tabStripItems={tabStripItems}
position="bottom"
scrollable
scrollButtonsPosition="end"
header={false} />
scrollButtonsPosition="end" />
</div>
);
};
Expand Down
4 changes: 2 additions & 2 deletions packages/html/src/spreadsheet/tests/spreadsheet-print.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export default () =>(
<Button themeColor="primary">Save</Button>
</>
}>
<TabStripNormal header={null}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first active value="Print" />
Expand Down Expand Up @@ -138,7 +138,7 @@ export default () =>(
<Button themeColor="primary">Save</Button>
</>
}>
<TabStripNormal header={null}
<TabStripNormal
tabStripItems={
<>
<TabStripItem first value="Print" />
Expand Down
4 changes: 0 additions & 4 deletions packages/html/src/tabstrip/tabstrip.spec.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,13 @@ export type KendoTabStripProps = {
tabStripItems?: React.JSX.Element | React.JSX.Element[];
scrollable?: boolean;
scrollButtonsPosition?: "around" | "start" | "end";
header?: boolean | null;
dir?: "rtl" | "ltr";
};

const defaultOptions = {
position: "top",
tabAlignment: "start",
scrollButtonsPosition: "around",
header: true
};

export const TabStrip = (
Expand All @@ -35,7 +33,6 @@ export const TabStrip = (
position = defaultOptions.position,
tabAlignment = defaultOptions.tabAlignment,
scrollButtonsPosition = defaultOptions.scrollButtonsPosition,
header = defaultOptions.header,
dir,
...other
} = props;
Expand Down Expand Up @@ -72,7 +69,6 @@ export const TabStrip = (
{
[`${TABSTRIP_CLASSNAME}-${position}`]: position,
[`${TABSTRIP_CLASSNAME}-scrollable`]: scrollable,
["k-header"]: header,
},
)}>
{position === "bottom" && children}
Expand Down

0 comments on commit 9642807

Please sign in to comment.