Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

스타일 가이드 - 베이스 컴포넌트 이름 #25 - (draft 상태로 처리) #60

Merged
merged 2 commits into from
Nov 16, 2022

Conversation

Mccreate
Copy link
Collaborator

제목

스타일 가이드 - 베이스 컴포넌트 이름 #25

동기

미번역 작업 번역 실시

기타

코멘트 남겨둔 부분은 번역 미흡으로 인해 좀 더 매끄럽게 진행할 수 있도록 도와주세요😢

Check list

  • PR 템플릿의 예시 문장을 지웠나요?
  • 번역 후 터미널에서 npm run dev를 사용해 번역(수정)한 부분을 브라우저로 확인했나요?


Their names often include the name of an element they wrap (e.g. `BaseButton`, `BaseTable`), unless no element exists for their specific purpose (e.g. `BaseIcon`). If you build similar components for a more specific context, they will almost always consume these components (e.g. `BaseButton` may be used in `ButtonSubmit`).
컴포넌트 이름에는 특정한 목적(e.g. `BaseIcon`)을 위한 요소가 있는 경우를 제외하고, `BaseButton`, `BaseTable` 같이 컴포넌트가 해당 요소를 감싸는 요소의 이름이 포함되는 경우가 많습니다. 만약 당신이 좀 더 구체적인 컨텍스트에 대해 비슷한 컴포넌트를 구성하는 경우, 컴포넌트들이 거의 항상 이 요소들을 소비하게 될 겁니다. 예를 들면, `BaseButton`이 `BaseSumbit`에서 쓰이듯이 말이죠.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

If you build similar components for a more specific context, they will almost always consume these components (e.g. BaseButton may be used in ButtonSubmit).

만약 당신이 좀 더 구체적인 컨텍스트에 대해 비슷한 컴포넌트를 구성하는 경우, 컴포넌트들이 거의 항상 이 요소들을 소비하게 될 겁니다. 예를 들면, BaseButtonBaseSumbit에서 쓰이듯이 말이죠.

@Mccreate
Copy link
Collaborator Author

#58 에서 해당 PR 새로 시작.

@Mccreate Mccreate marked this pull request as ready for review November 16, 2022 23:51
@Mccreate Mccreate merged commit c88510f into ssafy-vue:master Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant