-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
양 끝에 아이콘을 추가할 수 있는 Edit Text & 검증 텍스트와 텍스트 제한 기능이 있는 Edit Text 구현 #12
Conversation
…viewBinding 설정
…t Start Icon 속성 구현
…t End Icon 속성 구현
…ext -> Royal Flush Icon Edit Text로 변경
…동작을 지정하는 확장 함수 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생많으셨습니다!!
추후에 정리할때 만드신 뷰의 다양한 케이스를 보여주면 좋을거같아요(iconedittext에서icon을 각각 안넣었을때 어떻게 된다던가, verticaledittext애서 글자 수를 넘을때는 어떻게 되는가 등등....)
물론 현수님아 테스트 다 했을테지만..ㅎㅎㅎ
검증된 텍스트의 길이가 엄청 길어지게 될 거라고는 생각을 못했어요..!! 고려(호랑이)해봐야겠습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니당 잘 쓰겠습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
컨벤션 수정만 해주시면 될거같슴다
<selector > | ||
<item xmlns:android="http://schemas.android.com/apk/res/android"> | ||
<shape android:shape="rectangle"> | ||
<solid android:color="#DFDBCC" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이런 리소스에 있는 hex color도 테마에 미리 정해놓은걸로 쓸수있나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그럼요
@@ -0,0 +1,54 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ConstraintLayout이 중첩으로 쓰인 이유가 있을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linear로 개선이 가능해보이네요 감사합니다!
android:background="@drawable/royal_flush_edit_text_background"> | ||
|
||
<ImageView | ||
android:id="@+id/start_ic" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어라라 컨벤션이
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 맞다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@soopeach 이거 해줘
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아이콘 넣을 수 있는거 이쁘네용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
resolved: #9
작업 내용
사용 방법
RoyaFlushIconEditText
royalFlushIconEditTextHint
속성으로 힌트 지정 가능.startIcon
속성으로 시작 쪽 아이콘 지정 가능.endIcon
속성으로 끝 쪽 아이콘 지정 가능.RoyalFlushVerificationEditText
royalFlushVerificationEditTextHint
속성으로 힌트 지정 가능.isCounterVisible
속성으로 텍스트 카운터 텍스트 visibility 여부 지정 가능.isVerificationVisible
속성으로 초기 검증 텍스트 visibility 지정 가능.verificationText
속성으로 초기 검증 텍스트 지정 가능.maxLength
최대 글자 수 제한 가능.RoyalFlushVerificationEditText 추가 기능