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

DataViewBasicInfoPanel.vue を削除 #5656

Conversation

shgtkshruch
Copy link
Contributor

👏 解決する issue / Resolved Issues

📝 関連する issue / Related Issues

⛏ 変更内容 / Details of Changes

  • DataViewBasicInfoPanel.vue コンポーネントを削除して、DataViewDataSetPanel.vue に統合しました
  • DataViewDataSetPanel.vue の CSS のクラス名が長くなっていたので、シンプルにしました
  • テンプレートの記述で冗長な箇所があったので、v-for で置き換えました

📸 スクリーンショット / Screenshots

before after
スクリーンショット 2020-11-07 20 51 34 スクリーンショット 2020-11-07 20 51 47
スクリーンショット 2020-11-07 20 52 07 スクリーンショット 2020-11-07 20 52 12

Copy link
Collaborator

@kaizumaki kaizumaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!LGTM!

@kaizumaki kaizumaki added the ready-for-merge コードレビューが終わり、マージができるもの label Nov 19, 2020
@soutaito soutaito merged commit 765f7d4 into Tokyo-Metro-Gov:development Nov 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-merge コードレビューが終わり、マージができるもの
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DataView***Panel コンポーネントを整理する
3 participants