Open
Conversation
KWY0218
requested changes
Dec 1, 2022
Contributor
KWY0218
left a comment
KWY0218
left a comment
There was a problem hiding this comment.
작업하시느라 수고하셨습니다.
번거롭겠지만 fragment_train 에서 리니어 레이아웃 사용한 것을
전부 constraint layout 방식으로 바꿔 보는 것을 추천합니다..
이미지를 보면 에뮬레이터 상에선 글자들이 원하는 대로 있는 것을 확인할 수 있습니다.
하지만 다른 기종으로 이 앱을 돌리면 무조건 다르게 나올 것입니다.
- height, width에 고정 값을 가능한 두면 안됩니다.
sp같은 경우 사용자의 폰트 크기에 비례하기 때문에 사용자가 글자 크기를 키운다면 여기서height = 29dp를 줬기 때문에 글자가 잘려서 나올 것 입니다.spdp차이에 대해 찾아보는 것을 추천합니다
- margin 값 할당이 적절하지 않습니다
- 현재 tool bar 내 TextView를 예시로 들면
marginStart = 150dp,marginTop=58dp...
이런 식으로 값을 할당 중입니다.
이 에뮬레이터에서만 textview가 중앙에 있지, 에뮬과 width가 다른 기기로 돌리면
해당 TextView는 중앙이 아닌 위치에 있을 것이 명백합니다. - 그래서 linear layout 을 사용할 것이면
gravity속성을 사용해서 위젯들을 배치하는 것이 좋지만 저는 차라리 constraint layout 에 대해 공부해서 제약을 거는 법을 공부하는 것을 추천드립니다..!
이전 피알에서 리뷰 했어야 했는데, 이제서야 리뷰 드린 점 죄송합니다...!
| <string name="drawer_text2">통신판매업신고 대전동구-0233호</string> | ||
| <string name="drawer_text3">고객센터 1544-7788</string> | ||
| <string name="drawer_text4">개인정보처리방침 | 여객 운송약관 한국철도공사</string> | ||
| <string name="drawer_text4">개인정보처리방침 | 여객 운송약관 한국철도공사</string> |
Contributor
There was a problem hiding this comment.
Suggested change
| <string name="drawer_text4">개인정보처리방침 | 여객 운송약관 한국철도공사</string> | |
| <string name="drawer_text4">개인정보처리방침 | 여객 운송약관\n한국철도공사</string> |
줄바꿈을 위해서 문자열 공백을 준거라면 \n 을 이용해서 줄바꿈 효과를 줄 수 있습니다
Member
Author
There was a problem hiding this comment.
네 알려주셔서 감사합니다. 더 공부한 후 추후에 따로 고쳐보겠습니다. 지금까지 리뷰 꼼꼼히 달아주셔서 감사합니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
관련 이슈
작업한 내용
-예약하기 버튼 연결
-drawer뜨게 하는 버튼 연결
-drawer 프래그먼트에서 액티비티로 변경
-서버통신 성공
PR 포인트