My page skeleton/#102

[gahyuun]

📌 작업 내용

구현 내용 및 작업 했던 내역

• 지난번에 작업하지 못했던 마이페이지 스켈레톤 작업했습니다!
  스켈레톤이 어색하지 않을까 걱정했는데 괜찮은 것 같습니다

🤔 고민 했던 부분

• headerInfo 가 skeleton , taken lecture에서 둘 다 사용이 되었습니다 utils에 constants라는 파일을 만들까 고민을 했었지만, ui component에서만 사용될 변수라고 판단되어서 ui 폴더 내 taken lecture 폴더에 위치시켰습니다.

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-zpssltxfpk.chromatic.com/

[seonghunYang]

• 수고하셨습니다
• 사진으로 봤을 때 좋아 보이네요

[yougyung]

수고하셨습니다

[yougyung]

파일 명이 기존 네이밍과 조금 다른 것 같습니다 통일이 필요할 것 같아요

[gahyuun]

기존 네이밍이 어떤 거 말씀하시는 지 알려주실 수 있나요?

[yougyung]

taken-lecture.skeleton.tsx 형식을 의미하는 것 입니다

[gahyuun]

아하 다른 분들은 taken-lecture.skeleton.tsx 형식으로 하셨네요! 이제 봤습니다 혹시 그렇다면 .skeleton.tsx로 네이밍을 지으신 이유가 있으실까요?? skeleton은 atom 공통 컴포넌트인데 .business.ts나 .type.ts 처럼 네이밍을 지을 필요가 있나 싶네요! 저희가 result.dialog.tsx로 네이밍을 짓지 않고 result-dialog.tsx 로 짓는 것처럼요!

[yougyung]

저는 기본적으로 네이밍은 저희 팀 안에서만 합의되면 되는 부분이라고 생각을 하는데요. 제가 skeleton을 개발할 당시에는 이미 해당 네이밍으로 사용되고 있었으며, 해당 네이밍 별도로 부자연스럽다는 생각을 하지 않아서 저도 동일하게 사용한 것으로 기억이 납니다. 해서 성훈 선배의 의견도 궁금하네요.

제가 부자연스럽다고 생각하지 않았던 이유는 taken-lecture의 skeleton을 해당 컴포넌트가 표현하기 때문입니다. taken-lecture의 storybook을 taken-lecture.stories표기법으로 작성하듯 taken-lecture에 대한 skeleton component라고 생각해서 해당 네이밍이 자연스럽게 와닿았던 것 같네요.

result-dialog.tsx은 result의 dialog로서의 의미보다는 content를 담당하는 component이기 때문에 동일한 대상으로 보기는 어렵다는 생각이 드는데, 어떻게 생각하시나요?

[gahyuun]

저는 skeleton을 단순히 컴포넌트로만 보고 , 언니는 skeleton을 storybook, business, command 처럼 어떤 것을 표현하기 위한 종류나 기능이라고 생각했던 것 같네요!

dialog가 dialog 자체의 역할은 강하지 않은데 이에 비해 skeleton은 skeleton 자체 역할이 강하다는 말씀 저도 동의합니다! 그러한 이유로 .skeleton.tsx를 사용할 거라면 atom 폴더가 아닌 util 폴더가 좀 더 적합하지 않을까하네요 이에 대해서는 혹시 어떻게 생각하시나요?

[yougyung]

기존의 Skeleton atom component가 util 폴더로의 이동을 통해서 도구적 차원의 의미를 더 강하게 나타낼 수 있는 것 같다는 생각이 드네요. 좋습니다.

[gahyuun]

638ecf1
반영했습니다

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-etzrwdacsm.chromatic.com/

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-zccxjicpzi.chromatic.com/

[seonghunYang]

• skeleton이 atom에 존재하는 뷰 컴포넌트라면, 가현이가 말한 것처럼 taken-lecture-skeleton이 적합해 보입니다. 그러나 util에 존재한다면, taken-lecture.skeleton이 더 자연스러울 것 같습니다. 두 분이서 의미적으로 문제가 없도록 잘 합의하신 것 같습니다.