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

QA: React-Mentions 의 appendSpaceOnAdd 기능으로 모바일에서 멘션 delete 시 글자 남아있는 이슈 해결 #800

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

borimong
Copy link
Contributor

@borimong borimong commented Jul 19, 2024

🚩 관련 이슈

📋 작업 내용

  • 기능 구현
  • 페이지 구조화 및 스타일링

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지

  • 정확한 원인 파악은 실패했으나, 모바일에서 '@누군가' 을 하고, 바로 지우면, '군' 이런 식으로 글자가 남아있는 이슈가 잇었습니다. 왜 모바일에서만 이러는지 모르겠는데, 사파리 크롬 모두 이런 상황이었어요..! 그래서 콘솔을 다 찍어봐도 이건 크로스 브라우징 이슈도 아닌데 왜 이러지.. 싶어서 다른 방법을 찾아봣는데, 멘션을 하고 한 칸을 띄우면, 그 뒤에 지워도 아무 문제가 없더라구요..?? 그래서 멘션 라이브러리에 있는 appendSpaceOnAdd 기능을 가져와서, '@누군가 ' 이렇게 멘션 후 한 칸 띄워주는 식으로 fix 해주었습니다...

  • ++ 테스트에 쓰인 React Mentions 안 콘솔 로그 다 지워주었어요!

  • 어떤 위험이나 우려가 발견되었는지

  • 원인 파악을 제대로 못했기에 후속 조사가 필요해요!

  • 어떤 부분에 리뷰어가 집중해야 하는지

  • 개발하면서 어떤 점이 궁금했는지

📸 스크린샷

@borimong borimong requested a review from R1mmm as a code owner July 19, 2024 11:12
Copy link

height bot commented Jul 19, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@borimong borimong changed the title fix: appendSpaceOnAdd 로 모바일 이슈 해결 QA: React-Mentions 의 appendSpaceOnAdd 기능으로 모바일에서 멘션 delete 시 글자 남아있는 이슈 해결 Jul 19, 2024
@borimong borimong merged commit 4cf242b into develop Jul 19, 2024
2 checks passed
borimong added a commit that referenced this pull request Jul 19, 2024
borimong added a commit that referenced this pull request Jul 19, 2024
@borimong borimong deleted the feat/#799 branch July 22, 2024 14:53
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.

모바일 멘션 delete 시 value 남는 이슈
1 participant