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

fix: 롤링페이퍼 만들기 페이지의 가입한 모임 목록 empty state ui 추가 #634

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

fix: 롤링페이퍼 만들기 페이지의 가입한 모임 목록 empty state ui 추가 #634

wants to merge 1 commit into from

Conversation

soyi47
Copy link
Collaborator

@soyi47 soyi47 commented Oct 29, 2022

작업 내용

  • RollingpaperCreationPage의 Step1에 가입한 모임이 없는 상태의 UI를 추가했습니다.

변경 전

변경 후

논의할 부분

  • 다른 empty state ui와 똑같이 형식으로 추가했습니다. 코드 레벨이나 디자인 관련해서 의견 있으면 주세요!

closed #633

@soyi47 soyi47 requested a review from prefer2 October 29, 2022 05:20
prefer2
prefer2 approved these changes Oct 31, 2022
View reviewed changes
Copy link
Collaborator

@prefer2 prefer2 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수정 사항 확인했습니다! 이곳에도 empty state가 있는데 빼먹고 있었군요
이 상황에서는 progress bar가 안보이는건 어떤가요?? 진행을 할 수 없는데 오히려 사용자가 혼란스러울수도 있을 것 같아요.
MyTeamsPage와 동일한 empty state로 넘어가는건 어떤가요??

스크린샷 2022-10-31 오후 11 46 58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prefer2 prefer2 prefer2 approved these changes

Assignees
No one assigned
Labels
UI 버그 프론트엔드
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FE] 롤링페이퍼 만들기 페이지 Empty State UI를 추가한다.
2 participants
@soyi47 @prefer2