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

PushableListViewの読み込み件数が0のときアイコンを表示しない #280

Conversation

poppingmoon
Copy link
Contributor

@poppingmoon poppingmoon commented Aug 30, 2023

PushableListViewのinitializeFutureの結果が空のときに「ありません」と表示するようにしました

変更前は、公開クリップのないユーザーのクリップ一覧やリプライのないノートの詳細画面などに反応しない下向き矢印が表示されていました

この変更によって、結果が空のとき、次の画面以外のPushableListViewでは「ありません」と表示するようになります

  • ノート詳細画面
    • リプライがないときにないことを示す必要がないため
  • ドライブファイル選択ダイアログのフォルダ一覧
    • 下にファイル一覧が続くため

@codecov-commenter
Copy link

codecov-commenter commented Sep 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c9acf09) 38.52% compared to head (117c934) 38.55%.

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

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #280      +/-   ##
===========================================
+ Coverage    38.52%   38.55%   +0.02%     
===========================================
  Files          248      248              
  Lines        12113    12116       +3     
===========================================
+ Hits          4667     4671       +4     
+ Misses        7446     7445       -1     

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

@poppingmoon poppingmoon force-pushed the fix-pushable-listview branch from c3a678f to 117c934 Compare December 17, 2023 01:59
@shiosyakeyakini-info shiosyakeyakini-info changed the base branch from develop to feature/miria_v2 July 14, 2024 02:49
@shiosyakeyakini-info shiosyakeyakini-info merged commit 8edb699 into shiosyakeyakini-info:feature/miria_v2 Jul 14, 2024
@poppingmoon poppingmoon deleted the fix-pushable-listview branch July 14, 2024 02:56
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