Fix/v15/code storage #67
Conversation
| @@ -38,7 +38,7 @@ export function FileInput(props) { | |||
| <img src={src} width="64" height="64" alt={alt} /> | |||
| </a> | |||
| {editMode === 'preview' ? null : `<!-- END ${item.name}@path:veil -->`} | |||
| <input type="file" name={item.name} id={id} /> | |||
| <input type="file" name={item.name} id={item.name} /> | |||
There was a problem hiding this comment.
@1000-x-t30
せっかくなら削除のチェックボックスもid統一したい気がしますがいかがでしょうか？
| @@ -51,7 +51,7 @@ export function ImageInput(props) { | |||
| {editMode === 'preview' ? null : '<!-- END_IF -->'} | |||
| <input | |||
| type="file" | |||
| id={id} | |||
| id={item.name} | |||
There was a problem hiding this comment.
@1000-x-t30
こちらも削除チェックボックスのidも統一したい気がしますがいかがでしょうか？
| @@ -112,7 +112,7 @@ export function Media(props) { | |||
| <input | |||
| type="hidden" | |||
| name={item.name} | |||
| id={id} | |||
| id={item.name} | |||
There was a problem hiding this comment.
mode === 'customfield' のところだけ id がついてて他ついてないのですが、これってどういう違いでしたっけ？
There was a problem hiding this comment.
@uidev1116
意図的に違いをつけたわけではありません。fileやmediaはinputがたくさんあり、label をどれに紐づけるべきか、そもそも紐づけるものなのか悩んでバラバラになっている状態です。最終的につけないで統一したつもりでしたが抜けていました。
どうするのがいいでしょうか？
There was a problem hiding this comment.
@1000-x-t30
なるほど。input[type="hidden"] にidをつけてもlabel と紐づかないので、メディアとリッチエディターはつけないで統一でいいかもです。
ファイルは input[type="hidden"] ではないのでつけたほうが良いです！
| @@ -63,7 +63,7 @@ export function RichEditor(props) { | |||
| {isValue ? ( | |||
| <> | |||
| <input | |||
| id={id} | |||
| id={item.name} | |||
There was a problem hiding this comment.
@1000-x-t30
ここも customunit と unitgroup はidがついてないのですが意図的ですか？？
There was a problem hiding this comment.
@uidev1116
こちらもfile, mediaと同様の理由です。
|
@1000-x-t30 |
はい、その通りです！ |
[カスタムフィールド名]-[値をエスケープしたもの]に修正