search

e2d3 / web

0 stars 3 forks source link

徒競走のグラフ組み込み #8

Closed hiracky16 closed 6 years ago

hiracky16 commented 6 years ago

5 animal_olympic の組み込みです。

デモページ: http://asobiba.tech/wordpress/index.php/animal_olympic/#drawArea

hiracky16 commented 6 years ago

@xiushu53 @Yeshi push したあとに気づいたのですが、 要素に直接 style 当てちゃってるので必要であれば css に逃します。。 ご確認よろしくおねがいします！

xiushu53 commented 6 years ago

index.jsで、styleで設定されている

width: ICON_WIDTH, height: ICON_HEIGHT

をattrに設定することで、FireFoxでは表示できました。

Safariではうまく表示できません(泣き

hiracky16 commented 6 years ago

@xiushu53 さんの指摘の通りにアイコンのサイズ指定を attr に変更しました。 結果 Firefox と IE11 で動くことは検証済みです。 ただ、前述の通り Safari では動かないため、まだ修正する必要があります。 原因に関しては引き続き調査します。

xiushu53 commented 6 years ago

@hiracky16 ありがとうございます！

hiracky16 commented 6 years ago

image タグに使われている href が Safari の SVG だとサポートされていなくて画像が表示されていませんでした… xlink:href だと参照できたのでそれに変更しました！ 参考: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/href

hiracky16 commented 6 years ago

@xiushu53 @Yeshi リファクタリングも済んだので改めてご確認よろしくおねがいします。

xiushu53 commented 6 years ago

@hiracky16 ありがとうございます！ IE,Safari,Chrome,FireFoxで、表示確認しました。

@Yeshi その他、問題なければmergeお願いします。

Yeshi commented 6 years ago

@hiracky16 確認遅れましてすみません！ ローカルのWordpressで動作確認をしようとしたのですが、確かWordpressのfunctions.phpも追記されてましたっけ。d3本体が読み込めず…。 functions.phpもaddお願いできますでしょうか。

Yeshi commented 6 years ago

あと <?php $template_path = get_template_directory_uri(); ?> の記述と、images/animal_icons/ のデータも必要そうですね。

hiracky16 commented 6 years ago

@Yeshi 指摘事項直しました！ functions.php はどうしましょうか？

12 で用意していただいているのでこちらがマージされ次第って感じでも大丈夫そうですが…