Skip to content

Commit 8c88fcd

Browse files
committed
Remove deprecated HdsMenuPrimitive component (#3224)
1 parent 588ad08 commit 8c88fcd

File tree

26 files changed

+12
-600
lines changed

26 files changed

+12
-600
lines changed

.changeset/legal-swans-occur.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@hashicorp/design-system-components": major
3+
---
4+
5+
Removed the deprecated `MenuPrimitive` component

packages/components/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,6 @@
305305
"./components/hds/layout/grid/item.js": "./dist/_app_/components/hds/layout/grid/item.js",
306306
"./components/hds/link/inline.js": "./dist/_app_/components/hds/link/inline.js",
307307
"./components/hds/link/standalone.js": "./dist/_app_/components/hds/link/standalone.js",
308-
"./components/hds/menu-primitive.js": "./dist/_app_/components/hds/menu-primitive.js",
309308
"./components/hds/modal/body.js": "./dist/_app_/components/hds/modal/body.js",
310309
"./components/hds/modal/footer.js": "./dist/_app_/components/hds/modal/footer.js",
311310
"./components/hds/modal/header.js": "./dist/_app_/components/hds/modal/header.js",

packages/components/src/components.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -390,8 +390,5 @@ export { default as HdsDismissButton } from './components/hds/dismiss-button/ind
390390
// Interactive
391391
export { default as HdsInteractive } from './components/hds/interactive/index.ts';
392392

393-
// MenuPrimitive
394-
export { default as HdsMenuPrimitive } from './components/hds/menu-primitive/index.ts';
395-
396393
// PopoverPrimitive
397394
export { default as HdsPopoverPrimitive } from './components/hds/popover-primitive/index.ts';

packages/components/src/components/hds/dropdown/index.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import {
1515
} from './types.ts';
1616

1717
import type { ComponentLike } from '@glint/template';
18-
import type { MenuPrimitiveSignature } from '../menu-primitive';
18+
import type { HdsPopoverPrimitiveSignature } from '../popover-primitive/index.ts';
1919
import type { HdsDropdownFooterSignature } from './footer';
2020
import type { HdsDropdownHeaderSignature } from './header';
2121
import type { HdsDropdownListItemCheckboxSignature } from './list-item/checkbox';
@@ -39,15 +39,16 @@ export const POSITIONS: HdsDropdownPositions[] = Object.values(
3939
);
4040

4141
export interface HdsDropdownSignature {
42-
Args: MenuPrimitiveSignature['Args'] & {
42+
Args: {
4343
height?: string;
4444
isInline?: boolean;
45-
isOpen?: boolean;
45+
isOpen?: HdsPopoverPrimitiveSignature['Args']['isOpen'];
4646
listPosition?: HdsDropdownPositions;
4747
width?: string;
4848
enableCollisionDetection?: HdsAnchoredPositionOptions['enableCollisionDetection'];
4949
preserveContentInDom?: boolean;
5050
matchToggleWidth?: boolean;
51+
onClose?: HdsPopoverPrimitiveSignature['Args']['onClose'];
5152
};
5253
Blocks: {
5354
default: [
@@ -69,7 +70,7 @@ export interface HdsDropdownSignature {
6970
},
7071
];
7172
};
72-
Element: MenuPrimitiveSignature['Element'];
73+
Element: HTMLDivElement;
7374
}
7475

7576
export default class HdsDropdown extends Component<HdsDropdownSignature> {

packages/components/src/components/hds/menu-primitive/index.hbs

Lines changed: 0 additions & 25 deletions
This file was deleted.

packages/components/src/components/hds/menu-primitive/index.ts

Lines changed: 0 additions & 109 deletions
This file was deleted.

packages/components/src/styles/@hashicorp/design-system-components.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@
3939
@use "../components/icon-tile";
4040
@use "../components/layout"; // multiple components
4141
@use "../components/link"; // multiple components
42-
@use "../components/menu-primitive";
4342
@use "../components/modal";
4443
@use "../components/page-header";
4544
@use "../components/pagination";

packages/components/src/styles/components/dropdown.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ $hds-dropdown-toggle-border-radius: $hds-button-border-radius;
207207

208208
// LIST
209209
// UL ELEMENT
210-
// GOES INSIDE HDS::MenuPrimitive's :content block
210+
// GOES INSIDE HDS::PopoverPrimitive's popover element
211211

212212
.hds-dropdown__content {
213213
position: relative;

packages/components/src/styles/components/menu-primitive.scss

Lines changed: 0 additions & 13 deletions
This file was deleted.

packages/components/src/template-registry.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,6 @@ import type HdsLayoutGridComponent from './components/hds/layout/grid';
168168
import type HdsLayoutGridItemComponent from './components/hds/layout/grid/item';
169169
import type HdsLinkInlineComponent from './components/hds/link/inline';
170170
import type HdsLinkStandaloneComponent from './components/hds/link/standalone';
171-
import type HdsMenuPrimitiveComponent from './components/hds/menu-primitive';
172171
import type HdsModalBodyComponent from './components/hds/modal/body';
173172
import type HdsModalFooterComponent from './components/hds/modal/footer';
174173
import type HdsModalHeaderComponent from './components/hds/modal/header';
@@ -806,10 +805,6 @@ export default interface HdsComponentsRegistry {
806805
'Hds::Link::Standalone': typeof HdsLinkStandaloneComponent;
807806
'hds/link/standalone': typeof HdsLinkStandaloneComponent;
808807

809-
// MenuPrimitive
810-
'Hds::MenuPrimitive': typeof HdsMenuPrimitiveComponent;
811-
'hds/menu-primitive': typeof HdsMenuPrimitiveComponent;
812-
813808
// Modal
814809
'Hds::Modal': typeof HdsModalComponent;
815810
'hds/modal': typeof HdsModalComponent;

0 commit comments

Comments
 (0)