Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 增加电视机雪花特效(close #256) #260

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Conversation

ChangeSuger
Copy link
Contributor

#256

补充电视机雪花特效。

因为是纯手搓,没有引入额外的图片资源,和原先的没找到相关资源就不执行特效的逻辑有冲突,稍微改了一下这个逻辑,希望不会影响其他特效。

测试效果:

@github-actions github-actions bot added the module: story-player 播放器的问题 label Oct 14, 2024
@mark9804
Copy link
Contributor

和原先的没找到相关资源就不执行特效的逻辑有冲突,稍微改了一下这个逻辑

这个我得明天看一下,今天要赶论文

@mark9804 mark9804 linked an issue Oct 14, 2024 that may be closed by this pull request
@mark9804 mark9804 merged commit 58f40d7 into dev Oct 17, 2024
8 checks passed
@mark9804
Copy link
Contributor

夸夸

@mark9804
Copy link
Contributor

没找到相关资源就不执行特效的逻辑

说实话我确实想不到为什么一年半以前幻想 @ourandream 写了这个逻辑,现在想想大概是为了表示“没资源就是没做”。如果是这样的话之后可能加个 warn 或者 error 提示会好一些

但是如果你觉得“既然代码上线了那一定有它上线的原因”,那么也有不用改的做法。对于特效素材而言我们有维护一张资源表,在资源表里声明这个特效不需要额外资源就可以

691e956#diff-2d7f803b2538d68127a075424c2569c4be260a4c15b1ce795997a8d6d7a4348cR114-R115

这样进入原本的逻辑之后,!![] === true 就可以开始执行

第二点是操作 screen 对象的时间太靠后了,加上初始化的 texture 是白色,因此特效开始的时候会爆闪一下。我暂时先 tint 成黑色了,不过如果要真正解决这个问题需要调整一下执行顺序

https://github.com/ba-archive/blue-archive/pull/260/files#diff-33a625f526d868fa9361ebc01514466513d4ee29cf99a764deeb1e2163a45271R48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: story-player 播放器的问题
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Enhancement] 电视机雪花特效 BG_TvNoise_Sound
2 participants