Skip to content

feat: 校对字幕支持单行删除#369

Open
nightt5879 wants to merge 1 commit into
buxuku:mainfrom
nightt5879:nightt5879/issue-365-delete-subtitle-line
Open

feat: 校对字幕支持单行删除#369
nightt5879 wants to merge 1 commit into
buxuku:mainfrom
nightt5879:nightt5879/issue-365-delete-subtitle-line

Conversation

@nightt5879

@nightt5879 nightt5879 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

close #365

修改内容

  • 在校对字幕的展开行按钮区新增“删除字幕”按钮,位置与“AI 优化”“拆分”同组。
  • 新增删除字幕的状态操作:删除当前行后自动重新编号,并根据删除位置修正当前选中字幕索引。
  • 删除操作接入现有撤销/重做历史,误删后可以直接撤销恢复。
  • 补充中英文文案,包括按钮提示和删除成功提示。

实现说明

校对编辑器原有的合并、拆分都通过字幕 hook 统一更新字幕数组和撤销历史。本次删除也复用同一套区间命令模型,把“1 行变 0 行”记录为一条可撤销命令,避免直接修改数组导致历史栈、字幕 ID 或当前播放索引错位。

验证

  • npm run check:i18n
  • npm run build

@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

@nightt5879 is attempting to deploy a commit to the buxuku's projects Team on Vercel.

A member of the Team first needs to authorize it.

@nightt5879 nightt5879 marked this pull request as ready for review July 6, 2026 07:50
@buxuku

buxuku commented Jul 6, 2026

Copy link
Copy Markdown
Owner

感谢 PR!实现思路和我们几乎完全一致(同样基于 range-command 的可撤销删除 + currentIndex 重整),代码质量很好。

同样抱歉的是,这个功能我刚在 feat/cloud-asr-providers 分支上实现了(df097091bc7544d5b1b5d4bdf93b4e486e836287),当时没在 #365 上标记,让你重复劳动了 🙏 那边的版本会随下个版本一起发布(toast 里额外提示了可撤销恢复)。

你给按钮加 aria-label 的细节很好,我已经借鉴过来,一并补到了校对行的三个操作按钮上。本 PR 会在功能发布后关闭,再次感谢,期待后续继续参与!

buxuku added a commit that referenced this pull request Jul 6, 2026
buxuku added a commit that referenced this pull request Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

功能建议:希望可以在校对时单独删除某一行字幕

2 participants