Merged
Conversation
jeongkyueun
changed the title
Collaborator
There was a problem hiding this comment.
1주차 미션 완료하시느라 정말 수고 많으셨습니다! 😃 색상 리소스(colors.xml) 분리도 완벽하게 해주셨고, 이미지 파일명도 ic_ 접두어를 써서 아주 깔끔하게 잘 정리해주셨네요!
코드 전반적으로 훌륭하게 동작하지만, 더 완성도 높은 앱을 위해 몇 가지 개선하면 좋을 부분들을 적어두었습니다! 또한 사진같은 요소들을 올리실때에는 마크다운(Markdown)을 활용하면 훨씬 보기 좋게 작성할 수 있습니다!
ex) 파이프 기호(|)로 칸을 나누고 하이픈(---)으로 두 번째 줄을 구분해 주는 방식
나중에 pr제출하실때 사진이 많다면은 표로 정리해서 올려주세요!
Comment on lines
+24
to
+39
| val ivHappy = findViewById<ImageView>(R.id.iv_happy) | ||
| val tvHappy = findViewById<TextView>(R.id.tv_happy) | ||
|
|
||
| val ivExcited = findViewById<ImageView>(R.id.iv_excited) | ||
| val tvExcited = findViewById<TextView>(R.id.tv_excited) | ||
|
|
||
| val ivSoso = findViewById<ImageView>(R.id.iv_soso) | ||
| val tvSoso = findViewById<TextView>(R.id.tv_soso) | ||
|
|
||
| val ivUnrest = findViewById<ImageView>(R.id.iv_unrest) | ||
| val tvUnrest = findViewById<TextView>(R.id.tv_unrest) | ||
|
|
||
| val ivUpset = findViewById<ImageView>(R.id.iv_upset) | ||
| val tvUpset = findViewById<TextView>(R.id.tv_upset) | ||
|
|
||
| val textViews = listOf(tvHappy, tvExcited, tvSoso, tvUnrest, tvUpset) |
Collaborator
There was a problem hiding this comment.
enum~ 안녕하세요 미션아저씨에요 ! findViewById를 10번이나 써서 구현을 해주셨는데 이거는 옛날에 쓰던 방식입니다! 요즘에는 뷰바인딩을 써서 구현을 하는 추세이니 다음 미션때부터는 고려해서 써주세요!
Comment on lines
+146
to
+166
| <ImageView | ||
| android:id="@+id/iv_upset" | ||
| android:layout_width="80dp" | ||
| android:layout_height="90dp" | ||
| android:layout_marginTop="20dp" | ||
| android:src="@drawable/ic_upset" | ||
| app:layout_constraintEnd_toEndOf="parent" | ||
| app:layout_constraintStart_toStartOf="parent" | ||
| app:layout_constraintTop_toBottomOf="@id/tv_unrest" /> | ||
|
|
||
| <TextView | ||
| android:id="@+id/tv_upset" | ||
| android:layout_width="wrap_content" | ||
| android:layout_height="wrap_content" | ||
| android:layout_marginTop="8dp" | ||
| android:text="부글부글 화가 나요" | ||
| android:textColor="@color/black" | ||
| app:layout_constraintEnd_toEndOf="parent" | ||
| app:layout_constraintStart_toStartOf="parent" | ||
| app:layout_constraintTop_toBottomOf="@id/iv_upset" /> | ||
|
|
Collaborator
There was a problem hiding this comment.
맨 아래에 있는 '화가 나요' 텍스트가 화면 밑으로 살짝 잘린 것 같아요! 안드로이드 기기마다 화면 세로 길이가 다르기 때문에 생기는 자연스러운 현상입니다, 복잡하게 코드를 다 뜯어고칠 필요 없이 아주 간단한 설정만으로도 화면 안에 쏙 들어오게 맞출 수 있어요! 예를 들어서
-
- 지금 코드에 layout_marginTop="40dp", "20dp" 처럼 여백이 넉넉하게 들어가 있는데, 이 수치들이나 이미지 크기(80dp)를 전체적으로 살짝만 줄여줘도 밑에 공간이 확보되면서 글자가 위로 쏙 올라올 수 있습니다!
-
- 맨 마지막 텍스트뷰에만 app:layout_constraintBottom_toBottomOf="parent"를 걸고 밑에 layout_marginBottom을 살짝 줘서 화면 안쪽으로 전체 뷰들을 끌어올리는 방법도 편하고 좋습니다. 제일 간단하고 편한 방식으로 수치만 살짝 조정해 보시면 완벽할 것 같습니다!
Comment on lines
+49
to
+50
| android:text="선택한 감정우표를 기반으로 맞춤형 질문이 배달됩니다" | ||
| android:textColor="@color/gray_text" |
Collaborator
There was a problem hiding this comment.
색상은 colors.xml에 아주 잘 분리해주셨는데, xml의 android:text="오늘 하루는 어땠나요?" 같은 텍스트 값들이 아직 하드코딩되어 있습니다!
이런 값들을 res/values/strings.xml로 분리해 두면, 나중에 텍스트를 수정해야 할 때 코드를 일일이 찾아다니며 바꿀 필요 없이 xml 파일 한 곳에서만 싹 수정하면 되기 때문에 훨씬 편리해집니다!
Lemon0610
reviewed
Apr 1, 2026
Comment on lines
+5
to
+11
| <color name="gray_text">#767676</color> | ||
|
|
||
| <color name="happy_color">#FFD64B</color> | ||
| <color name="excited_color">#5EACE0</color> | ||
| <color name="soso_color">#7D88D7</color> | ||
| <color name="unrest_color">#61A072</color> | ||
| <color name="upset_color">#D94E4E</color> |
Collaborator
There was a problem hiding this comment.
역시 기본색보단 커스텀 색이 최고죠! 텍스트 색까지 예쁘게 바꾸셨네요.
sua710
reviewed
Apr 1, 2026
Comment on lines
+3
to
+10
| <string name="title_how_was_your_day">오늘 하루는 어땠나요?</string> | ||
| <string name="subtitle_select_stamp">감정우표를 선택해주세요</string> | ||
| <string name="description_custom_question">선택한 감정우표를 기반으로 맞춤형 질문이 배달됩니다</string> | ||
| <string name="stamp_happy">더없이 행복한 하루였어요</string> | ||
| <string name="stamp_excited">들뜨고 흥분돼요</string> | ||
| <string name="stamp_soso">평범한 하루였어요</string> | ||
| <string name="stamp_unrest">생각이 많아지고 불안해요</string> | ||
| <string name="stamp_upset">부글부글 화가 나요</string> |
mookeunji05
reviewed
Apr 1, 2026
| android:text="@string/stamp_excited" | ||
| android:textColor="@color/black" | ||
| app:layout_constraintEnd_toEndOf="parent" | ||
| app:layout_constraintStart_toStartOf="parent" |
Collaborator
|
수고하셨습니다~@~@ |
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
5 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.
📌feat: 1주차 미션 - 단일 화면 UI 만들기
🔗 관련 이슈
Closes #9
✨ 변경 사항
🔍 테스트
📸 스크린샷 (선택)
🚨 추가 이슈