Merge pull request #6808 from nextcloud-libraries/backport/6807/stable8

[stable8] feat(NcPopover): add new `noFocusTrap` prop

Author: susnux

CHANGELOG.md

@@ -5,6 +5,13 @@
 
 # Changelog
 
+## [v8.26.0](https://github.com/nextcloud-libraries/nextcloud-vue/tree/v8.26.0) (UNRELEASED)
+
+### 📝 Notes
+#### NcPopover
+The `focusTrap` property is now deprecated and will be replaced with `noFocusTrap`,
+the reason behind this is to only have boolean properties with default value of `false` allowing shortcut props.
+
 ## [v8.25.1](https://github.com/nextcloud-libraries/nextcloud-vue/tree/v8.25.1) (2025-04-17)
 [Full Changelog](https://github.com/nextcloud-libraries/nextcloud-vue/compare/v8.25.0...v8.25.1)
 

src/components/NcPopover/NcPopover.vue

@@ -208,13 +208,25 @@ export default {
 			type: String,
 			default: '',
 		},
+
 		/**
 		 * Enable popover focus trap
+		 *
+		 * @deprecated use noFocusTrap instead
 		 */
 		focusTrap: {
 			type: Boolean,
 			default: true,
 		},
+
+		/**
+		 * Disable the popover focus trap.
+		 */
+		noFocusTrap: {
+			type: Boolean,
+			default: false,
+		},
+
 		/**
 		 * Set element to return focus to after focus trap deactivation
 		 *
@@ -310,7 +322,7 @@ export default {
 		async useFocusTrap() {
 			await this.$nextTick()
 
-			if (!this.focusTrap) {
+			if (this.noFocusTrap || !this.focusTrap) {
 				return
 			}