最新への追従とレイアウト微調整

packages/frontend/src/components/MkSortOrderEditor.vue

@@ -4,26 +4,22 @@ SPDX-License-Identifier: AGPL-3.0-only
 -->
 
 <template>
-<MkFolder :spacerMax="8" :spacerMin="8">
-	<template #icon><i class="ti ti-arrows-sort"></i></template>
-	<template #label>{{ i18n.ts._customEmojisManager._gridCommon.sortOrder }}</template>
-	<div :class="$style.sortOrderArea">
-		<div :class="$style.sortOrderAreaTags">
-			<MkTagItem
-				v-for="order in sortOrders"
-				:key="order.key"
-				:iconClass="order.direction === 'ASC' ? 'ti ti-arrow-up' : 'ti ti-arrow-down'"
-				:exButtonIconClass="'ti ti-x'"
-				:content="order.key"
-				@click="onToggleSortOrderButtonClicked(order)"
-				@exButtonClick="onRemoveSortOrderButtonClicked(order.key)"
-			/>
-		</div>
-		<MkButton :class="$style.sortOrderAddButton" @click="onAddSortOrderButtonClicked">
-			<span class="ti ti-plus"/>
-		</MkButton>
+<div :class="$style.sortOrderArea">
+	<div :class="$style.sortOrderAreaTags">
+		<MkTagItem
+			v-for="order in sortOrders"
+			:key="order.key"
+			:iconClass="order.direction === 'ASC' ? 'ti ti-arrow-up' : 'ti ti-arrow-down'"
+			:exButtonIconClass="'ti ti-x'"
+			:content="order.key"
+			@click="onToggleSortOrderButtonClicked(order)"
+			@exButtonClick="onRemoveSortOrderButtonClicked(order.key)"
+		/>
 	</div>
-</MkFolder>
+	<MkButton :class="$style.sortOrderAddButton" @click="onAddSortOrderButtonClicked">
+		<span class="ti ti-plus"/>
+	</MkButton>
+</div>
 </template>
 
 <script setup lang="ts">

packages/frontend/src/components/grid/MkDataCell.vue

@@ -5,6 +5,7 @@ SPDX-License-Identifier: AGPL-3.0-only
 
 <template>
 <div
+	v-if="cell.row.using"
 	ref="rootEl"
 	class="mk_grid_td"
 	:class="$style.cell"
@@ -26,7 +27,7 @@ SPDX-License-Identifier: AGPL-3.0-only
 			[needsContentCentering ? $style.center : {}],
 		]"
 	>
-		<div v-if="!editing" ref="contentAreaEl">
+		<div v-if="!editing" ref="contentAreaEl" :class="$style.contentArea">
 			<div :class="$style.content">
 				<div v-if="cellType === 'text'">
 					{{ cell.value }}
@@ -51,7 +52,7 @@ SPDX-License-Identifier: AGPL-3.0-only
 				</div>
 			</div>
 		</div>
-		<div v-else ref="inputAreaEl">
+		<div v-else ref="inputAreaEl" :class="$style.inputArea">
 			<input
 				v-if="cellType === 'text'"
 				type="text"
@@ -105,13 +106,15 @@ const props = defineProps<{
 	bus: GridEventEmitter,
 }>();
 
-const { cell, rowSetting, bus } = toRefs(props);
+const { cell, bus } = toRefs(props);
 
 const rootEl = shallowRef<InstanceType<typeof HTMLTableCellElement>>();
 const contentAreaEl = shallowRef<InstanceType<typeof HTMLDivElement>>();
 const inputAreaEl = shallowRef<InstanceType<typeof HTMLDivElement>>();
 
+/** 値が編集中かどうか */
 const editing = ref<boolean>(false);
+/** 編集中の値. {@link beginEditing}と{@link endEditing}内、および各inputタグやそのコールバックからの操作のみを想定する */
 const editingValue = ref<CellValue>(undefined);
 
 const cellWidth = computed(() => cell.value.column.width);
@@ -148,7 +151,8 @@ function onCellDoubleClick(ev: MouseEvent) {
 function onOutsideMouseDown(ev: MouseEvent) {
 	const isOutside = ev.target instanceof Node && !rootEl.value?.contains(ev.target);
 	if (isOutside || !equalCellAddress(cell.value.address, getCellAddress(ev.target as HTMLElement))) {
-		endEditing(true);
+		endEditing(true, editingValue.value);
+		editingValue.value = undefined;
 	}
 }
 
@@ -166,13 +170,15 @@ function onCellKeyDown(ev: KeyboardEvent) {
 	} else {
 		switch (ev.code) {
 			case 'Escape': {
-				endEditing(false);
+				endEditing(false, editingValue.value);
+				editingValue.value = undefined;
 				break;
 			}
 			case 'NumpadEnter':
 			case 'Enter': {
 				if (!ev.isComposing) {
-					endEditing(true);
+					endEditing(true, editingValue.value);
+					editingValue.value = undefined;
 				}
 			}
 		}
@@ -243,7 +249,7 @@ async function beginEditing(target: HTMLElement) {
 	}
 }
 
-function endEditing(applyValue: boolean) {
+function endEditing(applyValue: boolean, newValue: CellValue) {
 	if (!editing.value) {
 		return;
 	}
@@ -251,11 +257,10 @@ function endEditing(applyValue: boolean) {
 	emit('operation:endEdit', cell.value);
 	unregisterOutsideMouseDown();
 
-	if (applyValue && editingValue.value !== cell.value.value) {
-		emitValueChange(editingValue.value);
+	if (applyValue && newValue !== cell.value.value) {
+		emitValueChange(newValue);
 	}
 
-	editingValue.value = undefined;
 	editing.value = false;
 
 	rootEl.value?.focus();
@@ -279,11 +284,15 @@ useTooltip(rootEl, (showing) => {
 	}
 
 	const content = cell.value.violation.violations.filter(it => !it.valid).map(it => it.result.message).join('\n');
-	os.popup(defineAsyncComponent(() => import('@/components/grid/MkCellTooltip.vue')), {
+	const result = os.popup(defineAsyncComponent(() => import('@/components/grid/MkCellTooltip.vue')), {
 		showing,
 		content,
 		targetElement: rootEl.value!,
-	}, {}, 'closed');
+	}, {
+		closed: () => {
+			result.dispose();
+		},
+	});
 });
 
 onMounted(() => {
@@ -339,6 +348,11 @@ $cellHeight: 28px;
 	}
 }
 
+.contentArea, .inputArea {
+	display: flex;
+	align-items: center;
+}
+
 .content {
 	display: inline-block;
 	padding: 0 8px;

packages/frontend/src/components/grid/MkGrid.vue

@@ -82,14 +82,14 @@ const props = defineProps<{
 }>();
 
 // non-reactive
-
+// eslint-disable-next-line vue/no-setup-props-reactivity-loss
 const rowSetting: Required<GridRowSetting> = {
 	...defaultGridRowSetting,
 	...props.settings.row,
 };
 
 // non-reactive
-
+// eslint-disable-next-line vue/no-setup-props-reactivity-loss
 const columnSettings = props.settings.cols;
 
 // non-reactive

packages/frontend/src/components/grid/grid-utils.ts

@@ -8,7 +8,7 @@ import { DataSource, SizeStyle } from '@/components/grid/grid.js';
 import { CELL_ADDRESS_NONE, CellAddress, CellValue, GridCell } from '@/components/grid/cell.js';
 import { GridRow } from '@/components/grid/row.js';
 import { GridContext } from '@/components/grid/grid-event.js';
-import copyToClipboard from '@/scripts/copy-to-clipboard.js';
+import { copyToClipboard } from '@/scripts/copy-to-clipboard.js';
 import { GridColumn, GridColumnSetting } from '@/components/grid/column.js';
 
 export function isCellElement(elem: HTMLElement): boolean {

packages/frontend/src/pages/admin/custom-emojis-manager.local.list.vue

@@ -117,7 +117,11 @@ SPDX-License-Identifier: AGPL-3.0-only
 						</MkInput>
 					</div>
 
-					<XSortOrderFolder :sortOrders="sortOrders" @update="onSortOrderUpdate"/>
+					<MkFolder :spacerMax="8" :spacerMin="8">
+						<template #icon><i class="ti ti-arrows-sort"></i></template>
+						<template #label>{{ i18n.ts._customEmojisManager._gridCommon.sortOrder }}</template>
+						<MkSortOrderEditor :sortOrders="sortOrders" @update="onSortOrderUpdate"/>
+					</MkFolder>
 
 					<div :class="[[spMode ? $style.searchButtonsSp : $style.searchButtons]]">
 						<MkButton primary @click="onSearchRequest">
@@ -172,13 +176,13 @@ import { GridCellValidationEvent, GridCellValueChangeEvent, GridEvent } from '@/
 import { misskeyApi } from '@/scripts/misskey-api.js';
 import MkPagingButtons from '@/components/MkPagingButtons.vue';
 import XRegisterLogsFolder from '@/pages/admin/custom-emojis-manager.logs-folder.vue';
-import XSortOrderFolder from '@/pages/admin/custom-emojis-manager.sort-order-folder.vue';
 import MkFolder from '@/components/MkFolder.vue';
 import MkSelect from '@/components/MkSelect.vue';
 import { deviceKind } from '@/scripts/device-kind.js';
 import { GridSetting } from '@/components/grid/grid.js';
 import { selectFile } from '@/scripts/select-file.js';
 import { copyGridDataToClipboard, removeDataFromGrid } from '@/components/grid/grid-utils.js';
+import MkSortOrderEditor from '@/components/MkSortOrderEditor.vue';
 
 type GridItem = {
 	checked: boolean;

packages/frontend/src/pages/admin/custom-emojis-manager.remote.vue

@@ -53,7 +53,11 @@ SPDX-License-Identifier: AGPL-3.0-only
 					</MkInput>
 				</div>
 
-				<XSortOrderFolder :sortOrders="sortOrders" @update="onSortOrderChanged"/>
+				<MkFolder :spacerMax="8" :spacerMin="8">
+					<template #icon><i class="ti ti-arrows-sort"></i></template>
+					<template #label>{{ i18n.ts._customEmojisManager._gridCommon.sortOrder }}</template>
+					<MkSortOrderEditor :sortOrders="sortOrders" @update="onSortOrderChanged"/>
+				</MkFolder>
 
 				<div :class="[[spMode ? $style.searchButtonsSp : $style.searchButtons]]">
 					<MkButton primary @click="onSearchRequest">
@@ -93,11 +97,11 @@ import { emptyStrToUndefined, GridSortOrder, RequestLogItem } from '@/pages/admi
 import { GridCellValueChangeEvent, GridEvent } from '@/components/grid/grid-event.js';
 import MkFolder from '@/components/MkFolder.vue';
 import XRegisterLogsFolder from '@/pages/admin/custom-emojis-manager.logs-folder.vue';
-import XSortOrderFolder from '@/pages/admin/custom-emojis-manager.sort-order-folder.vue';
 import * as os from '@/os.js';
 import { GridSetting } from '@/components/grid/grid.js';
 import { deviceKind } from '@/scripts/device-kind.js';
 import MkPagingButtons from '@/components/MkPagingButtons.vue';
+import MkSortOrderEditor from '@/components/MkSortOrderEditor.vue';
 
 type GridItem = {
 	checked: boolean;