[UI/#9] 취향 태그 뷰 / UI 구현 #24

leeeyubin · 2024-01-04T16:19:40Z

closed [UI] 취향 태그 뷰 / UI 구현 #9

⛳️ Work Description

어댑터 구현 (List Adapter)
Scroll View 구현
버튼 중복 클릭 방지 구현

📸 Screenshot

device-2024-01-06-004809.mp4

📢 To Reviewers

아직 디자인이 다 나오지 않아서 색상이나 폰트는 임시로 설정해뒀습니다!
피드백 언제나 환영입니당

버튼 눌렀을 때 텍스트 색상도 바뀌게 다시 수정했습니다!

Marchbreeze

고생했어요~! 첫 PR인데 야무지네용 ㅎㅎ
다만 레이아웃 내에 고정값으로 박힌 크기들이 많이 보이네요 ㅜㅜ 이 친구들이랑 그래들 부분 수정 부탁드려욥

Marchbreeze · 2024-01-04T16:29:00Z

app/src/main/AndroidManifest.xml

+        <activity
+            android:name="com.going.presentation.preferencetag.PreferenceTagActivity"
+            android:exported="true" />


요기 다른 애들처럼 화면회전 막아주시고, pr 올릴때는 exported false로 해서 올려주셍요 ~

Marchbreeze · 2024-01-04T16:29:29Z

build.gradle.kts

@@ -9,6 +9,7 @@ buildscript {
        classpath(ClassPathPlugins.kotlinGradle)
        classpath(ClassPathPlugins.hilt)
        classpath(ClassPathPlugins.oss)
+        classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.8.20")


엇 요친구 없으면 안돌아가나요? 이미 들어가있는것처럼 보이는데!

빼도 잘 돌아가네용 코끼리와 친해지기,,메모,,,

Marchbreeze · 2024-01-04T16:29:50Z

presentation/build.gradle.kts

+    implementation("androidx.appcompat:appcompat:1.6.1")
+    implementation("com.google.android.material:material:1.5.0")
+    implementation("androidx.constraintlayout:constraintlayout:2.1.4")


요 친구들도 이미 들어가있어 보이는데 확인해주세요 ~

Marchbreeze · 2024-01-04T16:30:53Z

presentation/src/main/java/com/going/presentation/preferencetag/PreferenceTagActivity.kt

+    private var _adapter: PreferenceTagAdapter? = null
+    private val adapter get() = requireNotNull(_adapter) { getString(R.string.adapter_not_initialized_error_msg) }
+    private val viewModel by viewModels<PreferenceTagViewModel>()


사소한거긴 한데, 저는 용도가 다른 변수들 사이에는 한줄 띄워두어용 ! 지금 보면 15,16 / 17 느낌으로?

사소한 피드백 좋습니다!

Marchbreeze · 2024-01-04T16:31:14Z

presentation/src/main/java/com/going/presentation/preferencetag/PreferenceTagAdapter.kt

+    context: Context,
+    private val listener: OnPreferenceSelectedListener
+) :
+    ListAdapter<PreferenceData, PreferenceTagViewHolder>(diffUtil) {


히야 굿굿 !

Marchbreeze · 2024-01-04T16:37:06Z

presentation/src/main/res/layout/item_preference_tag.xml

+                android:layout_width="48dp"
+                android:layout_height="48dp"


라디오 버튼들도 크기를 고정해두면 안될 것 같아요 ! 5개를 chain을 걸어서 packed 상태로 두던가, 아니면 weight를 활용하던가 해서 반응성 있는 레이아웃으로 구현해주세요 ~

Marchbreeze · 2024-01-04T16:38:08Z

presentation/src/main/res/values/themes.xml

+        <item name="android:layout_width">48dp</item>
+        <item name="android:layout_height">48dp</item>


여기도 고정해두면 안될 것 같아요 ~~

Marchbreeze · 2024-01-04T16:39:48Z

presentation/src/main/res/drawable/ic_line_black.xml

+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="177dp"
+    android:height="2dp"
+    android:viewportWidth="177"
+    android:viewportHeight="2">
+  <path
+      android:pathData="M0,1L177,1"
+      android:strokeWidth="0.5"
+      android:fillColor="#00000000"
+      android:strokeColor="#000000"/>
+</vector>


선을 이미지로 활용하기 보다는, View의 색상을 검정색으로 두고 고정 height를 줘서 선처럼 보이도록 하는 방법, Material Divider를 활용하는 방법 중에 선택해서 활용하는게 더 바람직해보입니당

Marchbreeze · 2024-01-04T16:41:31Z

presentation/src/main/res/layout/activity_preference_tag.xml

+            android:id="@+id/btn_preference_start"
+            style="@style/button_style"
+            android:layout_width="0dp"
+            android:layout_height="50dp"


여기도 wrap_content와 padding 값으로 구현해봅시다 ~
만약 내부 텍스트가 엄청 커지면 아마 잘릴거에요

Marchbreeze · 2024-01-04T16:41:50Z

presentation/src/main/res/layout/item_preference_tag.xml

+            android:layout_width="24dp"
+            android:layout_height="24dp"


여기도 고정 dp 금지 ~

chattymin

코드 완전 잘 보고갑니다!!!
너무 잘했구 고생한게 많이 보여서 고맙네욤 ㅎㅎ

chattymin · 2024-01-04T18:06:06Z

presentation/src/main/java/com/going/presentation/preferencetag/PreferenceTagViewHolder.kt

+            rgPreferenceTag.setOnCheckedChangeListener { group, checkedId ->
+                val selectedButtonIdList = listOf(
+                    R.id.rb_preference_1,
+                    R.id.rb_preference_2,
+                    R.id.rb_preference_3,
+                    R.id.rb_preference_4,
+                    R.id.rb_preference_5
+                )
+
+                if (checkedId in selectedButtonIdList) {
+                    listener.onPreferenceSelected(item)
+                }
+            }


group의 경우 사용하지 않는 변수기때문에 _ 로 바꾸면 좋을 것 같습니다!
아마 그렇게 하라고 가이드도 뜰거에요 :)

오호 좋은 정보 감사합니당

chattymin · 2024-01-04T18:14:37Z

presentation/src/main/res/drawable/sel_radiobutton_text_color.xml

+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+    <item android:color="@color/white_000" android:state_checked="true" />
+    <item android:color="@color/black_000" />
+</selector>


개인적인 생각이긴 하지만 저는 checked="false" 도 써두는게 명확하게 표시할 수 있어서 좋다고 생각해요!
true일때는 이거, false일때는 저거 이렇게 표기돼있는게 더 명확하고 한눈에 볼 수 있다고 생각합니당 :)

Marchbreeze

코리 대응 야무지게 해주셔서 감사합니닷 :)

Marchbreeze · 2024-01-05T20:18:51Z

presentation/src/main/res/layout/item_preference_tag.xml

+            app:layout_constraintTop_toTopOf="parent"
+            tools:text="관광" />
+
+        <RadioGroup


여기 width에 chain 추후에 디자인 나온거 수정할때 꼭 걸어주세요~

Marchbreeze · 2024-01-05T20:19:19Z

presentation/src/main/res/layout/item_preference_tag.xml

+            app:layout_constraintTop_toTopOf="parent"
+            tools:text="휴식" />
+
+        <TextView


요거 그냥 view로 바꾸고 tv 대신 view_ 로 가도 될거가타용 ~

Marchbreeze · 2024-01-05T20:20:10Z

presentation/src/main/res/layout/item_preference_tag.xml

+            android:layout_marginStart="31dp"
+            android:layout_marginTop="27dp"
+            android:fontFamily="@font/pretendard_bold"
+            android:textColor="@color/black_000"


요친구들도 디잔대로 바꿀때 textView의 style로 적용시켜주세요~

leeeyubin added 13 commits December 31, 2023 19:33

[ADD/#9] 취향 태그 뷰 파일 추가

f55ffc8

[ADD/#9] 아이템 파일 및 리소스 추가

eed05db

[UI/#9] 아이템 UI 구현

3ced80c

[UI/#9] 리사이클러뷰 구현

ba55740

[ADD/#9] 리스트 어댑터 파일 추가

e16d065

[FEAT/#9] 리스트 어댑터 구현

910e515

[MOD/#9] 파일 이름 변경

57d3877

[MOD/#9] 파일 이름 변경

6658c4a

[FEAT/#9] 리스트 어댑터 연결

6e604ae

[FEAT/#9] 버튼 클릭 리스너 구현

40ab234

[UI/#9] RadioButton UI 변경

6a9883c

[FEAT/#9] 중복 클릭 방지 구현

da52f03

[UI/#9] 버튼 마진값 수정

4948b47

leeeyubin added 유빈 ❄ UI 💐 UI 작업 labels Jan 4, 2024

leeeyubin added this to the 1차 스프린트 UI 작업 milestone Jan 4, 2024

leeeyubin requested review from chattymin, Marchbreeze and crownjoe January 4, 2024 16:19

leeeyubin self-assigned this Jan 4, 2024

leeeyubin added 2 commits January 5, 2024 01:38

[UI/#9] 버튼 클릭 시 색상 수정

295d451

[MOD/#9] Manifest 수정

664de4f

Marchbreeze requested changes Jan 4, 2024

View reviewed changes

leeeyubin added 2 commits January 5, 2024 02:49

[MOD/#9] 코드리뷰 반영 (그래들, 매니페스트, 리스너)

dffb7a1

[MOD/#9] 코드리뷰 반영 (리스트 어댑터 아이템 UI)

0bf16fa

chattymin reviewed Jan 4, 2024

View reviewed changes

leeeyubin added 2 commits January 6, 2024 00:49

[FIX/#9] 고정 값 고치기

7f30678

[FIX/#9] 고정 값 고치기

8b8c792

Marchbreeze approved these changes Jan 5, 2024

View reviewed changes

[FIX/#9] View로 고치기

8604eae

leeeyubin merged commit 3aee068 into develop Jan 6, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[UI/#9] 취향 태그 뷰 / UI 구현 #24

[UI/#9] 취향 태그 뷰 / UI 구현 #24

leeeyubin commented Jan 4, 2024 •

edited

Loading

Marchbreeze left a comment •

edited

Loading

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

leeeyubin Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

leeeyubin Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

Marchbreeze Jan 4, 2024

chattymin left a comment

chattymin Jan 4, 2024

leeeyubin Jan 5, 2024

chattymin Jan 4, 2024

Marchbreeze left a comment

Marchbreeze Jan 5, 2024

Marchbreeze Jan 5, 2024

Marchbreeze Jan 5, 2024

		<item name="android:layout_width">48dp</item>
		<item name="android:layout_height">48dp</item>

[UI/#9] 취향 태그 뷰 / UI 구현 #24

[UI/#9] 취향 태그 뷰 / UI 구현 #24

Conversation

leeeyubin commented Jan 4, 2024 • edited Loading

⛳️ Work Description

📸 Screenshot

📢 To Reviewers

Marchbreeze left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

chattymin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Marchbreeze left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leeeyubin commented Jan 4, 2024 •

edited

Loading

Marchbreeze left a comment •

edited

Loading