鲁树人 lsr
9 Followers · 0 Following
Repositories Projects Packages Public Activity Starred Repositories
lsr pushed to main at um/um-react 2023-05-20 01:20:10 +00:00
2deb27f0c5 chore: remove unused src config file
lsr pushed to main at um/um-react 2023-05-18 21:14:02 +00:00
bababc1aa8 docs: update progress in readme
lsr pushed to main at um/um-react 2023-05-18 21:12:58 +00:00
55e258f4ef chore: use url format for request id
d00ba50fcd test: added test for DecryptionQueue (#8)
Compare 2 commits »
lsr pushed to main at um/um-react 2023-05-18 21:01:20 +00:00
ab554b0470 fix: replace setImmediate with a cross environment compatible version
lsr commented on issue um/um-react#4 2023-05-17 23:42:45 +00:00
浏览器支持情况嗅探 + 错误处理

若是 window.Worker 不可用，启动时可能就会报错了。

lsr commented on issue um/um-react#8 2023-05-17 23:41:01 +00:00
单元测试

当前的测试覆盖率已经是项目整体 80% (当然，decrypt-worker 有很多文件还没包含进来…)

暂时已经算可用了，以后再调整吧。

lsr closed issue um/um-react#8 2023-05-17 23:41:01 +00:00
单元测试
lsr pushed to main at um/um-react 2023-05-17 23:40:21 +00:00
604eb6f939 chore: add lint-staged & husky
3b26584f4d test: proper sanity check; exclude WasmTest from coverage (#8)
Compare 2 commits »
lsr commented on pull request um/um-react#14 2023-05-17 23:23:18 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

感觉 src 和 fallbackSrc 可以直接合并成 `src={metadata?.cover

lsr commented on pull request um/um-react#14 2023-05-17 23:22:18 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

变量名可以考虑 noCoverFallbackImageURL

lsr pushed to main at um/um-react 2023-05-17 23:21:19 +00:00
4e9e360751 refactor: simplify logic handling worker message
lsr pushed to main at um/um-react 2023-05-17 23:18:28 +00:00
fb14eabaf1 chore: remove abuse of any in WorkerEventBus
lsr pushed to main at um/um-react 2023-05-17 23:04:59 +00:00
8597fe0d97 test: added test for ConcurrentQueue
6e6a8b1f5c fix: type safe ConcurrentQueue
be76a4590f chore: make eslint happy
6299302ff9 chore: configure eslint rules
ba5901d1eb chore: fix types for build & test
Compare 6 commits »
lsr commented on pull request um/um-react#14 2023-05-17 01:27:49 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

我刚刚合并了主分支的修改，发现按照主分支的代码，这个 svg 图片无法展示

如果你有兴趣，可以尝试修复下… 我在主分支的提交只确定了测试…

lsr commented on pull request um/um-react#14 2023-05-17 01:19:52 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

会再渲染，但我的看法是没必要这样优化，因为一般情况下缓存这些东西（简短的函数，基本不涉及复杂计算）的开销比不缓存更大。

这个项目的…

lsr commented on pull request um/um-react#14 2023-05-17 00:51:49 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

如果我快速点两次，是不是会收起然后马上又展开…？

lsr commented on pull request um/um-react#14 2023-05-17 00:47:39 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

主分支为了方便测试增加了一些更改，会与该 PR 的更改产生冲突。

主分支已手动集成了 "使用 Image 组件" 来展示封面部分。

lsr pushed to main at um/um-react 2023-05-17 00:46:44 +00:00
cd1f6cb186 test: working test with TypeScript + vite (using vitest)
1069c4c6bb chore: apply indentation rule for cjs/mjs as well
Compare 2 commits »
lsr pushed to main at um/um-react 2023-05-16 21:48:12 +00:00
71014c3a90 test: get coverage work
lsr commented on pull request um/um-react#14 2023-05-16 21:33:00 +00:00
WIP: 歌曲封面使用 Image 组件，删除歌曲行数据时使用一个动画效果来过渡一下

虽然很方便，但是建议手写一个 svg 导入，尽量避免动态加载第三方网站资源。