-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: add review item responsive and color #59
Conversation
90dc27d
to
fdf9a80
Compare
components/review-item.js
Outdated
connectedCallback() { | ||
this.attachShadow({ mode: "open" }); | ||
this.render(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 패턴은 쓰지 않기로 한 것 같은데, 아직 작업 중이신거죠? 아직 PR이 Ready for review가 아니시라면 Draft 상태로 변경해주시면 혼선이 적을 것 같습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DaleSeo
아, 이 pr이 저희 이번에 결정나기 전에 지난 이터레이션에 올렸던 review item의 responsive pr이었는데, 지금까지 approve가 안되어서 혼선을 드린 것 같습니다!!
이런 경우에는, 제가 draft로 돌리고 이번에 바뀐 패턴으로 적용해서 pr을 올리는게 나을까요?
어차피 지금 하고 있는 step 컴포넌트 작업이 끝나면 review item 부터 바뀐 패턴으로 수정하려던 참이었거든요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
바뀐 컴포넌트 패턴으로 변경 완료하였습니다! 컬러까지 적용할 예정이어서, draft로 돌린 점도 참고 부탁드립니다!
color 까지 적용완료 되었습니다. |
리뷰를 반영하여, 불필요한 미디어쿼리 삭제 및 네이밍 업데이트 완료하였습니다. |
Checklist before merging