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

[FE] StepList에 isFixed 디자인시스템 반영 #465

Merged
merged 3 commits into from
Aug 21, 2024
Merged

Conversation

Todari
Copy link
Contributor

@Todari Todari commented Aug 21, 2024

issue

구현 사항

현재 지출 내역에 참여 인원별 차등적용되었는지, 그렇지 않은지 확인하는 방법이 없었습니다.
차등계산이 적용된 지출 내역의 경우 별표 표시가 됩니다.
image
image

논의하고 싶은 부분(선택)

전에 얘기했던 변경에 유연하게 대응하는 방법에서 얘기했던것 중에
저는 제일 큰게 역할분리라고 생각해요~~ 컴포넌트가 기능적으로 몇개의 역할을 하느냐 도 있겠지만,
하나의 컴포넌트 안에서도 디자인만 연관있는 부분, 데이터를 보여주는 부분, 기능을 수행하는 부분을 구분해 놓아야 된다고 생각합니다.

그리고 그 중 디자인만 연관있는 부분을 디자인 시스템에 위임하려고 했던 거였어요.
전에도 한번 말했지만, 디자인시스템에서 데이터나 기능에 대한 부분이 많이 들어있는것같아서
조금씩 조금씩 바쁘지 않을 때 리팩토링 해나가려는 생각입니다.

제가 원하는 이상적인 component는
디자인은 HDesign에서 가져와서 처리.
데이터는 react-query 및 zustand에서 가져와서 디자인시스템에 주입
기능은 custom Hook에 data을 넘겨줘서 처리
정도일 것 같아요~!~!
``
(물론 변경에 따른 client 코드도 적용할거에요!!)

🫡 참고사항

@Todari Todari added this to the lev3 milestone Aug 21, 2024
@Todari Todari self-assigned this Aug 21, 2024
Copy link
Contributor

@jinhokim98 jinhokim98 left a comment

Choose a reason for hiding this comment

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

낮엔 예비군 밤엔 개발 정말 고생 많습니다!
맞아요 진짜 디자인 데이터 기능 3가지 별로 나눠서 리팩토링 하는 것 레벨 4때 가져가봐요

hasDragHandler={isAdmin}
// TODO: (@todari) dnd 없으므로 false
// hasDragHandler={isAdmin}
hasDragHandler={false}
Copy link
Contributor

Choose a reason for hiding this comment

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

dnd 없어서 false 처리까지 세세!

prefix={action.name}
suffix={`${action.price.toLocaleString('ko-kr')} 원`}
backgroundColor="lightGrayContainer"
onClick={() => handleDragHandleItemClick(index)}
isFixed={action.isFixed}
Copy link
Contributor

Choose a reason for hiding this comment

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

good!

Copy link
Contributor

@soi-ha soi-ha left a comment

Choose a reason for hiding this comment

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

밤에도 일하느라 고생많아요 토다리!
빠르게 반영해줘서 고마워용

Copy link
Contributor

@pakxe pakxe left a comment

Choose a reason for hiding this comment

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

스피디 반영 고맙숨니다~!!!

@pakxe pakxe merged commit af39b52 into fe-dev Aug 21, 2024
@pakxe pakxe deleted the feature/#436 branch August 21, 2024 15:51
@Todari Todari linked an issue Aug 22, 2024 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[FE] StepList에 isFixed 디자인시스템 반영
4 participants