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

[FE] 멤버 추가 시 제대로 작동하지 않는 문제 수정 #672

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

Todari
Copy link
Contributor

@Todari Todari commented Sep 26, 2024

issue

구현 목적

환경에 따라 다양한 버그가 발생합니다.
IOS, AN 두 운영체제에서 다르게 동작하는 버그도 존재하고,
focus, blur, isComposing, defaultEvent 등 다양한 문제가 엮여있는 상황입니다.
멤버 추가 시 Input 입력 후 엔터를 누를 경우 위의 다양한 이유들로 인해 예상대로 작동되지 않았습니다.

이를 정상적으로 작동하게 하도록 구현합니다.

구현 사항

기존의 LabelInput component는 Input component를 사용하는데, 이 내부에 기초적으로 Enter를 위한 핸들러를 적용하고 있었습니다.
사용처에서 부터 LabelInput, Input, input에 이르기까지 너무 많은 불필요한 상태를 컨트롤하고, ref를 전달하며 중복되는 이벤트 핸들링이 존재하여
이미 사용하지 않게된 Input component를 현재 LabelInput으로 통일성 있게 구현하고 적용하였습니다.
또한 isIOS 함수를 이용하여 환경 별로 동작 방식을 다르게 하여, os 별 텍스트 및 input 처리에 대해서 효과적으로 작동할 수 있도록 처리해 주었습니다.

@Todari Todari added 🖥️ FE Frontend 🚨 bug bug 🔍 QC quality check labels Sep 26, 2024
@Todari Todari added this to the v2.0.0 milestone Sep 26, 2024
@Todari Todari self-assigned this Sep 26, 2024
Copy link

@jinhokim98 jinhokim98 merged commit 95dfd30 into fe-dev Sep 26, 2024
2 checks passed
@jinhokim98 jinhokim98 deleted the feature/#671 branch September 26, 2024 12:43
Copy link

@soi-ha
Copy link
Contributor

soi-ha commented Sep 26, 2024

이거 고치느라ㅜ너무 고샹많았어요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 bug bug 🖥️ FE Frontend 🔍 QC quality check
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants