redmica / redmica_ui_extension

This plugin adds useful UI improvements to RedMica.
GNU General Public License v2.0
41 stars 12 forks source link

Add preview of Attachment using BigPicture.js (2.6.1) #31

Closed takenory closed 2 years ago

takenory commented 2 years ago

I propose a feature to preview Attachment using BigPicture.js. This feature allows you to preview image files, audio files, video files, and PDF files without screen transitions.

takenory commented 2 years ago

@yui-har wrote:

コメント欄の添付ファイルをBigPictureで表示後、BigPictureを閉じるとチケット画面のトップに戻ります。

ほんとですね;よく見ると表示する段階で画面のトップに移動してますね; そして、虫眼鏡アイコンをマウスオーバーしてみると... http://localhost:3000/issues/1# のようにリンクしてることがわかりました。 画面遷移しないよう見直します!

takenory commented 2 years ago

@yui-har

コメント欄の添付ファイルをBigPictureで表示後、BigPictureを閉じるとチケット画面のトップに戻ります。

以下のコミットで修正しました。ご確認ください。 https://github.com/redmica/redmica_ui_extension/pull/31/commits/f9dba128401173a77e37eabc0fbdab8a0ff6401f

takenory commented 2 years ago

@yui-har https://github.com/redmica/redmica_ui_extension/pull/31#issuecomment-1060241684 の件、テストの修正が漏れていました; テストを修正してから再度レビューの依頼をさせてください。

takenory commented 2 years ago

@yui-har

コメント欄の添付ファイルをBigPictureで表示後、BigPictureを閉じるとチケット画面のトップに戻ります。

以下のコミットで修正しました。ご確認ください。

yui-har commented 2 years ago

@yui-har

コメント欄の添付ファイルをBigPictureで表示後、BigPictureを閉じるとチケット画面のトップに戻ります。

以下のコミットで修正しました。ご確認ください。

@takenory チケット画面がトップに移動すること無く、クリックした箇所でBigPictureが表示されることを確認しました。

takenory commented 2 years ago

@ishikawa999 @yui-har レビューいただきありがとうございました。 @ishikawa999 マージをお願いいたします。

ishikawa999 commented 2 years ago

マージしました。