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

[Feat/#18] 영수증 인식 실패 UI 제작 #19

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from
Open

Conversation

sikkzz
Copy link
Collaborator

@sikkzz sikkzz commented Jan 31, 2025

💡 변경사항 & 이슈

영수증 인식 실패 UI 제작

✍️ 관련 설명

UI 제작이라 특별한건 없고 각 뷰마다 배경 관련해서 겹치는 스타일링이 계속 늘어날 거 같은데 이건 이후에 공통화 시킬 예정이야
우선은 전체 페이지 다 만들어 놓고 이후에 리팩토링 하는게 좋을 거 같아

하나 의견이 필요한게 있는데 클래스 네이밍 관련해서 어떻게 하면 좋은지 모르겠어서!
지금은 css-in-js 쓰는거 마냥 했는데 의견 있으면 얘기해줘!

스크린샷 2025-01-31 오후 3 50 03 스크린샷 2025-01-31 오후 3 49 55

UI 페이지는 /recognition-fail 에서 확인 가능한데 url도 내 마음대로 정한거라 의견 있으면 부탁해!

⭐️ Review point

  • UI 관련해서 피드백이나 의견
  • 컴포넌트 네이밍 관련 의견
  • 클래스 네이밍 관련 의견
  • url 관련 의견

📷 Demo

스크린샷 2025-01-31 오후 3 51 44

@sikkzz sikkzz added the Feature new feature label Jan 31, 2025
@sikkzz sikkzz self-assigned this Jan 31, 2025
@sikkzz sikkzz requested a review from lgrin-byte as a code owner January 31, 2025 06:52
@sikkzz sikkzz linked an issue Jan 31, 2025 that may be closed by this pull request
1 task
Copy link

🧷 Storybook: https://6798a432c75b75bf6b81bc26-rrfsigfoff.chromatic.com/

⏰ Update: 2025년 01월 31일 15시 54분

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 영수증 인식 실패 UI 제작
1 participant