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

[WYSIWYG]画像ダイアログの詳細タブで保存すると画像がリサイズされない問題を修正しました #1962

Merged
merged 2 commits into from Feb 28, 2024

Conversation

gakigaki
Copy link
Contributor

@gakigaki gakigaki commented Feb 28, 2024

概要

詳細設定タブで保存ボタンを押すと、パラメータに画像サイズが設定されないのが原因でした。
下記コードで値を設定していましたが、詳細設定タブを開くと一般タブの要素はすべて入れ替わるため、値を設定できていませんでした。

document.getElementById('cc-resized-image-size-' + frame_id).value = jQuery('.tox-listbox--select')[0].dataset.value

そこで、以下のように修正しました。

document.getElementById('cc-resized-image-size-' + frame_id).value = event.value.resize;

併せて、TinyMCEのコードも修正し、event.valueに画像サイズのデータを追加しました。
この修正により詳細設定タブでも画像サイズがサーバーに送信されます。

レビュー完了希望日

関連Pull requests/Issues

#1961

参考

DB変更の有無

無し

チェックリスト

@gakigaki gakigaki added the bug バグ・不具合連絡 label Feb 28, 2024
@gakigaki gakigaki changed the title [WYSIWYG]画像ダイアログの詳細タブで画像を保存すると画像がリサイズされない問題を修正しました [WYSIWYG]画像ダイアログの詳細タブで保存すると画像がリサイズされない問題を修正しました Feb 28, 2024
@gakigaki gakigaki marked this pull request as ready for review February 28, 2024 04:44
@akagane99 akagane99 self-requested a review February 28, 2024 04:45
@gakigaki
Copy link
Contributor Author

@akagane99
バージョン変えました!

Copy link
Contributor

@akagane99 akagane99 left a comment

Choose a reason for hiding this comment

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

レビューOK!

@gakigaki gakigaki merged commit cbd5abd into master Feb 28, 2024
1 check passed
@gakigaki gakigaki deleted the wysiwyg-image-advtab-resize branch March 4, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグ・不具合連絡
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants