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 に画像をアップロードした際にHTMLタグ形式で表示する #8009

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

goruchanchan
Copy link
Contributor

@goruchanchan goruchanchan commented Aug 11, 2024

Issue

概要

以前のPRにて textarea に画像をアップロードした際の記法を、マークダウン形式から HTML タグ形式にしましたが、画像の縦横サイズが取得できなかったバグについての修正です。

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

変更確認方法

  1. bug/textarea_image_format_markdown_to_htmltag をローカルに取り込む
  2. bin/setup で npm を更新
  3. foreman start -f Procfile.dev でアプリを立ち上げる
  4. 任意ユーザ でログイン
  5. お知らせ作成のフォームページにアクセスし、テキストエリアに画像をアップロードし、テキストが <img src="ファイルパス" width="画像横サイズ" height="画像縦サイズ" loading="lazy" decoding="async" alt=”image.png”> となっていることを確認する
    ※ textarea へ画像をアップロードするページは複数ありますが、代表例として『お知らせ作成』のみを確認いただこうと思います。

Screenshot

変更前

  • ファイル名がある場合

    image
  • ファイル名がない場合

    image

変更後

  • ファイル名がある場合

    Image from Gyazo

  • ファイル名がない場合

    Image from Gyazo

@goruchanchan goruchanchan marked this pull request as ready for review August 13, 2024 22:08
@goruchanchan goruchanchan self-assigned this Aug 13, 2024
@goruchanchan
Copy link
Contributor Author

@thmz337 お疲れ様です!お手数をおかけして申し訳ないのですがコードレビューお願いできないしょうか?お忙しいなどありましたら、遠慮なく言っていただければと思います🙇‍♂️

@thmz337
Copy link
Contributor

thmz337 commented Aug 14, 2024

@goruchanchan

ご指名ありがとうございます!確認させていただきます。

@goruchanchan goruchanchan requested a review from thmz337 August 14, 2024 22:23
@goruchanchan
Copy link
Contributor Author

@thmz337 ありがとうございます!よろしくお願いします🙇‍♂️

@thmz337
Copy link
Contributor

thmz337 commented Aug 15, 2024

@goruchanchan
お疲れ様です。確認させていただきました。
問題ないかと思いますので、approveさせていただきます。

@goruchanchan
Copy link
Contributor Author

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

@komagata お疲れ様です!メンバーの承認をいただけましたので、ご確認よろしくお願いいたします🙇‍♂️

@goruchanchan goruchanchan requested a review from komagata August 15, 2024 13:20
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です〜🙆‍♂️

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