[UI/#216] 취향 태그 뷰 / UI 구현 #224
Conversation
…into ui/#216-preference-tag
| private fun setResultTextColor() { | ||
| binding.tvCheckFriendsResult.apply { | ||
| text = SpannableStringBuilder(text).apply { | ||
| setSpan( | ||
| ForegroundColorSpan( | ||
| colorOf(R.color.red_500) | ||
| ), 0, 9, Spanned.SPAN_EXCLUSIVE_EXCLUSIVE |
There was a problem hiding this comment.
요부분으로 프로그래스바 색 다르게 구현하는군요!!!
역시 오비 이유빈.. 다르다..👍👍
There was a problem hiding this comment.
엇 요 부분은 성향 검사 결과 Text의 색깔을 다르게 구현하는 함수입니다!
프로그레스바 색상은 layer_list_trip_friends_progress_bar.xml파일에 미리 만들어두었어요!
추후에 서버에서 값 넘겨주면 setProgressBarStatus() 함수에서 값만 쇽샥 넣어줄 예정입니당
There was a problem hiding this comment.
아!! 요부분은 텍스트 색깔이군엽 설명 감사합니당!
| <ProgressBar | ||
| android:id="@+id/progress_bar_check_friends_1_revert" | ||
| style="@style/Widget.AppCompat.ProgressBar.Horizontal" | ||
| <LinearLayout |
There was a problem hiding this comment.
내부 코드들이 너무 중복이 많은데 컴포넌트화 해보는건 어떨까요??
제가 만든 것 처럼 커스텀 뷰를 만들어도 되지만, 이건 내부 연산이 많이 필요하지 않아 보이네요
단순 화면만 중복되게 만드는 용도라면 include를 사용서 만들어보면 좋을 것 같습니다!
물론 스프린트 끝나고 리펙토링때 시도해봐도 좋아요 :)
There was a problem hiding this comment.
오 이런 기능도 있군요!! 넵 리팩토링 기간 때 야무지게 고쳐보겠습니닷
| android:background="@drawable/shape_rect_6_gray50_fill" | ||
| android:gravity="center" | ||
| android:paddingVertical="14dp" | ||
| android:text="식당, 여행 계획 취향이 잘 맞는 조합이네요!" |
There was a problem hiding this comment.
해당 text는 추후 서버에서 받아오는 값으로 변경 될 값으로 보입니다.
서버 통신이 늦어질 경우 여기에 있는 값이 화면에 나와있다가 갑자기 값이 변경되는 것 처럼 사용자에게 보여질 염려가 있습니다!
그래서 tools에 있는 text를 사용한다면, xml 화면에는 보이지만 앱에서는 보이지 않기 때문에 tools를 사용하는 것을 저는 더 선호합니다!
There was a problem hiding this comment.
현재 해당 뷰의 text 색깔을 고치는 로직이 있어 tools로 바꾸면 에러가 발생하는 것 같습니다..!
추후에 서버 통신할 때 tools로 다시 바꿔놓겠습니다!!
…into ui/#216-preference-tag
| @@ -1,21 +1,25 @@ | |||
| <layer-list xmlns:android="http://schemas.android.com/apk/res/android"> | |||
| <shape xmlns:android="http://schemas.android.com/apk/res/android" | ||
| android:shape="rectangle"> | ||
| <solid android:color="@color/gray_50" /> | ||
| <stroke android:width="6.35dp" /> |
There was a problem hiding this comment.
6으로 하면 피그마랑 너무 달라지나요? 소수점 사용을 지양합시다 !
There was a problem hiding this comment.
피그마에서 디쟌선생님들이 6.35로 줘서 그대로 반영했었습니닷,, 사실 6으로 해도 상관 없을 것 같아서 6으로 수정하겠습니다!
⛳️ Work Description
📸 Screenshot
📢 To Reviewers
위 사진은 예시입니당
아직 서버 연동이 안 돼서 프로그레스바 색상이 하나만 뜰 거예요!
여기서 "철저하게"랑 "1명"의 폰트랑 색상이 다 다른데, 하나의 TextView로 했을 때
Spannable을 각각 15번 사용해주는 것보다 두 개의 TextView로 만들어주는 게 좋을 것 같아서 그렇게 했습니닷