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

画像以外のファイルもアップロードできるように #373

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

poppingmoon
Copy link
Contributor

@poppingmoon poppingmoon commented Oct 3, 2023

画像編集に入るときと画像を表示するとき以外は画像とその他のファイルを区別せずに扱うようにしました

  • MisskeyPostFile を変更し、新しくアップロードするファイルを表す PostFile と、既にドライブにあるファイルを表す AlreadyPostedFile の2種類にまとめました
    • それぞれのクラスはファイルの中身のデータを持たず、必要な時にFileやDriveFileからデータを取得するようにしています
    • 画像編集の終了時には変更後のデータを新しいファイルに書き込み、新しいファイルを持った PostFile に置き換えます
  • 画像以外のファイルがノートに添付されているとき、サムネイルがなければファイルの種類に応じたアイコンを表示するようにしました
    • MimeTypeで区別しています
    • 編集と削除のアイコンボタンはファイルの種類にかかわらず表示するようにしました
    • ドライブからファイルを選択するダイアログでも同じようにアイコンを表示するようにしました

Fix #363
Fix #365

@codecov-commenter
Copy link

codecov-commenter commented Oct 3, 2023

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 44.37500% with 89 lines in your changes missing coverage. Please review.

Project coverage is 27.74%. Comparing base (8512cea) to head (5152702).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...r/note_create_page/note_create_state_notifier.dart 22.80% 44 Missing ⚠️
lib/view/note_create_page/create_file_view.dart 57.14% 18 Missing ⚠️
lib/model/misskey_post_file.dart 42.85% 16 Missing ⚠️
lib/view/note_create_page/drive_modal_sheet.dart 0.00% 6 Missing ⚠️
...iew/note_create_page/drive_file_select_dialog.dart 0.00% 3 Missing ⚠️
...ier/photo_edit_page/photo_edit_state_notifier.dart 0.00% 1 Missing ⚠️
lib/view/photo_edit_page/photo_edit_page.dart 0.00% 1 Missing ⚠️

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

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #373   +/-   ##
========================================
  Coverage    27.73%   27.74%           
========================================
  Files          312      313    +1     
  Lines        16296    16219   -77     
========================================
- Hits          4520     4500   -20     
+ Misses       11776    11719   -57     

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

@poppingmoon poppingmoon force-pushed the media-remove-data branch 2 times, most recently from 4faaab0 to bd3b980 Compare October 9, 2023 07:02
@poppingmoon poppingmoon force-pushed the media-remove-data branch 3 times, most recently from 54ed65c to 31d12cf Compare October 23, 2023 21:31
@poppingmoon poppingmoon force-pushed the media-remove-data branch 2 times, most recently from 854b681 to c7e4d4e Compare November 5, 2023 05:47
@poppingmoon poppingmoon force-pushed the media-remove-data branch 3 times, most recently from 20cd80b to 5e438d0 Compare November 24, 2023 16:09
@poppingmoon poppingmoon force-pushed the media-remove-data branch 2 times, most recently from d4f5544 to 07f95ed Compare January 28, 2024 00:12
@Npepperlinux
Copy link
Contributor

今までの「アップロード(画像のみ)」では、スマホでは写真アプリ、デスクトップでは画像ファイルでの絞り込みが行われていることで写真が探しやすかったため、「画像をアップロード」「ファイルをアップロード」の2つに分かれていると便利だと思います。

@poppingmoon
Copy link
Contributor Author

今までの「アップロード(画像のみ)」では、スマホでは写真アプリ、デスクトップでは画像ファイルでの絞り込みが行われていることで写真が探しやすかったため、「画像をアップロード」「ファイルをアップロード」の2つに分かれていると便利だと思います。

「メディアをアップロード」を追加しました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants