Merge pull request #21417 from nextcloud/bugfix/noid/modal-support-for-vue-apps

Fix modal support for vue apps

core/src/OC/dialogs.js

@@ -107,7 +107,7 @@ const Dialogs = {
 			'none',
 			buttons,
 			callback,
-			modal
+			modal === undefined ? true : modal
 		)
 	},
 	/**
@@ -1245,7 +1245,7 @@ const Dialogs = {
 			dir: '',
 			name: '' // Ugly but works ;)
 		}, { escapeFunction: null }).prependTo(this.$dirTree)
-		
+
 	},
 	/**
 	 * handle selection made in the tree list

core/src/jquery/css/jquery.ocdialog.scss

@@ -72,6 +72,10 @@
 	height: 100%;
 }
 
+body.dark .oc-dialog-dim {
+	opacity: .8;
+}
+
 .oc-dialog-content {
 	width: 100%;
 	max-width: 550px;

core/src/jquery/ocdialog.js

@@ -210,9 +210,14 @@ $.widget('oc.ocdialog', {
 		}
 
 		const self = this
+		let contentDiv = $('#content')
+		if (contentDiv.length === 0) {
+			// nextcloud-vue compatibility
+			contentDiv = $('.content')
+		}
 		this.overlay = $('<div>')
 			.addClass('oc-dialog-dim')
-			.appendTo($('#content'))
+			.appendTo(contentDiv)
 		this.overlay.on('click keydown keyup', function(event) {
 			if (event.target !== self.$dialog.get(0) && self.$dialog.find($(event.target)).length === 0) {
 				event.preventDefault()