Skip to content

Conversation

@JaeYoung290
Copy link
Owner

실습 과제 3 PR입니다.

Copy link

@skdud0629 skdud0629 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네이밍 컨벤션만 잘 지키시면 될 것 같습니다!
ui 구성 잘 하셨어요!😊

<!-- EditText 색상 -->
<style name="EditTextStyle">
<item name="android:backgroundTint">@color/light_edit_text_background</item>
<item name="android:textColorHint">@color/light_hint_text_color</item>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

꼭 색깔만 정의할 수 있는 게 아니라 gravity나 textSize 등 다른 요소도 활용할 수 있습니다!

android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_marginTop="16dp"
android:text="로그인"/>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

string resource 활용해주세요!

<style name="Theme.Assignment_week_02" parent="Base.Theme.Assignment_week_02" />

<!-- TextView 색상 -->
<style name="TEXT">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

왜 얘만 이름이 대문자인거죵
어떤 view 에 활용되는 테마인지 명시하면 나중에 알아보기 편하겠죠?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants