refactor(NcActions): use provide/inject instead of $parent search

src/components/NcActionButton/NcActionButton.vue

@@ -372,6 +372,7 @@ import ChevronRightIcon from 'vue-material-design-icons/ChevronRight.vue'
 import ChevronLeftIcon from 'vue-material-design-icons/ChevronLeft.vue'
 import ActionTextMixin from '../../mixins/actionText.js'
 import { isRtl } from '../../utils/rtl.ts'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 /**
  * Button component to be used in Actions
@@ -389,7 +390,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionButtonGroup/NcActionButtonGroup.vue

@@ -89,6 +89,7 @@ export default {
 import { defineComponent } from 'vue'
 import { createElementId } from '../../utils/createElementId.ts'
 import { t } from '../../l10n.js'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 /**
  * A wrapper for allowing inlining NcAction components within the action menu
@@ -98,7 +99,7 @@ export default defineComponent({
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionCaption/NcActionCaption.vue

@@ -21,12 +21,14 @@ This component is made to be used inside of the [NcActions](#NcActions) componen
 </template>
 
 <script>
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
+
 export default {
 	name: 'NcActionCaption',
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionCheckbox/NcActionCheckbox.vue

@@ -40,6 +40,7 @@ This component is made to be used inside of the [NcActions](#NcActions) componen
 <script>
 import ActionGlobalMixin from '../../mixins/actionGlobal.js'
 import { createElementId } from '../../utils/createElementId.ts'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 export default {
 	name: 'NcActionCheckbox',
@@ -48,7 +49,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionLink/NcActionLink.vue

@@ -109,6 +109,7 @@ export default {
 
 <script>
 import ActionTextMixin from '../../mixins/actionText.js'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 export default {
 	name: 'NcActionLink',
@@ -117,7 +118,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionRadio/NcActionRadio.vue

@@ -69,6 +69,7 @@ So that only one of each name set can be selected at the same time.
 import { useModel } from 'vue'
 import { createElementId } from '../../utils/createElementId.ts'
 import ActionGlobalMixin from '../../mixins/actionGlobal.js'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 export default {
 	name: 'NcActionRadio',
@@ -77,7 +78,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionRouter/NcActionRouter.vue

@@ -50,6 +50,7 @@
 
 <script>
 import ActionTextMixin from '../../mixins/actionText.js'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 export default {
 	name: 'NcActionRouter',
@@ -58,7 +59,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActionText/NcActionText.vue

@@ -46,6 +46,7 @@
 
 <script>
 import ActionTextMixin from '../../mixins/actionText.js'
+import { NC_ACTIONS_IS_SEMANTIC_MENU } from '../NcActions/useNcActions.ts'
 
 export default {
 	name: 'NcActionText',
@@ -54,7 +55,7 @@ export default {
 
 	inject: {
 		isInSemanticMenu: {
-			from: 'NcActions:isSemanticMenu',
+			from: NC_ACTIONS_IS_SEMANTIC_MENU,
 			default: false,
 		},
 	},

src/components/NcActions/NcActions.vue

@@ -928,6 +928,7 @@
 import IconDotsHorizontal from 'vue-material-design-icons/DotsHorizontal.vue'
 import NcButton from '../NcButton/index.ts'
 import NcPopover from '../NcPopover/index.js'
+import { NC_ACTIONS_CLOSE_MENU, NC_ACTIONS_IS_SEMANTIC_MENU } from './useNcActions.ts'
 
 const focusableSelector = '.focusable'
 
@@ -958,7 +959,8 @@
 			 * Provide the role for NcAction* components in the NcActions content.
 			 * @type {import('vue').ComputedRef<boolean>}
 			 */
-			'NcActions:isSemanticMenu': computed(() => this.actionsMenuSemanticType === 'menu'),
+			[NC_ACTIONS_IS_SEMANTIC_MENU]: computed(() => this.actionsMenuSemanticType === 'menu'),
+			[NC_ACTIONS_CLOSE_MENU]: this.closeMenu,
 		}
 	},
 
@@ -1408,7 +1410,7 @@
 		},
 
 		/**
 		 * @return {NodeListOf<HTMLElement>}
 		 */
 		getFocusableMenuItemElements() {
 			return this.$refs.menu.querySelectorAll(focusableSelector)

src/components/NcActions/useNcActions.ts

@@ -0,0 +1,11 @@
+/**
+ * SPDX-FileCopyrightText: 2025 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+import type { InjectionKey, ComputedRef } from 'vue'
+
+export const NC_ACTIONS_IS_SEMANTIC_MENU: InjectionKey<ComputedRef<boolean>> = Symbol.for('NcActions:isSemanticMenu')
+export const NC_ACTIONS_CLOSE_MENU: InjectionKey<(returnFocus: boolean) => void> = Symbol.for('NcActions:closeMenu')
+
+// TODO: once NcActions migrated from mixins - add composables to created and use NcAction provider/context

src/mixins/actionText.js

@@ -4,7 +4,7 @@
  */
 
 import actionGlobal from './actionGlobal.js'
-import { getParent } from '../utils/getParent.ts'
+import { NC_ACTIONS_CLOSE_MENU } from '../components/NcActions/useNcActions.ts'
 
 export default {
 	mixins: [actionGlobal],
@@ -55,6 +55,12 @@ export default {
 		},
 	},
 
+	inject: {
+		closeMenu: {
+			from: NC_ACTIONS_CLOSE_MENU,
+		},
+	},
+
 	emits: [
 		'click',
 	],
@@ -83,10 +89,7 @@ export default {
 			this.$emit('click', event)
 
 			if (this.closeAfterClick) {
-				const parent = getParent(this, 'NcActions')
-				if (parent && parent.closeMenu) {
-					parent.closeMenu(false)
-				}
+				this.closeMenu(false)
 			}
 		},
 	},