money-fragments / money-fragments-client

https://money-fragments-client.vercel.app
2 stars 1 forks source link

fix : black과 white부분 theme에 추가하여 배경 색상 사용 시 자동완성으로 사용 가능. #24

Closed KoreaMoney closed 1 year ago

KoreaMoney commented 1 year ago

What is this PR? :mag:

-mono부분에서 B와 W모두가 생성되어 자동 완성 시 모두 사용이 가능합니다.

branch

Changes :memo:

-mono부분에서 B만 있고 W가 없어서 생성하였습니다.

by @dollorofkorea

KoreaMoney commented 1 year ago

그럼 색깔 구현할때 필요한 monoW 색은 임의적으로 색깔을 줘서 구분할까요?

Jeremy-Kr commented 1 year ago

그럼 balck과 white로 구분하는건 어떨까요? mono는 다크모드... 를 하게되면 사용해야 할 것 같아서요!

KoreaMoney commented 1 year ago

앗!! 넵 그러면 white와 black으로 심플하게 구분해서 ui제작 하겠습니다!! 그럼 저 PR은 삭제 부탁드립니다. 제가 삭제 할 수 있나요?

Jeremy-Kr commented 1 year ago

PR 삭제하기 보다 요 브랜치에 그냥 새로 커밋 날려주시고 리뷰 리퀘스트 다시 주심 좋을 것 같아요

KoreaMoney commented 1 year ago

넵. 그럼 다시 원복해서 커밋하고 PR하겠습니다.

KoreaMoney commented 1 year ago

What is this PR? 🔍

-mono부분 다시 원복하여 기존과 동일하게 사용가능합니다.

branch

  • fix/theme.ts -> develop

Changes 📝

-mono부분에서 B부분 삭제, w부분 삭제하여 원복진행함.

by @dollorofkorea

KoreaMoney commented 1 year ago

@Jeremy-Kr 원복하였습니다!

Jeremy-Kr commented 1 year ago

엇.... 제말은 black과 white를 추가하면 어떨까 였는데 죄송해요 제가 말을 자세히 안했네요... ㅜㅜ

KoreaMoney commented 1 year ago

제가 이해한건 white랑 black을 사용하라고 이해했네요. .제가 제대로 이해를 못했군요. ㅜㅜ 죄송합니다. 제가 다시 수정해서 commit하겠습니다!

KoreaMoney commented 1 year ago

What is this PR? 🔍

-Black과 white를 추가하여 앞으로 mono와 구분해서 색상을 사용할 수 있습니다.

branch

  • fix/theme.ts -> develop

Changes 📝

black과 white색상 추가하였습니다.

by @dollorofkorea

Jeremy-Kr commented 1 year ago

고생하셨습니다~! 다른분들 리뷰 끝나시거나, 내일 정오 이전까지 리뷰 안끝나면 내일 정오에 머지 진행하겠습니다~!