-
Notifications
You must be signed in to change notification settings - Fork 653
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
4.1-beta3 : 管理画面 ファイルアップロード UI の修正 #5125
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.1 #5125 +/- ##
==========================================
- Coverage 68.16% 68.16% -0.01%
==========================================
Files 457 457
Lines 24980 24988 +8
==========================================
+ Hits 17027 17032 +5
- Misses 7953 7956 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
動作確認しました。 変更前商品登録画面<div class="custom-file">
<input type="file" id="admin_product_product_image" name="admin_product[product_image][]" multiple="multiple" accept="image/*" style="display:none;" class="custom-file-input">
<label for="admin_product_product_image" class="custom-file-label"></label>
</div> ファイル管理<div class="custom-file">
<input type="file" id="form_file" name="form[file][]" required="required" multiple="multiple" class="custom-file-input"><label for="form_file" class="custom-file-label"></label>
</div> 変更後商品登録画面<input type="file" id="admin_product_product_image" name="admin_product[product_image][]" multiple="multiple" accept="image/*" style="display:none;" class="form-control-file"> ファイル管理<input type="file" id="form_file" name="form[file][]" required="required" multiple="multiple" class="form-control-file"> 4.0商品登録画面<input type="file" id="admin_product_product_image" name="admin_product[product_image][]" multiple="multiple" accept="image/*" style="display:none;" class="form-control-file"> ファイル管理<input type="file" id="form_file" name="form[file][]" required="required" multiple="multiple" class="form-control-file"> |
独自プラグインのアップデートの際のファイルアップロードも対応しました。 |
独自プラグインアップデートの file_widget の出力HTMLの調整
@okazy ありがとうございます、修正漏れでしたね、マージさせてもらいました! |
ありがとうございます!マージしました。 |
概要(Overview・Refs Issue)
4.1 beta3 にて、管理画面のファイルアップロードの UI が 4.0と異なっていた問題に対応しました
関連issue
#5114
方針(Policy)
file_widget で出力されるHTMLコードは 4.0 と 4.1 とで差異があったため、bootstrap_4_horizontal_layout.html.twig にて新規に file_widget ブロックを作成し、出力されるHTMLを 4.0時と同等にする (CSSは変更しない)
管理画面の widget 追加のみのため、フロントには影響なしの想定
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
レビュワー確認項目