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

component のディレクトリ構成を変更する #5327

Conversation

nard-tech
Copy link
Contributor

@nard-tech nard-tech commented Aug 22, 2020

@nard-tech nard-tech marked this pull request as draft August 22, 2020 08:32
@nard-tech nard-tech force-pushed the feature/#2790-change-directory-of-components branch from 7b3719d to 574b6c2 Compare August 22, 2020 09:11
@nard-tech nard-tech force-pushed the feature/#2790-change-directory-of-components branch 4 times, most recently from 59dcf4f to 9659942 Compare August 22, 2020 11:25
@munierujp
Copy link
Contributor

munierujp commented Aug 22, 2020

PrinterButton.vueは今はたまたま/flowでしか使われていないだけで、実際にはExternalLink.vueなどと同じくどこでも使用できる汎用的なコンポーネントのように見えます。

@nard-tech nard-tech force-pushed the feature/#2790-change-directory-of-components branch from 9659942 to 07002cf Compare August 23, 2020 08:44
@nard-tech
Copy link
Contributor Author

@munierujp

PrinterButton.vueは今はたまたま/flowでしか使われていないだけで、実際にはExternalLink.vueなどと同じくどこでも使用できる汎用的なコンポーネントのように見えます。

「どこでどのコンポーネントが使われているかを明確にする」というのがディレクトリ構成の変更の目的のひとつです.
現状,PrinterButton.vuepages/flow.vue でしか使われていないので,そのことがディレクトリ構成からも一目瞭然になる方が好ましいと思います.
「どこでも使用できる汎用的なコンポーネント」であるのは確かですが,というだからといって現時点でそのようなディレクトリに配置する必要はないと思います.
もともと「/flow のページを印刷したい」という要求から生まれたコンポーネントのはずですし,もし他のページでも使うことになったらそのときにディレクトリを変更すれば済むことです.

@nard-tech nard-tech force-pushed the feature/#2790-change-directory-of-components branch 2 times, most recently from 205c200 to 7d3ad6a Compare August 25, 2020 00:32
…s' to 'components/index/SiteTopUpper/WhatsNew'
…nents/index/CardsReference/ConfirmedCasesByMunicipalities/DataTable.vue'
…ponents/index/CardsReference/MonitoringConsultationDeskReportsNumber/Chart.vue'
…ts/index/CardsReference/PositiveNumberByDevelopedDate'
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants