feat: Inline editing - usergroup

Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>

Author: enjeck

src/shared/components/ncTable/partials/TableCellUsergroup.vue

@@ -3,26 +3,70 @@
   - SPDX-License-Identifier: AGPL-3.0-or-later
 -->
 <template>
-	<!-- eslint-disable-next-line vue/no-v-html -->
-	<div v-if="value" class="table-cell-usergroup">
-		<div v-for="item in value" :key="item.id" class="inline usergroup-entry">
-			<NcUserBubble :user="item.id" :avatar-image="getAvatarImage(item)" :is-no-user="!isUser(item)" :display-name="item.displayName ?? item.id" :show-user-status="isUser(item) && column.showUserStatus" :size="column.showUserStatus ? 34 : 20" :primary="isCurrentUser(item)" />
+	<div class="cell-usergroup">
+		<div v-if="!isEditing" class="non-edit-mode" @click="startEditing">
+			<div v-if="value" class="table-cell-usergroup">
+				<div v-for="item in value" :key="item.id" class="inline usergroup-entry">
+					<NcUserBubble :user="item.id" :avatar-image="getAvatarImage(item)" :is-no-user="!isUser(item)" :display-name="item.displayName ?? item.id" :show-user-status="isUser(item) && column.showUserStatus" :size="column.showUserStatus ? 34 : 20" :primary="isCurrentUser(item)" />
+				</div>
+			</div>
+		</div>
+		<div v-else
+			ref="editingContainer"
+			class="edit-mode"
+			tabindex="0"
+			@keydown.enter="saveChanges"
+			@keydown.escape="cancelEdit">
+			<NcSelect v-model="editValue"
+				style="width: 100%;"
+				:loading="loading || localLoading"
+				:options="options"
+				:placeholder="getPlaceholder()"
+				:searchable="true"
+				:get-option-key="(option) => option.key"
+				label="displayName"
+				:aria-label-combobox="getPlaceholder()"
+				:user-select="true"
+				:close-on-select="false"
+				:multiple="column.usergroupMultipleItems"
+				data-cy="usergroupCellSelect"
+				@search="asyncFind">
+				<template #noResult>
+					{{ noResultText }}
+				</template>
+			</NcSelect>
+			<div v-if="localLoading" class="loading-indicator">
+				<div class="icon-loading-small icon-loading-inline" />
+			</div>
 		</div>
 	</div>
 </template>
 
 <script>
 import { getCurrentUser } from '@nextcloud/auth'
-import { NcUserBubble } from '@nextcloud/vue'
+import { NcUserBubble, NcSelect } from '@nextcloud/vue'
 import { USERGROUP_TYPE } from '../../../constants.ts'
+import cellEditMixin from '../mixins/cellEditMixin.js'
+import searchUserGroup from '../../../mixins/searchUserGroup.js'
+import ShareTypes from '../../../mixins/shareTypesMixin.js'
+import { translate as t } from '@nextcloud/l10n'
 
 const currentUser = getCurrentUser()
 
 export default {
 	name: 'TableCellUsergroup',
+
 	components: {
 		NcUserBubble,
+		NcSelect,
 	},
+
+	mixins: [
+		cellEditMixin,
+		searchUserGroup,
+		ShareTypes,
+	],
+
 	props: {
 		column: {
 			type: Object,
@@ -37,6 +81,16 @@ export default {
 			default: () => [],
 		},
 	},
+
+	data() {
+		return {
+			editValue: [],
+			selectUsers: this.column?.usergroupSelectUsers ?? true,
+			selectGroups: this.column?.usergroupSelectGroups ?? false,
+			selectCircles: false,
+		}
+	},
+
 	computed: {
 		isCurrentUser() {
 			return (item) => this.isUser(item) && item.id === currentUser?.uid
@@ -45,7 +99,32 @@ export default {
 			return (item) => item.type === USERGROUP_TYPE.USER
 		},
 	},
+
+	watch: {
+		isEditing(isEditing) {
+			if (isEditing) {
+				this.editValue = this.value ? [...this.value] : []
+
+				// Use a small delay to prevent the same click event that triggered editing
+				// from immediately triggering the click outside handler
+				this.$nextTick(() => {
+					setTimeout(() => {
+						document.addEventListener('click', this.handleClickOutside)
+					}, 10)
+				})
+			} else {
+				document.removeEventListener('click', this.handleClickOutside)
+			}
+		},
+	},
+
+	created() {
+		this.selectCircles = this.isCirclesEnabled ? this.column?.usergroupSelectTeams ?? false : false
+	},
+
 	methods: {
+		t,
+
 		getAvatarImage(item) {
 			if (item.type === USERGROUP_TYPE.GROUP) {
 				return 'icon-group'
@@ -55,18 +134,74 @@ export default {
 			}
 			return ''
 		},
+
+		formatResult(autocompleteResult) {
+			return {
+				id: autocompleteResult.id,
+				type: this.getType(autocompleteResult.source),
+				key: autocompleteResult.source + '-' + autocompleteResult.id,
+				displayName: autocompleteResult.label,
+			}
+		},
+
+		filterOutUnwantedItems(list) {
+			return list
+		},
+
+		async saveChanges() {
+			if (this.localLoading) {
+				return
+			}
+
+			let newValue = !Array.isArray(this.editValue) ? [this.editValue] : this.editValue
+			// If the column does not allow multiple items, limit to one item
+			// in case we switched from multiple to single selection
+			if (!this.column.usergroupMultipleItems) {
+				newValue = newValue.slice(0, 1)
+			}
+			const success = await this.updateCellValue(newValue)
+
+			if (!success) {
+				this.cancelEdit()
+			}
+
+			this.localLoading = false
+			this.isEditing = false
+		},
+
+		handleClickOutside(event) {
+			if (this.$refs.editingContainer && !this.$refs.editingContainer.contains(event.target)) {
+				this.saveChanges()
+			}
+		},
 	},
 }
 </script>
 
 <style lang="scss" scoped>
-	.table-cell-usergroup {
-		display: flex;
-		flex-wrap: wrap;
-		padding: 10px;
+.cell-usergroup {
+	width: 100%;
+
+	.non-edit-mode {
+		cursor: pointer;
 	}
+}
+
+.table-cell-usergroup {
+	display: flex;
+	flex-wrap: wrap;
+	padding: 10px;
+}
+
+.usergroup-entry {
+	padding-right: 10px;
+}
 
-	.usergroup-entry {
-		padding-right: 10px;
+.edit-mode {
+	padding: 10px 0;
+
+	.icon-loading-inline {
+		margin-left: 4px;
 	}
+}
 </style>

src/shared/components/ncTable/partials/TableRow.vue

@@ -112,7 +112,6 @@ export default {
 		nonInlineEditableColumnTypes() {
 			return [
 				ColumnTypes.TextRich,
-				ColumnTypes.Usergroup,
 				ColumnTypes.SelectionMulti,
 				ColumnTypes.TextLink,
 				ColumnTypes.NumberStars,