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.

Sign up for GitHub

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

Jump to bottom

[이슈 해결] 홈화면 다크모드시 배경색 이슈, 포켓런 결과란 입력값 넘칠 경우 스크롤 처리, 로그인 없이 포켓런시 토스트 #59

Merged
merged 3 commits into from
Dec 2, 2024
Merged

[이슈 해결] 홈화면 다크모드시 배경색 이슈, 포켓런 결과란 입력값 넘칠 경우 스크롤 처리, 로그인 없이 포켓런시 토스트 #59

merged 3 commits into from
Dec 2, 2024

Conversation

hailey-hy
Copy link
Collaborator

@hailey-hy hailey-hy commented Nov 29, 2024 •
edited
Loading

🏆 Details

  • 홈 컴포넌트에 배경색 지정
  • 포켓런 결과란의 입력값 표시 컴포넌트에 overflow: scroll 처리

📸 Screenshot

2024-11-29.10.41.27.mov
2024-11-29.10.59.40.mov

@hailey-hy hailey-hy requested a review from yusiny as a code owner November 29, 2024 13:42
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 29, 2024 •
edited
Loading

👀 Preview Success!

Name Link
🔨 Latest commit 467d04d
🔍 Latest deploy log https://github.com/ai-surfers/pocket-prompt-frontend/actions/runs/12114682353
🖇️ Deploy Preview Url https://2b5a6d2a.pocket-prompt-frontend.pages.dev
🌳 Environment preview

@hailey-hy hailey-hy mentioned this pull request Nov 29, 2024
Closed
1 task
@hailey-hy hailey-hy changed the title [이슈 해결] 홈화면 다크모드시 배경색 이슈, 포켓런 결과란 입력값 넘칠 경우 스크롤 처리 [이슈 해결] 홈화면 다크모드시 배경색 이슈, 포켓런 결과란 입력값 넘칠 경우 스크롤 처리, 로그인 없이 포켓런시 토스트 Nov 29, 2024
yusiny
yusiny approved these changes Nov 30, 2024
View reviewed changes
Copy link
Member

@yusiny yusiny left a comment

Choose a reason for hiding this comment

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

고생하셨어요!

  1. 포켓런 실행 오류 발생 시에도 모달이나 알럿에 에러 메시지라도 노출해 주는 게 좋을 것 같아요~!
  2. 이건 그냥 의견인데, 칩을 사람들이 스크롤하는 영역이라고 인지하긴 어려울 것 같아서요 토글이나 다른 UI로 개선 의견 드려보는 것도 좋을 것 같긴 하네요,, (꼭 반영해 달라는 건 아니고, 고민만 해주시면 좋을 것 같아요)

@hailey-hy
Copy link
Collaborator Author

  1. 반영하겠습니다!
  2. 운영환경에서 이슈가 발견되어서 기획 기다리지 않고 바로 핫픽스로 나간 건이어서 그런 것 같아요! 기획분께서도 인지하고 계실텐데 한번 더 노티 드릴게요.

피드백 감사해요 :D

@hailey-hy hailey-hy closed this Dec 2, 2024
@hailey-hy hailey-hy reopened this Dec 2, 2024
@hailey-hy hailey-hy merged commit db9f69c into develop Dec 2, 2024
2 checks passed
hailey-hy added a commit that referenced this pull request Dec 3, 2024
@hailey-hy @yusiny
24.12.03 (#63)
0bd5851 
* chore : api 헤더에 source-location 추가 (#55)

* [프롬프트 상세보기] 포켓런 기능 구현 (#54)

* chore: prod env store ID 변경 적용

* feat: 포켓런 구현

* chore: 포켓런 모델에 chatgpt만 포함되도록 임시 수정

* [프롬프트 상세] 포켓런 부가 기능 (#57)

* feat: 포켓런 실행시 로딩 기능 추가

* feat: 결과 복사하기 기능 추가

* feat: 포켓런 최초 실행 이후 다른 입력값으로 실행했을 때 버튼 string 변경 기능 추가

* chore: 프롬프트 상세 페이지 접근시 포켓런 결과 상태 초기화

* [사이드바] 모바일 헤더 사이드바구현  (#56)

* style : 사이드바 기본 퍼블

* chore : 레이아웃 수정

* refac : 메뉴 core 로 분리

* refac : 사이드바 컴포넌트 분리

* [이슈 해결] 홈화면 다크모드시 배경색 이슈, 포켓런 결과란 입력값 넘칠 경우 스크롤 처리, 로그인 없이 포켓런시 토스트 (#59)

* fix: 다크모드시 홈 화면 배경색 바뀌는 이슈 해결

* fix: 포켓런 결과창 입력값 넘칠 경우 스크롤 처리

* fix: 로그인하지 않고 포켓런시 토스트 띄우도록 수정

* [홈] 모바일 반응형 (#60)

* chore : useMobile 훅으로 분리

* style : LNB 반응형

* style : 배너 반응형

* style : 프롬프트 영역 반응형

* style : footer width 수정

* fix : import 미사용 제거

* chore : 메타 태그 세팅

* chore : meta 수정

* feat : 모바일에서 prompt 5개씩 조회

* style : empty 화면 추가

* fix : 빌드 오류

* chore: extension, pricing 페이지에만 footer 적용 (#62)

---------

Co-authored-by: Yooshin Kim <yusinkim.or@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yusiny yusiny yusiny approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hailey-hy @yusiny