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

省略されたノートを一定の行まで表示する #295

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

poppingmoon
Copy link
Contributor

省略されたノートの表示にSimpleMfmTextの代わりにMfmTextを使い、リアクションをしたノートは1行、長いノートと判定されたノートは10行まで表示するようにしました

  • 行数は根拠があって決めたものではありません
  • 折りたたまれたノートはこの行数より短いことも長いこともあります
    • 短い場合、続きを表示のボタンを押してもボタンが消えるだけで内容に変化は起こりません
  • この変更によってmfmのタグが表示されたりすることが無くなります

shiosyakeyakini-info/mfm_renderer#1 に依存します

@poppingmoon
Copy link
Contributor Author

ボタンの位置がずれていることがある

@poppingmoon
Copy link
Contributor Author

ボタンの位置がずれていることがある

TextOverflowをfadeから変更すると起こらなくなった

@poppingmoon poppingmoon force-pushed the note-max-lines branch 4 times, most recently from e6253dc to af39efc Compare October 19, 2023 13:50
@shiosyakeyakini-info
Copy link
Owner

mfm_rendererのほうはマージしました!

$[position$[scaleでボタンが押せなくなってしまわないでしょうか…?

@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2023

Codecov Report

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

Comparison is base (07e6895) 37.56% compared to head (591036c) 37.51%.

Files Patch % Lines
lib/view/common/misskey_notes/misskey_note.dart 75.00% 8 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #295      +/-   ##
===========================================
- Coverage    37.56%   37.51%   -0.06%     
===========================================
  Files          251      251              
  Lines        13282    13263      -19     
===========================================
- Hits          4990     4976      -14     
+ Misses        8292     8287       -5     

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

@poppingmoon
Copy link
Contributor Author

長いノートを折りたたむ設定でカスタム絵文字がボタンに重なるようなノートを表示したところ、ボタンを押すことができることを確認しました

ノート (misskey.io)

1

2

3

4

$[position.y=3 $[scale.x=10,y=10 :yami_wo_kakeru_shikkoku_no_ryotak:]]

6

7

8

9

0

スクリーンショット

これを再現する際は改行コードがLFになっていることを確認してください

positionやscaleに関係して不具合が起きそうなところが他にありそうなら教えてください

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.

3 participants