[add/#5] 아이콘 추가 및 README 작성 #7
Conversation
dmp100
added
add ➕
vvan2
left a comment
vvan2
left a comment
There was a problem hiding this comment.
고생하셨으요!! icon 이랑 img 네이밍 잘해준것 같아요!
딱 봤을 때 역할을 알 수 있어서 좋은것 같아잉 굿굿
근데 button이나 box 안에 들어가는 아이콘은 추가 안된거 같은데 확인한 번 부탁드려요잉
There was a problem hiding this comment.
p2) profile icon 이 kid 랑 parent 구분되어서 사용되는 거니까
ic_kid_profile 이 더 좋을 것 같긴한디 어떻게 생각하시나요? 다른 아이콘 추가되면 ic_kid_<역할> 이런식으로
There was a problem hiding this comment.
음,, 개인적으로는 kid 라는 단어가 들어가는 icon들이 좀 많을것 같아서
차라리 쓰이는 뷰나 컴포넌트 기준으로 잡아서 사용하면 좋지않을까 해서 네이밍을 역할 먼저 적은거긴합니다 !!
다른 분들 의견도 한번 들어보고 만약 작업에서 kid로 시작하는게 더 구분이 좋을거같다하면 전체 수정하는건 어떨까용 ??
@seungjae708 @sonms
There was a problem hiding this comment.
규현님 방식은 UI 구조가 비슷하거나 똑같은데 이미지만 다르다 즉 컴포넌트 중심으로 생각하신 것 같아요!
하지만 저는 우리의 프로젝트 구조 상 아이 화면과 부모 화면이 거의 완전히 분리 되어 있어 대상 중심으로 생각되어 네이밍이 ic_kid_(역할) 형식이 되어야 나중에 검색하기 편할 것 같아요!
There was a problem hiding this comment.
컴포넌트로 분리되기 이전에 부모, 아이 화면으로 플로우가 나뉘기 때문에
저도 대상(kid와 parent)이 먼저 나타나는게 좋을 것 같다는 생각입니다!
| │ ├── parent/ auth와 동일한 구조 | ||
| │ └── di/ Data Layer DI 모듈 | ||
| │ | ||
| ├── domain |
There was a problem hiding this comment.
p3) 이거 내가 domain 빠진 버전으로 다시 노션에 올려줄게 ㅋㅋㅋ.. 까먹고 노션쪽에는 수정을 안했네..
seungjae708
left a comment
seungjae708
left a comment
There was a problem hiding this comment.
디자인 시스템에 나타나지 않은 아이콘들도 꼼꼼히 추가해주셨네요!! 고생많으셨습니다~
There was a problem hiding this comment.
컴포넌트로 분리되기 이전에 부모, 아이 화면으로 플로우가 나뉘기 때문에
저도 대상(kid와 parent)이 먼저 나타나는게 좋을 것 같다는 생각입니다!
sonms
left a comment
sonms
left a comment
There was a problem hiding this comment.
굿굿 고생하셨습니다!! 깔끔하니 좋네요 mermaid까지 굿ㅎㅎ
vvan2
changed the title
ISSUE
❗ WORK DESCRIPTION
app/src/main/res/drawable에 추가했고, 3D icon과 피아노 배경은 png로 저장했습니다.📢 TO REVIEWERS
ic_arrow_down.xml등등..)(
ic_addschdule_mission)아이콘 컨벤션 지킬려고 노력했는데 맞는지 모르겠네요.,, 직관적이지 않거나 컨벤션에 어긋나는 네이밍
혹은 빼먹은 아이콘 있으면 말씀해주시면 추가하거나 수정하겠습니다 !
그외
|
📸 SCREENSHOT