add URLs preview #45
add URLs preview #45
Conversation
|
現段階の表示はたたき台だとこう jsfiddle |
|
デザインわからんくなってしまった |
| <frost-url-preview url={ url } /> | ||
| </li> | ||
| </ul> | ||
| <frost-post-status if={ urls.length > 0 } urls={ urls }/> |
There was a problem hiding this comment.
これだとfrost-post-statusが二重に生成されるし、直す前の方が良いような
There was a problem hiding this comment.
これfrost-url-previewの間違いっぽいな
|
frost-url-previewのretriverをclient/helpers/下の関数として移動して、frost-post-status内でそれを呼ぶことにしよう |
|
その場合、frost-post-statusへの変更を |
|
肝心の表示のされ方としては良い感じ |
|
では戻していきましょう、 |
|
あーでも ul, li はもう使いたくないので直接eachとifで |
|
普通のjsファイルとして、 ul, liを使いたくないっていうのはなぜだろ eachが非同期で行われてるか(？)が不明ってことだから適当に書いてみた |
|
正直セマンティクスを考えるなら ul, li でやるべき感じはあるのでそうしますかね |
#44