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

추가된 페이지에 대한 코드분할 적용, 일부 Common 폴더 내 컴포넌트 위치 이동 및 리스트의 누락된 key 속성 추가 #34

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

youngwan2
Copy link
Owner

@youngwan2 youngwan2 commented Jun 12, 2024

개요

일부 컴포넌트 폴더 이동

  • 기존 src/components/Common 내 존재했던 NaverDictionary 관련 컴포넌트 외 일부 컴포넌트들 src/componets 위치로 이동 함. 당초 Common 컴포넌트의 활용 목적은 모든 페이지에서 공통적으로 노출되는 컴포넌트를 보관하는 용도였으나, Input, Container, Label 등과 같이 특정 요소나 집합의 컨테이너 역할을 하는 요소를 보관하는 용도로 수정함. 따라서 변경된 용도와 연관성이 떨어지면서도 독립적인 역할을 하는 컴포넌트들을 상위 폴더로 이동

누락된 key 속성 추가

  • 기존에 Nutrition 페이지와 NaverDictionary 페이지 내에서 목록을 렌더링 하는 컴포넌트 내부에서 key 가 유니크 하지 않다는 경고문구가 발생함. key 속성을 각 목록을 나타내는 HTML 요소 내부에 직접 표기하기 보다 <Card key={card.id}/> 형식으로 지정한 것이 문제였음. 따라서 Card 컴포넌트 자체가 아닌 그 내부에 구성된 요소인 <div key={card.id}>~</div> 형식으로 키를 추가하여 개선

새로 추가된 페이지에 대한 코드 분리 적용

  • 예전에 빌드 파일의 크기가 너무 커서, 이 문제를 개선하고자 코드 분리를 적용했었음. 하지만 2024년 5월 24일 이후로 새롭게 추가된 페이지의 경우 해당 코드 분리 적용이 되어 있지 않아서 빌드 시 파일 크기가 커지는 문제가 발생하였고, 해당 문제를 개선하고자 나머지 추가 페이지에 대해서도 코드 분리를 적용
  • 추가로 import 시 절대 경로로 지정한 @/ 으로 상대 경로를 대체함(일관성을 위해)

@youngwan2 youngwan2 self-assigned this Jun 12, 2024
@youngwan2 youngwan2 added the Function improvement Improvements in functionality label Jun 12, 2024
@youngwan2 youngwan2 merged commit d5dd200 into dev Jun 12, 2024
@youngwan2 youngwan2 deleted the maintenance branch June 12, 2024 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Function improvement Improvements in functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant