misskey icon indicating copy to clipboard operation
misskey copied to clipboard

enhance(frontend): ノートをたたむ基準を仮想行数から算定するように

Open kakkokari-gtyih opened this issue 1 year ago • 3 comments

What

MFMのASTを用いて、横の文字数のしきい値を基準にして仮想の行数を算出し、それが行数のしきい値を超えるか超えないかでノートをたたむ基準を判定するように

Why

Fix #13266 (あくまで一つの案です)

Additional info (optional)

クライアントの高さを測るのとどっちのほうがパフォーマンス良いかはわからない(畳まれる基準が横幅によらず統一されるという意味ではこちらのほうが良い気もするけど)

Checklist

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

kakkokari-gtyih avatar Apr 28 '24 12:04 kakkokari-gtyih

Codecov Report

Attention: Patch coverage is 5.33981% with 195 lines in your changes are missing coverage. Please review.

Project coverage is 77.37%. Comparing base (e2ff5f5) to head (b0fe6ea).

Files Patch % Lines
packages/frontend/src/scripts/collapsed.ts 1.51% 195 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13761      +/-   ##
===========================================
- Coverage    77.92%   77.37%   -0.55%     
===========================================
  Files          185      185              
  Lines        25535    25724     +189     
  Branches       487      487              
===========================================
+ Hits         19897    19903       +6     
- Misses        5631     5814     +183     
  Partials         7        7              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Apr 28 '24 12:04 codecov[bot]

(畳まれる基準が横幅によらず統一されるという意味ではこちらのほうが良い気もするけど)

横幅に応じてほしい気持ちが私はあります。

anatawa12 avatar Apr 28 '24 15:04 anatawa12

(畳まれる基準が横幅によらず統一されるという意味ではこちらのほうが良い気もするけど)

横幅に応じてほしい気持ちが私はあります。

完全にレスポンシブにすると統一基準として役立たなくなってしまいますが、PCとスマホの2パターンに大別するくらいはやってもよさそうな気がします

FineArchs avatar May 09 '24 21:05 FineArchs