feat(task-2.15): 素材选择器优化 — 批量选择+拖拽排序+预览缩略图+质量分筛选 #154

Merged

xiaoxia merged 3 commits from feature/task-2.15-asset-selector into develop 2026-07-01 19:19:30 +08:00

Conversation 5 Commits 3 Files Changed 8

+1345 -163

xiaoxia commented 2026-07-01 17:04:02 +08:00

Owner

Copy Link

Copy Source

No description provided.

xiaoxia added 1 commit 2026-07-01 17:04:02 +08:00

feat(task-2.15): 素材选择器优化 — 批量选择 + 拖拽排序 + 预览缩略图 + 质量分筛选 ... 7e954405f3

- 新增 AssetSelector 可复用组件（批量选择/拖拽排序/悬浮预览/质量分筛选/视图切换）
- MediaPanel 集成 AssetSelector，素材 Tab 使用 useQuery 内部获取数据
- TimelinePanel 支持批量素材拖入（application/x-media-assets）
- EditingPlanner 新增 handleBatchAddAssets 批量添加逻辑
- editPlans.ts 扩展 MediaAsset 类型（quality_score/classification_status）
- V21 设计系统响应式样式（1200/900/768/480 断点）

xiaoxia commented 2026-07-01 17:09:42 +08:00

Author

Owner

Copy Link

Copy Source

代码审查 — PR #154 任务 2.15：素材选择器优化

结论：❌ 不通过 — 存在 2 个 P1 功能缺陷，需修复后重审

---

审查范围

8 文件，+1323/-163：

• AssetSelector.tsx（新增 548 行）+ AssetSelector.css（新增 527 行）+ index.ts
• editPlans.ts（MediaAsset 扩展 + QUALITY_OPTIONS）
• EditingPlanner.tsx（移除素材查询 + 新增 handleBatchAddAssets）
• MediaPanel.tsx（重构：集成 AssetSelector）
• TimelinePanel.tsx（新增批量素材 drop 支持）

优点

1. AssetSelector 组件架构优秀：清晰的职责分离，props 设计合理（selectedIds/onSelectionChange/onReorder 受控模式），支持 grid/list 双视图
2. 批量选择功能完整：checkbox 多选 + 全选/反选 + Shift 连选，体验流畅
3. 拖拽排序实现规范：HTML5 DnD API 使用正确，dragIdx/dragOverIdx 状态管理清晰，视觉反馈到位
4. 悬浮预览细节好：400ms 延迟触发 + 组件卸载清理 timer，预览内容完整（缩略图/类型/时长/大小/质量分/标签）
5. MediaPanel 重构方向正确：将素材查询下沉到 MediaPanel 内部，EditingPlanner 不再持有素材数据，减少 prop drilling
6. TimelinePanel 兼容处理：同时支持单个素材（application/x-media-asset）和批量素材（application/x-media-assets）drop

---

P1 — 必须修复

P1-1：质量分筛选完全失效

AssetSelector.tsx 中质量分筛选逻辑：

if (filterQuality) {
  const minScore = parseInt(filterQuality, 10);
  list = list.filter(
    (a) => a.quality_score == null || a.quality_score >= minScore,
  );
}

filterQuality 的值来自 QUALITY_OPTIONS 的 value 字段，即 "high" / "medium" / "low"。

parseInt("high", 10) 返回 NaN。

a.quality_score >= NaN 始终为 false，因此筛选结果只保留 quality_score == null 的素材 — 恰好与预期相反。

修复方案：利用 QUALITY_OPTIONS 中已有的 min/max 字段：

if (filterQuality) {
  const option = QUALITY_OPTIONS.find(o => o.value === filterQuality);
  if (option?.min != null && option?.max != null) {
    list = list.filter(
      (a) => a.quality_score != null &&
             a.quality_score >= option.min &&
             a.quality_score <= option.max,
    );
  }
}

P1-2：AssetSelector → TimelinePanel 拖拽链路断裂

重构后素材拖入时间线的链路：

旧链路（PR #149）：MediaPanel handleDragStart 设置 e.dataTransfer.setData("application/x-media-asset", JSON.stringify(asset)) → TimelinePanel 读取该格式 ✅

新链路（本 PR）：AssetSelector handleDragStart 设置 e.dataTransfer.setData("text/plain", String(idx)) → TimelinePanel 仍期望 application/x-media-asset JSON ❌

onAssetDragStart 回调在 EditingPlanner 中是空函数（注释写着"可用于高亮时间线等"但未实现），不会设置任何 dataTransfer 数据。

结果：从 AssetSelector 拖拽素材到 TimelinePanel 完全无效，这是 PR #149 引入的功能回归。

修复方案：在 AssetSelector 的 handleDragStart 中同时设置素材数据：

const handleDragStart = useCallback(
  (e: React.DragEvent, idx: number) => {
    setDragIdx(idx);
    e.dataTransfer.effectAllowed = "copy";
    const asset = filteredAssets[idx];
    e.dataTransfer.setData("application/x-media-asset", JSON.stringify(asset));
    // 保持排序拖拽的 index 数据
    e.dataTransfer.setData("text/plain", String(idx));
    onAssetDragStart?.(asset);
  },
  [filteredAssets, onAssetDragStart],
);

---

P2 — 建议修复

P2-1：handleBatchAddAssets 忽略 insertIdx 参数

TimelinePanel 调用 onBatchAssetDrop(assets, insertIdx) 传入放置位置，但 EditingPlanner 的 handleBatchAddAssets 签名为 (assets: MediaAsset[]) 且始终 order: clips.length + i（追加到末尾）。

结果：批量拖拽到时间线中间位置时，素材不会插入到目标位置而是追加到末尾。

修复：让 handleBatchAddAssets 接受 insertIdx 并在该位置插入。

P2-2：拖拽语义不一致

AssetSelector 内部排序拖拽设置 effectAllowed = "move"，但拖到 TimelinePanel 应该是 copy 语义（素材从素材库复制到时间线，素材库中不删除）。当 AssetSelector 同时承担内部排序和外部拖出两种功能时，需要区分拖拽目标来设置正确的 effect。

---

P3 — 后续优化

1. 无测试覆盖：AssetSelector 是核心交互组件（548 行），建议至少添加批量选择、Shift 连选、筛选逻辑的单元测试
2. 预览浮层可能溢出视口：previewPos.x = rect.right + 12 未检查右侧边界，窄屏时预览可能被截断
3. 拖拽排序仅支持 AssetSelector 内部：onReorder 回调未在 EditingPlanner 中连接实现，选中素材的拖拽重排不会持久化

总结

AssetSelector 组件本身设计优秀，代码质量高。但 P1-1（质量分筛选 NaN）和 P1-2（拖拽链路断裂）两个功能缺陷导致核心交互失效，需修复后重审。

## 代码审查 — PR #154 任务 2.15：素材选择器优化 **结论：❌ 不通过 — 存在 2 个 P1 功能缺陷，需修复后重审** --- ### 审查范围 8 文件，+1323/-163： - `AssetSelector.tsx`（新增 548 行）+ `AssetSelector.css`（新增 527 行）+ `index.ts` - `editPlans.ts`（MediaAsset 扩展 + QUALITY_OPTIONS） - `EditingPlanner.tsx`（移除素材查询 + 新增 handleBatchAddAssets） - `MediaPanel.tsx`（重构：集成 AssetSelector） - `TimelinePanel.tsx`（新增批量素材 drop 支持） ### 优点 1. **AssetSelector 组件架构优秀**：清晰的职责分离，props 设计合理（selectedIds/onSelectionChange/onReorder 受控模式），支持 grid/list 双视图 2. **批量选择功能完整**：checkbox 多选 + 全选/反选 + Shift 连选，体验流畅 3. **拖拽排序实现规范**：HTML5 DnD API 使用正确，dragIdx/dragOverIdx 状态管理清晰，视觉反馈到位 4. **悬浮预览细节好**：400ms 延迟触发 + 组件卸载清理 timer，预览内容完整（缩略图/类型/时长/大小/质量分/标签） 5. **MediaPanel 重构方向正确**：将素材查询下沉到 MediaPanel 内部，EditingPlanner 不再持有素材数据，减少 prop drilling 6. **TimelinePanel 兼容处理**：同时支持单个素材（`application/x-media-asset`）和批量素材（`application/x-media-assets`）drop --- ### P1 — 必须修复 #### P1-1：质量分筛选完全失效 `AssetSelector.tsx` 中质量分筛选逻辑： ```ts if (filterQuality) { const minScore = parseInt(filterQuality, 10); list = list.filter( (a) => a.quality_score == null || a.quality_score >= minScore, ); } ``` `filterQuality` 的值来自 `QUALITY_OPTIONS` 的 `value` 字段，即 `"high"` / `"medium"` / `"low"`。 `parseInt("high", 10)` 返回 **NaN**。 `a.quality_score >= NaN` 始终为 `false`，因此筛选结果只保留 `quality_score == null` 的素材 — **恰好与预期相反**。 **修复方案**：利用 `QUALITY_OPTIONS` 中已有的 `min`/`max` 字段： ```ts if (filterQuality) { const option = QUALITY_OPTIONS.find(o => o.value === filterQuality); if (option?.min != null && option?.max != null) { list = list.filter( (a) => a.quality_score != null && a.quality_score >= option.min && a.quality_score <= option.max, ); } } ``` #### P1-2：AssetSelector → TimelinePanel 拖拽链路断裂 重构后素材拖入时间线的链路： **旧链路（PR #149）**：MediaPanel `handleDragStart` 设置 `e.dataTransfer.setData("application/x-media-asset", JSON.stringify(asset))` → TimelinePanel 读取该格式 ✅ **新链路（本 PR）**：AssetSelector `handleDragStart` 设置 `e.dataTransfer.setData("text/plain", String(idx))` → TimelinePanel 仍期望 `application/x-media-asset` JSON ❌ `onAssetDragStart` 回调在 EditingPlanner 中是**空函数**（注释写着"可用于高亮时间线等"但未实现），不会设置任何 dataTransfer 数据。 **结果**：从 AssetSelector 拖拽素材到 TimelinePanel 完全无效，这是 PR #149 引入的**功能回归**。 **修复方案**：在 AssetSelector 的 `handleDragStart` 中同时设置素材数据： ```ts const handleDragStart = useCallback( (e: React.DragEvent, idx: number) => { setDragIdx(idx); e.dataTransfer.effectAllowed = "copy"; const asset = filteredAssets[idx]; e.dataTransfer.setData("application/x-media-asset", JSON.stringify(asset)); // 保持排序拖拽的 index 数据 e.dataTransfer.setData("text/plain", String(idx)); onAssetDragStart?.(asset); }, [filteredAssets, onAssetDragStart], ); ``` --- ### P2 — 建议修复 #### P2-1：`handleBatchAddAssets` 忽略 `insertIdx` 参数 TimelinePanel 调用 `onBatchAssetDrop(assets, insertIdx)` 传入放置位置，但 EditingPlanner 的 `handleBatchAddAssets` 签名为 `(assets: MediaAsset[])` 且始终 `order: clips.length + i`（追加到末尾）。 **结果**：批量拖拽到时间线中间位置时，素材不会插入到目标位置而是追加到末尾。 **修复**：让 `handleBatchAddAssets` 接受 `insertIdx` 并在该位置插入。 #### P2-2：拖拽语义不一致 AssetSelector 内部排序拖拽设置 `effectAllowed = "move"`，但拖到 TimelinePanel 应该是 `copy` 语义（素材从素材库复制到时间线，素材库中不删除）。当 AssetSelector 同时承担内部排序和外部拖出两种功能时，需要区分拖拽目标来设置正确的 effect。 --- ### P3 — 后续优化 1. **无测试覆盖**：AssetSelector 是核心交互组件（548 行），建议至少添加批量选择、Shift 连选、筛选逻辑的单元测试 2. **预览浮层可能溢出视口**：`previewPos.x = rect.right + 12` 未检查右侧边界，窄屏时预览可能被截断 3. **拖拽排序仅支持 AssetSelector 内部**：`onReorder` 回调未在 EditingPlanner 中连接实现，选中素材的拖拽重排不会持久化 ### 总结 AssetSelector 组件本身设计优秀，代码质量高。但 P1-1（质量分筛选 NaN）和 P1-2（拖拽链路断裂）两个功能缺陷导致核心交互失效，需修复后重审。

xiaoxia referenced this issue from a commit 2026-07-01 17:34:50 +08:00

fix(asset-selector): 修复 PR #154 代码审查 4 项问题

xiaoxia added 1 commit 2026-07-01 17:34:50 +08:00

fix(asset-selector): 修复 PR #154 代码审查 4 项问题 ... 1fd93c95ba

P1-1: 质量分筛选 — parseInt 返回 NaN，改用 QUALITY_OPTIONS min/max 区间匹配
P1-2: AssetSelector→TimelinePanel 拖拽链断裂 — 恢复 application/x-media-asset(s) MIME 数据
P2-1: handleBatchAddAssets 忽略 insertIdx — 接受可选 insertIdx 参数，使用 splice 插入
P2-2: 拖拽语义区分 — 内部排序(text/plain)与外部素材投放(MIME types)互不干扰

同时修复 handleDragStart 依赖数组缺少 selectedIds 的问题

xiaoxia commented 2026-07-01 17:35:08 +08:00

Author

Owner

Copy Link

Copy Source

✅ PR #154 代码审查 4 项问题已修复

编号	问题	修复方案
P1-1	质量分筛选 parseInt("high") 返回 NaN	改用 QUALITY_OPTIONS 的 min/max 区间匹配
P1-2	AssetSelector→TimelinePanel 拖拽链断裂	恢复 application/x-media-asset / application/x-media-assets MIME 数据
P2-1	handleBatchAddAssets 忽略 insertIdx	接受可选 insertIdx 参数，使用 splice 在指定位置插入
P2-2	拖拽语义不一致	内部排序用 text/plain，外部素材投放用自定义 MIME types，互不干扰

额外修复：handleDragStart 依赖数组补充 selectedIds。

Commit: 1fd93c9 — 请 @灵应 复审 🙏

## ✅ PR #154 代码审查 4 项问题已修复 | 编号 | 问题 | 修复方案 | |------|------|----------| | **P1-1** | 质量分筛选 `parseInt("high")` 返回 NaN | 改用 `QUALITY_OPTIONS` 的 `min/max` 区间匹配 | | **P1-2** | AssetSelector→TimelinePanel 拖拽链断裂 | 恢复 `application/x-media-asset` / `application/x-media-assets` MIME 数据 | | **P2-1** | `handleBatchAddAssets` 忽略 `insertIdx` | 接受可选 `insertIdx` 参数，使用 `splice` 在指定位置插入 | | **P2-2** | 拖拽语义不一致 | 内部排序用 `text/plain`，外部素材投放用自定义 MIME types，互不干扰 | 额外修复：`handleDragStart` 依赖数组补充 `selectedIds`。 Commit: `1fd93c9` — 请 @灵应 复审 🙏

xiaoxia commented 2026-07-01 18:05:04 +08:00

Author

Owner

Copy Link

Copy Source

代码审查结果 — PR #154 修复版

结论：❌ 不通过（1 个新 P1 需修复）

---

上轮问题修复确认

编号	级别	问题	状态
P1-1	P1	质量分筛选 parseInt 返回 NaN	✅ 已修复
P1-2	P1	AssetSelector 拖拽到 TimelinePanel 链路断裂	✅ 已修复
P2-1	P2	handleBatchAddAssets 忽略 insertIdx	✅ 已修复
P2-2	P2	拖拽语义不一致（move vs copy）	✅ 合理

P1-1 修复正确：现在使用 QUALITY_OPTIONS 的 min/max 数值范围进行比较。
P1-2 修复正确：handleDragStart 现在正确设置了 application/x-media-asset 和 application/x-media-assets，TimelinePanel 也已同步支持两种数据类型。

---

新问题

🆕 P1-3：AssetSelector 导入路径错误，将导致 TS 编译失败

AssetSelector.tsx 第 6-10 行：

import type { MediaAsset } from "@/api/editingPlanner";
import {
  MATERIAL_TYPE_LABELS,
  MATERIAL_TYPE_ICONS,
  QUALITY_OPTIONS,
} from "@/api/editingPlanner";

但 MediaAsset、MATERIAL_TYPE_LABELS、MATERIAL_TYPE_ICONS、QUALITY_OPTIONS 均定义在 @/api/editPlans 中，editingPlanner.ts 并未 re-export 这些符号。

修复：将导入路径改为 @/api/editPlans。

---

P2-3：AssetSelector 内部拖拽排序未连接（onReorder 未传入）

MediaPanel 使用 AssetSelector 时未传入 onReorder prop，导致 AssetSelector 内部的拖拽排序逻辑（handleDragOver/handleDrop）虽然存在，但 drop 时因 onReorder 为 undefined 不会执行任何操作。

如果当前版本不需要排序功能，可以接受；否则需在 MediaPanel 中传入 onReorder 回调。

---

其余 P3 建议（不阻塞）

• P3-1：仍无前端单元测试覆盖
• P3-2：悬浮预览浮层未做视口边界检测，极端情况下可能溢出

---

总结：上轮两个 P1 均已正确修复，代码质量明显提升。但引入了新的 P1（导入路径错误导致编译失败），需修复后可合并。

## 代码审查结果 — PR #154 修复版 **结论：❌ 不通过（1 个新 P1 需修复）** --- ### 上轮问题修复确认 | 编号 | 级别 | 问题 | 状态 | |------|------|------|------| | P1-1 | P1 | 质量分筛选 parseInt 返回 NaN | ✅ 已修复 | | P1-2 | P1 | AssetSelector 拖拽到 TimelinePanel 链路断裂 | ✅ 已修复 | | P2-1 | P2 | handleBatchAddAssets 忽略 insertIdx | ✅ 已修复 | | P2-2 | P2 | 拖拽语义不一致（move vs copy） | ✅ 合理 | P1-1 修复正确：现在使用 QUALITY_OPTIONS 的 min/max 数值范围进行比较。 P1-2 修复正确：handleDragStart 现在正确设置了 `application/x-media-asset` 和 `application/x-media-assets`，TimelinePanel 也已同步支持两种数据类型。 --- ### 新问题 #### 🆕 P1-3：AssetSelector 导入路径错误，将导致 TS 编译失败 AssetSelector.tsx 第 6-10 行： ```tsx import type { MediaAsset } from "@/api/editingPlanner"; import { MATERIAL_TYPE_LABELS, MATERIAL_TYPE_ICONS, QUALITY_OPTIONS, } from "@/api/editingPlanner"; ``` 但 `MediaAsset`、`MATERIAL_TYPE_LABELS`、`MATERIAL_TYPE_ICONS`、`QUALITY_OPTIONS` 均定义在 `@/api/editPlans` 中，`editingPlanner.ts` 并未 re-export 这些符号。 **修复**：将导入路径改为 `@/api/editPlans`。 --- #### P2-3：AssetSelector 内部拖拽排序未连接（onReorder 未传入） MediaPanel 使用 AssetSelector 时未传入 `onReorder` prop，导致 AssetSelector 内部的拖拽排序逻辑（handleDragOver/handleDrop）虽然存在，但 drop 时因 `onReorder` 为 undefined 不会执行任何操作。 如果当前版本不需要排序功能，可以接受；否则需在 MediaPanel 中传入 onReorder 回调。 --- ### 其余 P3 建议（不阻塞） - P3-1：仍无前端单元测试覆盖 - P3-2：悬浮预览浮层未做视口边界检测，极端情况下可能溢出 --- **总结**：上轮两个 P1 均已正确修复，代码质量明显提升。但引入了新的 P1（导入路径错误导致编译失败），需修复后可合并。

xiaoxia commented 2026-07-01 18:37:33 +08:00

Author

Owner

Copy Link

Copy Source

🔄 修复版复审结果

原有 4 项问题修复验证

问题	级别	状态	说明
P1-1: 质量分筛选 parseInt NaN	P1	✅ 已修复	使用 QUALITY_OPTIONS.find() + opt.min/opt.max 进行数值比较，正确
P1-2: 拖拽链路 dataTransfer	P1	✅ 已修复	handleDragStart 正确设置 application/x-media-asset 和 application/x-media-assets，TimelinePanel 两端均能识别
P2-1: handleBatchAddAssets 忽略 insertIdx	P2	✅ 已修复	const at = insertIdx ?? clips.length，正确使用 insertIdx 并带 fallback
P2-2: 拖拽语义不一致	P2	✅ 设计合理	内部排序用 text/plain，跨组件用 application/x-media-asset(s)，语义清晰

🆕 新发现问题

[P1] AssetSelector.tsx 导入路径错误，将导致 TS 编译失败

AssetSelector.tsx 第 671-676 行：

import type { MediaAsset } from "@/api/editingPlanner";
import {
  MATERIAL_TYPE_LABELS,
  MATERIAL_TYPE_ICONS,
  QUALITY_OPTIONS,
} from "@/api/editingPlanner";

但以上 4 个符号 均定义在 @/api/editPlans.ts 中：

• MediaAsset（editPlans.ts:67）
• MATERIAL_TYPE_LABELS（editPlans.ts:340）
• MATERIAL_TYPE_ICONS（editPlans.ts:348）
• QUALITY_OPTIONS（editPlans.ts:365）

editingPlanner.ts 中 不存在 这些导出，也没有从 editPlans re-export。

修复方案： 将导入路径从 @/api/editingPlanner 改为 @/api/editPlans：

import type { MediaAsset } from "@/api/editPlans";
import {
  MATERIAL_TYPE_LABELS,
  MATERIAL_TYPE_ICONS,
  QUALITY_OPTIONS,
} from "@/api/editPlans";

---

结论

❌ 暂不通过 — 原有 4 项修复全部正确，但新引入的 P1 导入路径错误会导致 TypeScript 编译失败，修复后可通过。

## 🔄 修复版复审结果 ### 原有 4 项问题修复验证 | 问题 | 级别 | 状态 | 说明 | |------|------|------|------| | P1-1: 质量分筛选 parseInt NaN | P1 | ✅ 已修复 | 使用 `QUALITY_OPTIONS.find()` + `opt.min/opt.max` 进行数值比较，正确 | | P1-2: 拖拽链路 dataTransfer | P1 | ✅ 已修复 | `handleDragStart` 正确设置 `application/x-media-asset` 和 `application/x-media-assets`，TimelinePanel 两端均能识别 | | P2-1: handleBatchAddAssets 忽略 insertIdx | P2 | ✅ 已修复 | `const at = insertIdx ?? clips.length`，正确使用 insertIdx 并带 fallback | | P2-2: 拖拽语义不一致 | P2 | ✅ 设计合理 | 内部排序用 `text/plain`，跨组件用 `application/x-media-asset(s)`，语义清晰 | ### 🆕 新发现问题 **[P1] AssetSelector.tsx 导入路径错误，将导致 TS 编译失败** `AssetSelector.tsx` 第 671-676 行： ```tsx import type { MediaAsset } from "@/api/editingPlanner"; import { MATERIAL_TYPE_LABELS, MATERIAL_TYPE_ICONS, QUALITY_OPTIONS, } from "@/api/editingPlanner"; ``` 但以上 4 个符号 **均定义在 `@/api/editPlans.ts`** 中： - `MediaAsset`（editPlans.ts:67） - `MATERIAL_TYPE_LABELS`（editPlans.ts:340） - `MATERIAL_TYPE_ICONS`（editPlans.ts:348） - `QUALITY_OPTIONS`（editPlans.ts:365） `editingPlanner.ts` 中 **不存在** 这些导出，也没有从 `editPlans` re-export。 **修复方案：** 将导入路径从 `@/api/editingPlanner` 改为 `@/api/editPlans`： ```tsx import type { MediaAsset } from "@/api/editPlans"; import { MATERIAL_TYPE_LABELS, MATERIAL_TYPE_ICONS, QUALITY_OPTIONS, } from "@/api/editPlans"; ``` --- ### 结论 ❌ **暂不通过** — 原有 4 项修复全部正确，但新引入的 P1 导入路径错误会导致 TypeScript 编译失败，修复后可通过。

xiaoxia added 1 commit 2026-07-01 18:40:06 +08:00

fix(asset-selector): 修复导入路径 — 从 editingPlanner 改为 editPlans ... 1a7f46805c

修复代码审计发现的 P1 问题：
- MediaAsset / MATERIAL_TYPE_LABELS / MATERIAL_TYPE_ICONS / QUALITY_OPTIONS
  实际定义在 @/api/editPlans.ts，而非 @/api/editingPlanner.ts
- 修正导入路径，消除 TS 编译错误

xiaoxia commented 2026-07-01 18:42:13 +08:00

Author

Owner

Copy Link

Copy Source

✅ 复审通过

HEAD SHA: 1a7f46805c20

修复验证

问题	级别	状态	验证说明
P1-1: 质量分筛选 parseInt NaN	P1	✅ 已修复	QUALITY_OPTIONS.find() + opt.min/opt.max 数值比较，正确
P1-2: 拖拽链路 dataTransfer	P1	✅ 已修复	handleDragStart 正确设置 application/x-media-asset 和 application/x-media-assets，TimelinePanel 双端识别
P2-1: handleBatchAddAssets 忽略 insertIdx	P2	✅ 已修复	const at = insertIdx ?? clips.length，正确使用 insertIdx
P2-2: 拖拽语义不一致	P2	✅ 设计合理	内部排序用 text/plain，跨组件用 MIME 类型，语义清晰
P1-3: AssetSelector 导入路径错误	P1	✅ 已修复	新 commit 1a7f468 将导入从 @/api/editingPlanner 修正为 @/api/editPlans，TS 编译问题已解决

结论

所有 P1/P2 问题均已修复，代码可以合并。

## ✅ 复审通过 HEAD SHA: `1a7f46805c20` ### 修复验证 | 问题 | 级别 | 状态 | 验证说明 | |------|------|------|----------| | P1-1: 质量分筛选 parseInt NaN | P1 | ✅ 已修复 | `QUALITY_OPTIONS.find()` + `opt.min/opt.max` 数值比较，正确 | | P1-2: 拖拽链路 dataTransfer | P1 | ✅ 已修复 | `handleDragStart` 正确设置 `application/x-media-asset` 和 `application/x-media-assets`，TimelinePanel 双端识别 | | P2-1: handleBatchAddAssets 忽略 insertIdx | P2 | ✅ 已修复 | `const at = insertIdx ?? clips.length`，正确使用 insertIdx | | P2-2: 拖拽语义不一致 | P2 | ✅ 设计合理 | 内部排序用 `text/plain`，跨组件用 MIME 类型，语义清晰 | | **P1-3: AssetSelector 导入路径错误** | **P1** | **✅ 已修复** | 新 commit `1a7f468` 将导入从 `@/api/editingPlanner` 修正为 `@/api/editPlans`，TS 编译问题已解决 | ### 结论 所有 P1/P2 问题均已修复，代码可以合并。

xiaoxia merged commit 60edd59700 into develop 2026-07-01 19:19:30 +08:00

xiaoxia referenced this issue from a commit 2026-07-01 19:19:31 +08:00

Merge pull request 'feat(task-2.15): 素材选择器优化 — 批量选择+拖拽排序+预览缩略图+质量分筛选' (#154) from feature/task-2.15-asset-selector into develop

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#154