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

Jinn2u/hex colors gradient #4

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

jinn2u
Copy link
Contributor

@jinn2u jinn2u commented Feb 2, 2022

리뷰 시 참고사항

  1. 제가 실수로 커밋로그를 남기지 않았습니다 ㅎㅎ;; 다음부터는 세세히 나누겠읍니다,,
  2. 버튼이 누르면 랜덤한 두 색상이 나오는데,
    1. 버튼을 눌렀을 때 콜백함수에서 ColorSpan의 setState를 하는 방법
    2. 버튼을 눌렀을 때 콜백함수에서 App의 상태를 바꾸고 App의 setState에서 ColorSpan의 상태를 바꾸는 방법
      저는 이 둘 중 첫번째로 했습니다. 이유는 App에 color에 대한 상태를 저장할 이유가 있을까? 라는 생각이 들었습니다.
      다만 이렇게 되면 버튼과 ColorSpan의 의존성이 강하게 생기는게 아닌지 궁금합니다. 어떤게 더 나아보이나요??

실행 화면

@jinn2u jinn2u self-assigned this Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant