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

Mardown textarea に画像をアップロードしたとき、画像は Markdown 記法はではなく素のHTMLタグで、width と height と loading=”lazy” がついた状態で書き出されたい。 #7303

Closed
machida opened this issue Feb 2, 2024 · 9 comments
Assignees
Labels

Comments

@machida
Copy link
Member

machida commented Feb 2, 2024

現在は

![piyo](./images.jpg)

こうなるが、

<img src="./images.jpg" width="100" height="100" loading="lazy" decoding="async" alt=”piyo”>

こうしたい。

Copy link

github-actions bot commented Apr 8, 2024

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@goruchanchan
Copy link
Contributor

@komagata @machida
お疲れ様です🙇‍♂️バグ混入申し訳ないです🙇‍♂️
現状確認をしたところ、プロダクト環境にアップロードした画像のリンクにてバグが発生しているようでした🙇‍♂️

image

対策としては、machida さんが展開くださった『ダブルクォートでくくる』を導入しようと思いますが、いかがでしょうか?
また、属性については、width="", height="" のようにしようかと思っていますが、こちらはいかがでしょうか?

<!-- 記述例 -->
<img src="./image.png" width="" height="" loading="lazy" decoding="async" alt="image.png">

@machida
Copy link
Member Author

machida commented Apr 18, 2024

@goruchanchan
画像のwidth と height の値を取得し、その値を入れるようにしていただきたいです。
例えば、幅120000px * 高さ68000px の画像だったら、その値をそのままwidth、heightに指定するという形です。

lazyを指定することで、その部分までスクロールして画面に入ってから画像のローディングが始まります。しかし、画像の読み込み前の段階で、その部分の高さを確保しないと、読み込んだ後と前でレイアウトのずれが出てしまいます。レイアウトシフトというやつです。それを発生させないために、画像のサイズをimgタグに持たせます。

参考:
https://www.doe.co.jp/hp-tips/lazyload-setting-width-height/

画像のサイズを取得するのは難しいと思うので、ポイントの振り直しが必要な場合は言ってくださいー。

@goruchanchan
Copy link
Contributor

@komagata こちらの対応ですが、textarea-markdown npm に対応を入れる方針で良いでしょうか?

@goruchanchan
Copy link
Contributor

@komagata こちらの対応ですが、textarea-markdown npm に対応を入れる方針で良いでしょうか?

プレースホルダーか、オプションにて設定できるようにする

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jun 24, 2024
@komagata komagata removed the stale label Jun 24, 2024
@komagata komagata moved this to 作業中 in bootcamp Aug 21, 2024
@goruchanchan goruchanchan moved this from 作業中 to ステージング確認中 in bootcamp Aug 21, 2024
@goruchanchan goruchanchan moved this from ステージング確認中 to リリース待ち in bootcamp Aug 21, 2024
@goruchanchan
Copy link
Contributor

本番環境にて動作確認できましたのでクローズいたします🙇‍♂️

@github-project-automation github-project-automation bot moved this from リリース待ち to 完成 in bootcamp Aug 23, 2024
@machida
Copy link
Member Author

machida commented Aug 26, 2024

@goruchanchan

画像をクリックすると別タブで画像が表示されるという機能がなくなってしまっていました(拡大表示が必要なときはクリックをして別タブで見ます)。
別タブで画像を表示する機能を追加できますでしょうか?

<a href="{画像のURL}" target="_blank" rel="noopener noreferrer"><img src="{画像のURL}" width="xxxx" height="xxxx" alt="{画像のファイル名}"></a>

このようにしていただきたいです。

@goruchanchan
Copy link
Contributor

本番環境にて動作確認できましたのでクローズいたします🙇‍♂️

@github-project-automation github-project-automation bot moved this from リリース待ち to 完成 in bootcamp Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants