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

[entry2-web/2745] 네비게이션 탭에 따라 페이지가 달라지도록 수정 #316

Merged
merged 2 commits into from
May 26, 2021

Conversation

Tnks2U
Copy link
Contributor

@Tnks2U Tnks2U commented May 25, 2021

entry2-web 2745대응

  • '온라인 작품 불러오기' > '관심 작품' 에서 안내 코멘트를 변경
    • 선택된 네비게이션 타입 정보 type(selected)를 Popup > Projects 컴포넌트에서 Popup > Projects > EmptyContents 컴포넌트로도 props로 전달.
    • 전달된 type정보를 기반으로 각기 다른 멘트가 출력되도록 로직 수정
  • '작품 만들기' 버튼 삭제
    • 전달된 type정보를 기반으로 'favorites' 타입일 경우 버튼을 렌더링하지 않도록 수정

- '온라인 작품 불러오기' > '관심 작품' 에서 안내 코멘트를 변경
  - 선택된 네비게이션 타입 정보 type(selected)를 Popup > Projects 컴포넌트에서 Popup > Projects > EmptyContents 컴포넌트로도 props로 전달.
  - 전달된 type정보를 기반으로 각기 다른 멘트가 출력되도록 로직 수정
- '작품 만들기' 버튼 삭제
  - 전달된 type정보를 기반으로 'favorites' 타입일 경우 버튼을 렌더링하지 않도록 수정
@Tnks2U Tnks2U requested review from kimorkim, leunge and prisml May 25, 2021 07:56
@Tnks2U Tnks2U changed the title feat: 네비게이션 탭에 따라 페이지가 달라지도록 수정 [entry2-web/2745] 네비게이션 탭에 따라 페이지가 달라지도록 수정 May 25, 2021
@@ -118,6 +118,8 @@ class Popup extends Component {
break;
}
case 'projects':
view = <Projects type={selected} data={data} />;
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분은 굳이 나눈이유가 있을까요?

selected가 'project'일때와 'favorites'일때를 아직은 굳이 구분할 필요가 없을 것 같아 재수정하였습니다.
@prisml prisml merged commit 7e07e99 into develop May 26, 2021
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