Skip to content

Conversation

@MoonsuKang
Copy link
Member

@MoonsuKang MoonsuKang commented Feb 18, 2025

Related issue 🛠

closed #147

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • CustomClick을 각 컴포넌트 및 아이콘에 적용함(클릭 시 해당 아이템의 Alpha를 0.5로 설정)
  • TextField에 SingleLine추가 및 KeyBoardAction을 통해 엔터 클릭 시 포커스 해제

Uncompleted Tasks 😅

  • [ ]

To Reviewers 📢

CusomClickable이 문제가 있는 줄 알았는데 그게 아니라 SettingViewModel단에서 init으로 fetchUserInfo를 하면 두 번째 클릭부터 이벤트를 발행한다는 것을 알게되었습니다... 지금 현재 Route단에서 LaunchedEffect viewmodel.refreshUserInfo로 하면 되긴 하는데 이러면 비즈니스 로직 분리가 안되서 생각좀 해보고 다시 올리겠습니다

Copy link
Member

Choose a reason for hiding this comment

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

[p3]
닉네임의 경우 6글자 제한이 있고, 추후 다른 입력값에서 글자 제한이 있을 수 있기 때문에
maxLength 설정할 수 있게 해야 할 것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

닉네임 정규표현식 수정도 있어서 이건 다음 이슈에서 처리할 예정 입니다

@DongChyeon
Copy link
Member

[p1]
image
기본적으로 isActionEnabled 가 false 이기 때문에, true로 해야하며
기획, 디자인상에서는 상시 활성화, main 색상 사용

Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

굿굿

@MoonsuKang MoonsuKang merged commit 76ef1a2 into develop Feb 20, 2025
1 check passed
@DongChyeon DongChyeon deleted the refactor/#147-ripple-effect branch June 10, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] Click Ripple Effect를 수정하고 프로필 수정 버튼 Enable 속성을 부여합니다.

3 participants