depromeet / imgoing-frontend

암고잉 | 나만의 약속 준비 루틴 관리 서비스
5 stars 0 forks source link

설정 서브 페이지 마크업 및 프로필 이미지 크기 수정 #200

Closed ykss closed 2 years ago

ykss commented 2 years ago

관련 이슈 연결

It will close #191 설정 서브 페이지 마크업 It will close #198 프로필 사진 이미지 크기 수정

변경/추가 사항, 자세한 설명

최종 결과물 스크린샷

image image

안드로이드 갤10 5G

ykss commented 2 years ago

정말 세세한 리뷰 감사합니다! 일단 수정사항 1차적으로 반영하였고, getBottomSpace() 대한 부분만 해결되면 될것같은데, 일단은 명시적으로 패딩값으로 주니까 아이폰 안드로이드 다 괜찮은 것 같아서 그렇게 적용해놨습니다!

ykss commented 2 years ago

수고하셨습니다☺️

아, 그리고 사진 붙이실 때 바로 붙이면 width가 100%가 되어버려서 가독성이 너무 안좋아지네요ㅠㅠ

<img src="hihi" width=300/>

이렇게 태그로 해서 width 조정해주시면 보기 더 편할 것 같습니다 ㅎㅎ

혹시 이건 어떤 부분 말씀하신건가요..!?

eun-seong commented 2 years ago

앗 PR이나 코멘트에 이미지 첨부할 때 말한거였어요! 지금 있는 pr에는 제가 수정해놨습니다..!

! [image] (url) 이 형태로 쓰면 사진이 너무 커져버려서요ㅠㅠ @eungyeole 은결님도 img 태그로 부탁드려용

ykss commented 2 years ago

앗 PR이나 코멘트에 이미지 첨부할 때 말한거였어요! 지금 있는 pr에는 제가 수정해놨습니다..!

! [image] (url) 이 형태로 쓰면 사진이 너무 커져버려서요ㅠㅠ @eungyeole 은결님도 img 태그로 부탁드려용

아 복붙으로 했엇어서 신경을 못썼었네요~~ 앞으로 크기 조절 하겠습니다! ㅎㅎ 이 PR은 이제 머지해도될까요?

eun-seong commented 2 years ago

네! 머지해도 될 것 같습니다☺️