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

refactor: 웹에 맞게 Toast.style 수정 #79

Closed
2 tasks done
nijuy opened this issue May 4, 2024 · 6 comments · Fixed by #84
Closed
2 tasks done

refactor: 웹에 맞게 Toast.style 수정 #79

nijuy opened this issue May 4, 2024 · 6 comments · Fixed by #84
Assignees
Labels

Comments

@nijuy
Copy link
Collaborator

nijuy commented May 4, 2024

ISSUE ✅ : REFACTOR

📖 Summary

현재 토스트는 모바일 화면을 보고 만들어서 실제로 웹에서 사용했을 때 문제가 많음

TODO

  • 위치 기준 변경 (전체 화면 -> 현재 화면)

    전체 화면으로 기준을 잡으면 스크롤이 생기는 페이지에서는 스크롤 내려야 토스트 확인 가능한 문제가 있음

  • width props 추가

    현재는 Toast를 감싸는 부모 컴포넌트의 너비를 가져와서 사용하는데, 웹에서는 과하게 길어지는 문제가 있음
    차라리 사용자한테 입력 가능하게 해주는 게 나을듯?

@nijuy nijuy added the refactor label May 4, 2024
@Hanna922
Copy link
Member

Hanna922 commented May 4, 2024

Toast.style.ts 보면 빨간 줄 뜨고 있는데, 요것도 같이 수정하면 좋을 것 같네요

image

@nijuy
Copy link
Collaborator Author

nijuy commented May 4, 2024

저거 마유ㅜ스 올리면 내용 뭐라 떠요?? 저한텐 안 떠서 뭔지 확인 할 수가 ❌

image

@Hanna922
Copy link
Member

Hanna922 commented May 4, 2024

ts-styled-plugin 이라고 뜨는데, 보리 혹시 vscode-styled-components 익스텐션 사용 안 하시나요?? 익스텐션 제거하니까 전부 문자열로 떠서용

image

@nijuy
Copy link
Collaborator Author

nijuy commented May 5, 2024

넹 전 그거 안 써서 안 떴나봐여!

@Hanna922
Copy link
Member

Hanna922 commented May 5, 2024

옹 이 확장 없으면 styled-components 자동완성 안 되지 않나요??
이거 쓰면 자동완성이랑 하이라이팅 전부 해줘서,,, (Reference)
확장 추천으로 vscode-styled-components 뜨게끔 코드 설정하고, 이 코드도 수정하면 좋을 것 같아요

=> recommendations 추가한 PR 올려둘게용

@nijuy
Copy link
Collaborator Author

nijuy commented May 5, 2024

네~~

@nijuy nijuy self-assigned this May 5, 2024
@nijuy nijuy linked a pull request May 5, 2024 that will close this issue
1 task
@nijuy nijuy closed this as completed in #84 May 8, 2024
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 a pull request may close this issue.

2 participants