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

textarea に画像をアップロードしたときのプレビュー画面に表示された画像をクリックすると、別タブで開かれるようにしました #8045

Merged
merged 2 commits into from
Sep 12, 2024

Conversation

goruchanchan
Copy link
Contributor

Issue

概要

以前のPRにて textarea に画像をアップロードした際の記法を、マークダウン形式から HTML タグ形式にしましたが、画像をクリックすると別タブで表示できなくなってしまったため表示できるようにしました。参考

変更に伴い textarea-markdown npm を更新しています。npm のコード修正はこちらの PR にて実施済みとなっているため確認不要です。本 PR では FBC アプリでの機能確認が主となります。

変更確認方法

  1. bug/textarea_image_format_markdown_to_htmltag_href をローカルに取り込む
  2. bin/setup で npm を更新
  3. foreman start -f Procfile.dev でアプリを立ち上げる
  4. 任意ユーザ でログイン
  5. お知らせ作成のフォームページにアクセスし、テキストエリアに画像をアップロードし、テキストが <a href="{画像のURL}" target="_blank" rel="noopener noreferrer"><img src="{画像のURL}" width="xxxx" height="xxxx" alt="{画像のファイル名}"></a> となっていることを確認する
  6. 画像をクリックし、別タブで開かれるかを確認する

Note

textarea へ画像をアップロードするページは複数ありますが、代表例として『お知らせ作成』のみを確認いただこうと思います。

Screenshot

変更前

画像をテキストエリアに貼り付けて、プレビューに表示された画像をクリックしても別タブが開かれませんでした。

edfb4642eedf5160b04aef01782cd763

変更後

プレビューに表示された画像をクリックすると別タブに画像が開かれるようになりました。

Image from Gyazo

@goruchanchan goruchanchan changed the title Bug/textarea image format markdown to htmltag href textarea に画像をアップロードしたときのプレビュー画面に表示された画像をクリックすると、別タブで開かれるようにしました Aug 30, 2024
@goruchanchan goruchanchan self-assigned this Aug 30, 2024
@goruchanchan
Copy link
Contributor Author

@su-su-su-su
お疲れ様です!お手数をおかけして申し訳ないのですがコードレビューいただけないでしょうか?ご都合悪いなどありましたら遠慮なく仰ってください🙇‍♂️

@su-su-su-su
Copy link
Contributor

@goruchanchan
お疲れ様です!
レビュー承知しました!
1週間程で対応しますので、少々お待ちください🙇‍♂️

@goruchanchan
Copy link
Contributor Author

@su-su-su-su お手数をおかけしますがよろしくお願いいたします🙇‍♂️

Copy link
Contributor

@su-su-su-su su-su-su-su left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@goruchanchan
動作とコード確認しました。勉強になりました🙇‍♂️
Approveします!

@goruchanchan
Copy link
Contributor Author

@su-su-su-su
ご確認ありがとうございました!大変助かりました🙇‍♂️

@komagata
メンバーレビュー完了しましたのでお手数をおかけしますがご確認よろしくお願いいたします🙇‍♂️

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit ad910a0 into main Sep 12, 2024
6 checks passed
@komagata komagata deleted the bug/textarea_image_format_markdown_to_htmltag_href branch September 12, 2024 16:56
@github-actions github-actions bot mentioned this pull request Sep 12, 2024
4 tasks
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