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

最後のframeを先頭に移動して、アニメーション非対応の環境でも文字を読めるようにする #338

Closed
wants to merge 4 commits into from

Conversation

hata6502
Copy link
Contributor

@hata6502 hata6502 commented Aug 23, 2023

#339

最後のframeを先頭に移動することで、アニメーション非対応の環境でも文字を読めるようにしました。
ループ画像のため、frameをローテーションすることの問題は少ないと思います。

Image from Gyazo
Image from Gyazo

@hata6502 hata6502 marked this pull request as draft August 25, 2023 02:34
@hata6502
Copy link
Contributor Author

hata6502 commented Aug 25, 2023

最後のframeを先頭に移動すると、「爆散」「BLINK」などのアニメーションの1コマ目が読めなくなる問題に気づきました。
Draftに戻します。
Image from Gyazo

@hata6502
Copy link
Contributor Author

hata6502 commented Aug 25, 2023

Canvasが空白であるか判定して、「爆散」「BLINK」効果に対応してみました。
Image from Gyazo

  • 先頭frameが空白の場合
    • 最後のframeを先頭にrotateする。
  • 先頭frameが空白でない場合
    • 何もしない。

@hata6502 hata6502 marked this pull request as ready for review August 25, 2023 13:11
@hata6502
Copy link
Contributor Author

レビューお願いいたします。

@zk-phi
Copy link
Owner

zk-phi commented Oct 2, 2023

ありがとうございます&確認遅くなってしまってすみませんmm
完全に盲点でした、おっしゃる通りですね。

実装方法については、そもそもエフェクト側が先頭フレームに意味ある内容を出力すべきな気もしていて、ちょっと迷っています。

メモ https://scrapbox.io/MEGAMOJI/%E3%82%A2%E3%83%8B%E3%83%A1%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3_OFF_%E3%81%AE%E7%92%B0%E5%A2%83%E3%81%A7%E6%84%8F%E5%91%B3%E3%81%AE%E3%81%82%E3%82%8B%E7%94%BB%E3%82%92%E8%A1%A8%E7%A4%BA%E3%81%97%E3%81%9F%E3%81%84

@zk-phi
Copy link
Owner

zk-phi commented Oct 4, 2023

「そもそも先頭フレームが空白になってしまうようなエフェクトを避ける」という方向で対応しました 🙏 #345

@zk-phi zk-phi closed this Oct 4, 2023
@hata6502 hata6502 deleted the a11y branch October 6, 2023 03:56
@hata6502
Copy link
Contributor Author

hata6502 commented Oct 6, 2023

@zk-phi 対応していただきありがとうございました!

なるほど、各エフェクトごとに対応するのよいですね。
複数のエフェクトを組み合わせた場合でも、1フレーム目に文字が大きく映りそうです!

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.

None yet

2 participants