Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

新グラフ「新宿区エリアの来訪者推移」にnoscript時の処理を加える #1945

Closed
kaizumaki opened this issue Mar 20, 2020 · 5 comments · Fixed by #1969
Closed
Assignees
Labels
help-wanted 特に助けを必要としているもの IMPORTANT Issue の中でも特に重要なもの improvement 改善や新機能の要望

Comments

@kaizumaki
Copy link
Collaborator

改善詳細 / Details of Improvement

スクリーンショット / Screenshot

こちらのグラフです(javascript許可時)。
スクリーンショット 2020-03-20 21 41 31

期待する見せ方・挙動 / Expected behavior

例えばこんなふうに、noscript(javascriptブロック時)には表になっているのが期待する動作です。
スクリーンショット 2020-03-20 21 42 19

@kaizumaki kaizumaki added improvement 改善や新機能の要望 help-wanted 特に助けを必要としているもの IMPORTANT Issue の中でも特に重要なもの labels Mar 20, 2020
@t-tajiri
Copy link
Contributor

私でよろしければ対応させてください。

@kaizumaki
Copy link
Collaborator Author

@t-tajiri お願いします!

@KentarouTakeda
Copy link
Contributor

レス書いてたらタッチの差で先越されましたw

@t-tajiri
大本のnoscript実装は私が書いたので分かりづらい箇所は聞いて下さい!
おかしな実装があったらご遠慮無く変えちゃってください!笑

@t-tajiri
Copy link
Contributor

@KentarouTakeda
お気遣いありがとうございます!

早速質問させてください!
@KentarouTakeda さんの実装を参考にしてみましたが、表のヘッダーがnoscript時のメッセージより上に出力されています。これは実装が間違っていますかね。。。?
具体的には、下のスクリーンショットのようになってしまいます。

sheet

@KentarouTakeda
Copy link
Contributor

@t-tajiri 着手ありがとうございます。ていうか仕事早いw

「noscriptの際のv-tableのの実装」単独としては間違ってない(今の状態で良い)と思います。

ご質問の表示崩れは、 Issue にはまだ無いのですが、もっと前から存在していた「陽性患者の属性」の表でも発生していた(おそらく v-table 全てで発生する)ので、仮に直す場合「表のバグとして v-table を全箇所修正する」ではなく「モーダルウインドウの表示バグとして別件で修正する」が自然な気がします。(と思いそのままにしてしまってました 汗汗

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help-wanted 特に助けを必要としているもの IMPORTANT Issue の中でも特に重要なもの improvement 改善や新機能の要望
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants