升级到 Vue3（旧标题：升级 Vue 版本到 2.7)

[timongh]

Vue 2.7 是 Vue 2.x 的最后一个版本（支持到 2023 年 12 月 31 日），而且内置了组合式 API。可以考虑升级一下。

我自己先改改，改完了提个 PR 上来。

see

Vue 2 LTS， EOL & Extended Support

Migration to Vue 2.7

[the1812]

我挺想直接升 3 的, 换组合式 API 总归要改代码

[timongh]

我先把一些已经发现的 BUG 作为 issue 提上来

[the1812]

还有一个问题是，核心升级后，原来用 Vue2 编写的组件可能就不能跑了，因此可能需要禁止加载老版本的组件，还要引导升级，这部分我不太熟，可能要你帮忙。

升 3 就是要另外搞个分支, 一次性弄好合到主分支才行, 所以升级完应该就不存在 Vue 2 的组件了. (我觉得主要的工作量也是用 3 的 API 重写这些组件)

[timongh]

不是啊，大部分代码还是可以重用的，Vue3 也可以用选项式的定义。只用把 Vue.extend 改成 defineComponent。再改改 2 跟 3 的破坏式更新就好了（虽然工作量也不小）。

[timongh]

所以升级完应该就不存在 Vue 2 的组件了

我的意思是，用户的油猴存储里存的代码还是老版本的代码，要禁止新的 core 加载旧的组件代码。因为我看过核心里有关版本的代码，隐约记得其中的逻辑好像是：如果找不到 version 这个字段，仍然会继续加载代码。

[the1812]

我的意思是，用户的油猴存储里存的代码还是老版本的代码，要禁止新的 core 加载旧的组件代码。因为我看过核心里有关版本的代码，隐约记得其中的逻辑好像是：如果找不到 version 这个字段，仍然会继续加载代码。

那确实, 之前认为旧组件代码也是兼容可用的, 应该得加个特殊处理了

[timongh]

src/ui/RangeInput.vue 能接受一个 range，类型是 Range<?>，但这个类型参数我从上下文推测不出来。只能看出他能调用 toString() 方法。

[the1812]

在一旁的 range.ts 里

export interface Range<T = number> {
  start: T
  end: T
}

[timongh]

那这个 T 要限制为 number 吗？因为在上下文中对这个 T 调用了 toString，所以不能是 unknown，至少也应该是个 { toString: () => string }。

[the1812]

目前在用的都是 number, 直接去掉泛型也行

[timongh]

不对，我之前没注意，这个地方只能是 Range<string>。因为 RangeInput 内部使用的是 TextBox，而 TextBox 的输入输出都只能是 string。

[the1812]

我记错了, 重新看了下代码, 确实是 string. (夜间模式计划时段有用到)

[timongh]

src/ui/VDropdown.vue 能接受一个 value，这个 value 的类型我也看不出来。只能从上下文看出，这个类型里面有个 displayName 属性。

[timongh]

@the1812

[the1812]

只是任意有这个属性的对象就行了, 但也只是默认在 slot 里读取 displayName, 这个 slot 可以被替换掉

interface Item {
  displayName: string
}

[timongh]

另外，目前大部分升级工作已经完成。只不过所有修改后的代码都还没有测试。

[timongh]

@the1812

目前不需要大改动的代码已经完成（逻辑改变不大的）。

剩下的因为涉及逻辑更改，至少需要被审核后才能合并，所以要提单独的 PR 才行。这部分我就等单独分支开出来再做了（因为要提交 PR）

[timongh]

@the1812

目前的情况是：有一部分代码必须重构，否则无法正常工作。有部分代码推荐重构、优化。另外，版本的隔离和迁移没有做。所以现在是不能合并进 preview-features 的。

但是重构我认为至少还是要 review 一下。所以我的意见是建个 vue3 的分支。我通过提 PR 的方式来一个个地开发。

[the1812]

可以的, 我根据 preview-features 建一个?

[timongh]

恩恩

[the1812]

https://github.com/the1812/Bilibili-Evolved/tree/vue3

[timongh]

顺便问一下，如果不 @ 的话，不知道能否收到消息。

[the1812]

能的