Skip to content
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

Merged
merged 11 commits into from
Mar 26, 2024

Conversation

soopeach
Copy link
Contributor

resolved: #9

작업 내용

  • 양 끝에 아이콘을 추가할 수 있는 Edit Text 구현 (RoyaFlushIconEditText)
  • 텍스트 검증, 글자 수 제한 + 글자 수 세기 기능이 있는 Edit Text 구현 (RoyalFlushVerificationEditText)

사용 방법

RoyaFlushIconEditText

Screenshot 2023-05-17 at 12 29 14 AM
  • royalFlushIconEditTextHint 속성으로 힌트 지정 가능.
  • startIcon 속성으로 시작 쪽 아이콘 지정 가능.
  • endIcon 속성으로 끝 쪽 아이콘 지정 가능.
Screenshot 2023-05-17 at 12 32 44 AM

RoyalFlushVerificationEditText

Screenshot 2023-05-17 at 12 34 30 AM
  • royalFlushVerificationEditTextHint 속성으로 힌트 지정 가능.
  • isCounterVisible 속성으로 텍스트 카운터 텍스트 visibility 여부 지정 가능.
  • isVerificationVisible 속성으로 초기 검증 텍스트 visibility 지정 가능.
  • verificationText 속성으로 초기 검증 텍스트 지정 가능.
  • maxLength최대 글자 수 제한 가능.
Screenshot 2023-05-17 at 12 37 28 AM

RoyalFlushVerificationEditText 추가 기능

Screenshot 2023-05-17 at 12 41 25 AM
  • setVerificationText() 메서드로 검증 텍스트 변경 가능
Screenshot 2023-05-17 at 12 42 27 AM
  • addAfterTextChangedListener 확장함수와 setVerification() 메서드를 조합해서 실시간으로 검증 여부 설정 및 판단 가능

@soopeach soopeach requested a review from a team as a code owner May 16, 2023 15:45
@soopeach soopeach self-assigned this May 16, 2023
@soopeach soopeach linked an issue May 16, 2023 that may be closed by this pull request
6 tasks
@github-actions github-actions bot added 🎀 Design 디자인 & 리소스 변경 🖇 Chore 그 외의 것들 In Review 리뷰 중인 이슈 labels May 16, 2023
Copy link
Contributor

@swjw13 swjw13 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!!
추후에 정리할때 만드신 뷰의 다양한 케이스를 보여주면 좋을거같아요(iconedittext에서icon을 각각 안넣었을때 어떻게 된다던가, verticaledittext애서 글자 수를 넘을때는 어떻게 되는가 등등....)
물론 현수님아 테스트 다 했을테지만..ㅎㅎㅎ

@soopeach
Copy link
Contributor Author

verticaledittext애서 글자 수를 넘을때는 어떻개되는가 등등....)

검증된 텍스트의 길이가 엄청 길어지게 될 거라고는 생각을 못했어요..!! 고려(호랑이)해봐야겠습니다.
커스텀 뷰 위키에 사용법 올려두면 좋을 것 같아서 추후에 조금 더 상세하게 정리해놓을게용!!

bngsh
bngsh previously approved these changes May 18, 2023
Copy link
Contributor

@bngsh bngsh left a comment

Choose a reason for hiding this comment

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

감사합니당 잘 쓰겠습니다!

Copy link
Contributor

@ldh019 ldh019 left a 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" />
Copy link
Contributor

Choose a reason for hiding this comment

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

이런 리소스에 있는 hex color도 테마에 미리 정해놓은걸로 쓸수있나요?

Copy link
Contributor Author

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"
Copy link
Contributor

Choose a reason for hiding this comment

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

ConstraintLayout이 중첩으로 쓰인 이유가 있을까요?

Copy link
Contributor Author

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"
Copy link
Contributor

Choose a reason for hiding this comment

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

어라라 컨벤션이

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 맞다

Copy link
Contributor

Choose a reason for hiding this comment

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

@soopeach 이거 해줘

jeeminimini
jeeminimini previously approved these changes May 22, 2023
Copy link
Contributor

@jeeminimini jeeminimini left a comment

Choose a reason for hiding this comment

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

아이콘 넣을 수 있는거 이쁘네용

swjw13

This comment was marked as resolved.

@soopeach soopeach dismissed stale reviews from swjw13 and jeeminimini via cd5b180 March 26, 2024 05:01
Copy link
Contributor

@ldh019 ldh019 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ldh019 ldh019 added this pull request to the merge queue Mar 26, 2024
Merged via the queue into develop with commit 10b4546 Mar 26, 2024
@github-actions github-actions bot added Done 처리된 이슈 and removed In Review 리뷰 중인 이슈 labels Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖇 Chore 그 외의 것들 🎀 Design 디자인 & 리소스 변경 Done 처리된 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature/common/royal_flush_edit_text: Royal Flush Edit Text 구현
5 participants