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(frontend): MFM のアニメーション要素に delay オプションを追加 #12659

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

YAVIIGI
Copy link
Contributor

@YAVIIGI YAVIIGI commented Dec 14, 2023

What

MFM のアニメーション要素(tada, jelly, twitch, shake, spin, jump, bounce, rainbow)の開始タイミングをずらす animation-delay を追加できるように delay オプションを追加します。

記述例
$[jump.delay=0.1s 🤲]

動作テスト(0.1sずつずらしてウェーブパフォーマンスのようなものを表現している様子)
add-delay-function

Why

#12655 の Purpose の内容を転載します。

MFM は Misskey の特徴ともいえる重要な要素の一つであり、熟達したユーザの中では MFM を駆使してアートを作られる方も増えてきています。今後も継続して MFM をアップデートしていくことで Misskey の魅力が高まっていくのではないかと想像しているため、今回は

  • MFM の自由度を高める
  • 視認性を下げる可能性の低い

の観点から animation-delay 用のプロパティを導入したいと考えています。

Additional info (optional)

  • delay に記述した時間後にアニメーションが実行されること
  • delay に負の値を入れた際は指定した時間まで戻った位置からアニメーションが開始されること
  • delay を記述していないときは暗示的に 0s のふるまいをすること

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Dec 14, 2023
Copy link

codecov bot commented Dec 14, 2023

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (8416329) 81.90% compared to head (b0e77c6) 81.88%.
Report is 2 commits behind head on develop.

Files Patch % Lines
...src/components/global/MkMisskeyFlavoredMarkdown.ts 0.00% 16 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12659      +/-   ##
===========================================
- Coverage    81.90%   81.88%   -0.03%     
===========================================
  Files          184      184              
  Lines        28290    28300      +10     
  Branches       503      503              
===========================================
+ Hits         23171    23173       +2     
- Misses        5119     5127       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@syuilo syuilo merged commit d776616 into misskey-dev:develop Dec 14, 2023
13 of 15 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 14, 2023

👍

camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants