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

[GGFE-247] 아이템 사용 alert 메시지 정리 #1001

Merged
merged 11 commits into from
Sep 12, 2023

Conversation

yoouyeon
Copy link
Member

📌 개요

  • 아이템 사용 안내 Alert에 쓰이는 메시지들을 정리했습니다.

💻 작업사항

중복되어 사용되는 메시지들이 많고 이모지 적용이 번거로워 질 것 같아서 아이템 메시지를 관리하는 파일을 하나 만들었습니다. (ITEM_ALERT_MESSAGE)
기존에 효빈님이 작성해주신 메시지에는 이모지가 문장 뒤에 있었는데, 혹시라도 화면이 작아져서 줄바꿈이 생기게 되었을 때 이모지 중간에 줄바꿈이 생기면 이상해 질 것 같아서 이모지를 앞쪽으로 뺐습니다.

✅ 변경로직

@yoouyeon yoouyeon self-assigned this Sep 11, 2023
Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

확인했습니다! 훨씬 깔끔하고 통일감 있어서 좋네요! 나중에 문구 수정할 일 생겼을 때 수정하기도 더 편할 것 같습니다! constants 폴더가 있긴 한데 itemAlertMessage.ts 파일이 거기 들어가도 되는지는 잘 모르겠네요! 고생하셨습니다!! 👍

@yoouyeon
Copy link
Member Author

@PHJoon 오 그 폴더가 있는줄도 모르고 있었어요,,, constants 폴더쪽이 더 맞는 것 같아 보여서 그 폴더로 이동시킬게요!!

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

이렇게 정리해두니 훨씬 깔끔하네요! 이모지 앞에 두는 것도 좋은 생각같아요👍 상점 구매 쪽도 이런식으로 수정하겠습니다! 고생하셨습니다👏

@@ -0,0 +1,31 @@
const SAD_EMOJI = '(。•́︿•̀。)' + ' ';
const HAPPY_EMOJI = '(。•̀ᴗ-)✧' + ' ';
Copy link
Member

Choose a reason for hiding this comment

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

이모지 너무 귀여워요!!

@yoouyeon yoouyeon merged commit b1b5261 into main Sep 12, 2023
1 check passed
@yoouyeon yoouyeon deleted the GGFE-247-아이템-사용-alert-메시지-정리 branch September 12, 2023 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants