[REF/#633] Replace EmojiFilter with String extensions#689
Conversation
- Move the `removeEmoji` function from `EmojiFilter.kt` to `StringExt.kt`. - Delete the now-unused `EmojiFilter.kt` object. - Update the import statement in `HilingualLongTextField.kt` to use the new extension function.
nhyeonii
left a comment
There was a problem hiding this comment.
string.removeEmoji() 확장 함수로 변경하니 훨씬 코틀린스러워진 것 같아요 ! 빈 문자열 보여주는것으로 수정한것도 UX적으로 좋아진 것 같습니다. 고생하셨어요 !
nahy-512
left a comment
There was a problem hiding this comment.
서버에서 주제를 안 넣어주면 매번 글을 못 쓰는 게 불편했는데 빈 문자열을 넣어주는 방식 좋은 것 같습니다~!!
객체 없이 StringExt 바로 사용하는 방식도 좋네용
Hyobeen-Park
left a comment
There was a problem hiding this comment.
일기주제가 없을 때도 작성 가능하게 하는 것 너무 좋은 것 같아요 한 가지 궁금한게 있는데요 지금처럼 빈 string만 넘기면 오늘의 추천 주제 참고하기 드롭다운은 그대로 보이고 안에 주제만 비어있는 상태가 될 것 같은데 드롭다운을 계속해서 보이게 유지하신 이유가 있으실까요?
먼저 해당 UX는 기능명세서에 없던 부분이라 임의대로 수정했습니다. 기존에는 아에 작성자체가 불가능해서.... 작성 자체만 가능하게 했습니다👍🏻 |
Daljyeong
left a comment
There was a problem hiding this comment.
String의 확장함수로도 구현이 가능한 부분이었네요!
추후 기획과 논의는 필요하겠지만 주제가 없을 때 작성할 수 있는 것도 기획, 개발에게 둘 다 편할 것 같네요 :)
수고하셨습니다!! 🚀
Hyobeen-Park
left a comment
There was a problem hiding this comment.
오 제가 어푸를 깜빡했네요ㅎㅎ 수고하셨습니다
Related issue 🛠
Work Description ✏️
Screenshot 📸
N/A
Uncompleted Tasks 😅
N/A
To Reviewers 📢
불필요한 객체 방식을 수정했습니다ㅎㅎ
추가로 이번 PR 범위는 아니지만 사소한 UX개선을 하나 끼워넣었습니다. 일기 주제가 없으면 아에 글을 작성하지 못하는 구조였어서 이부분은 빈 문자열 보여주는것으로 수정했습니다. 주제는 강제가 아니기 때문에!