fix(menu): can't override submenu dropdown's trigger by `dropdown-p…

…rops`(#4148) * Fix dropdown trigger getting overriden in NMenu * Add changelog entry Co-authored-by: Albert Geantă <[email protected]> Co-authored-by: 07akioni <[email protected]>
tusen-ai · Dec 11, 2022 · 3410434 · 3410434
1 parent eb340ae
commit 3410434
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.en-US.md b/CHANGELOG.en-US.md
@@ -9,8 +9,9 @@
 
 ### Fixes
 
-- Fix 'n-image' preview toolbar error, close [#4157](https://github.com/tusen-ai/naive-ui/issues/4157).
-- Fix 'n-tree' cannot access 'mergedFilterRef' before initialization error, close [#4134](https://github.com/tusen-ai/naive-ui/issues/4134).
+- Fix `n-image` preview toolbar error, closes [#4157](https://github.com/tusen-ai/naive-ui/issues/4157).
+- Fix `n-tree` cannot access `mergedFilterRef` before initialization error, closes [#4134](https://github.com/tusen-ai/naive-ui/issues/4134).
+- Fix `n-menu` can't override submenu dropdown's `trigger` by `dropdown-props`, closes [#4147](https://github.com/tusen-ai/naive-ui/issues/4147).
 
 ## 2.34.2
 

diff --git a/CHANGELOG.zh-CN.md b/CHANGELOG.zh-CN.md
@@ -11,6 +11,7 @@
 
 - 修复 `n-image` 的预览工具栏报错，关闭 [#4157](https://github.com/tusen-ai/naive-ui/issues/4157)
 - 修复 `n-tree` 的`cannot access 'mergedFilterRef' before initialization`报错，关闭 [#4134](https://github.com/tusen-ai/naive-ui/issues/4134)
+- 修复 `n-menu` 无法通过 `dropdown-props` 覆盖子菜单 dropdown 的 `trigger`，关闭 [#4147](https://github.com/tusen-ai/naive-ui/issues/4147)
 
 ## 2.34.2
 

diff --git a/src/menu/src/Submenu.tsx b/src/menu/src/Submenu.tsx
@@ -179,6 +179,7 @@ export const NSubmenu = defineComponent({
     return this.root ? (
       <NDropdown
         size="large"
+        trigger="hover"
         {...this.menuProps?.dropdownProps}
         themeOverrides={this.mergedTheme.peerOverrides.Dropdown}
         theme={this.mergedTheme.peers.Dropdown}
@@ -187,7 +188,6 @@ export const NSubmenu = defineComponent({
           optionIconSizeLarge: '18px'
         }}
         value={this.mergedValue}
-        trigger="hover"
         disabled={!this.dropdownEnabled}
         placement={this.dropdownPlacement}
         keyField={this.menuProps.keyField}