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

[refactor] DataView.vue から SNS シェアに関するコードを分離する #4436

Closed
shgtkshruch opened this issue May 23, 2020 · 2 comments · Fixed by #4449
Closed

[refactor] DataView.vue から SNS シェアに関するコードを分離する #4436

shgtkshruch opened this issue May 23, 2020 · 2 comments · Fixed by #4449
Labels
improvement 改善や新機能の要望

Comments

@shgtkshruch
Copy link
Contributor

shgtkshruch commented May 23, 2020 •
edited
Loading

改善詳細 / Details of Improvement

  • DataView.vue の責務が大きい
    • チャートや説明文のレイアウト
    • チャートデータのテーブル
    • SNS シェア
  • SNS シェアに関するコードが半分程度を占めているので、これを別コンポーネントに切り出すと見通しが良くなりそう
    • template
    • data, computed, watch, methods
    • style

スクリーンショット / Screenshot

リファクタリングなので見た目の変更はありません。

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

  • DataView から SNS シェアに関するコードが分離されること

動作環境・ブラウザ / Environment

  • macOS / Windows / Linux / iOS / Android
  • Chrome / Safari / Firefox / Edge / Internet Explorer
@shgtkshruch shgtkshruch added the improvement 改善や新機能の要望 label May 23, 2020
@goki90210
Copy link
Contributor

#4176 の件と一緒にやれませんかね…。

@shgtkshruch
Copy link
Contributor Author

shgtkshruch commented May 23, 2020 •
edited
Loading

#4176 と一緒にやると影響範囲がかなり広がるので、別々に進めたほうがいいと思います。

@shgtkshruch shgtkshruch mentioned this issue May 23, 2020
Merged
This was referenced May 25, 2020
Merged
Closed
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
improvement 改善や新機能の要望
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[refactor] DataView.vue から SNS シェアに関するコードを分離 shgtkshruch/covid19
2 participants
@shgtkshruch @goki90210