Skip to content

Conversation

@casper-jr
Copy link
Member

🚀 이슈번호

✏️ 변경사항

  • 오류 처리 등에 사용할 수 있는 isError 파라미터 추가 (기본값은 false로 설정)
  • disabled인 경우에 대한 스타일 적용

📷 스크린샷

isError를 사용한 처리

스크린샷 2025-01-28 02 12 03

커서 색상 적용

스크린샷 2025-01-28 02 13 06

✍️ 사용법

  • 현재 예시를 위해서 TextFieldDefaults.colors의 파라미터의 값에 isError를 사용한 조건문을 작성해뒀는데, 필요에 따라 없애거나 수정해서 사용하면 됩니다. (ex: 비밀번호 오류 등)
  • 위에서 말한 TextFieldDefaults.colors에 지정할 수 있는 파라미터가 아주 많아서 혹시 추가적인 기능이 필요하면 알아서 추가하여 사용하면 됩니다.

🎸 기타

  • 이전에 disabled인 경우에 디자인이 반영되지 않았던 이유는 disabledContainerColor를 따로 지정해두지 않아서 기본 값으로 다른 디자인이 적용된 것 같습니다.
  • 커서 색상의 경우 BasicTextField의 cursorBrush 파라미터에서 설정해야 한다고 해서 수정했습니다.
  • 아직 27번 pr이 merge 되지 않아서 commit 이력이 같이 뜨는데, 마지막 3개의 commit만 이 pr에 해당합니다

@casper-jr casper-jr added ✨ feat Good for newcomers ⚡️ fix Extra attention is needed 🦦재우 ❌ No Merge labels Jan 27, 2025
@casper-jr casper-jr self-assigned this Jan 27, 2025
@ikseong00
Copy link
Collaborator

ikseong00 commented Jan 30, 2025

LGTM 👍

@casper-jr casper-jr merged commit 851b7d7 into OurMenu:develop Jan 30, 2025
1 check passed
@ikseong00 ikseong00 changed the title [Fix]: CustomTextField 버그 수정 (merge 하려면 27번 pr 이후에 merge 해주세요) [FIX]: CustomTextField 버그 수정 (merge 하려면 27번 pr 이후에 merge 해주세요) Jan 31, 2025
ikseong00 pushed a commit that referenced this pull request Feb 12, 2025
[Fix]: CustomTextField 버그 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat Good for newcomers ⚡️ fix Extra attention is needed 👌 OK Merge 🦦재우

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FIX] : CustomTextField 버그 수정

2 participants