Skip to content

그룹상세화면 #3

Open
Hongji0611 wants to merge 8 commits intomainfrom
groupDetail
Open

그룹상세화면 #3
Hongji0611 wants to merge 8 commits intomainfrom
groupDetail

Conversation

@Hongji0611
Copy link
Member

@Hongji0611 Hongji0611 commented Jan 18, 2022

개요

  • 그룹 상세화면, 그룹 정보 화면, 그룹정보 변경하는 화면 구현

상세내용

  • mainStack을 만들어서 모든 화면 흐름이 main에서 시작되도록 수정했습니다. (나중에 홈화면 tab이 생기면 바뀔예정)
  • top-tab을 사용해서 투표, 설문, 참여완료를 탭 형식으로 보여줍니다.
  • 투표와 설문 리스트 박스는 공통으로 빼놨습니다.
  • 각 화면에서 일반 사용자, 그룹원, 그룹마스터에게 보여지는 화면이 다르도록 구현했습니다.

리뷰어가 확인할 사항

  • top-tab과 관련된 패키지를 3개 설치했으므로 npm install 해주세요!

기타

  • 그룹 가입하기 버튼, 상호인증 버튼, 그룹원 관리 버튼, 이미지 변경 버튼 등의 세부 기능은 구현하지 않았으니 이를 고려해서 확인해주세요.
  • 임의의 데이터를 넣었으므로 추후 구조가 변경될 수 있습니다.

style={styles.image}
source={require('../../../asset/image/black50.png')}>
<View style={styles.topContainer}>
<Image onTouchEnd={() => { navigateTo.goBack() }} source={require('../../../asset/icon/backWhite.png')} />
Copy link

Choose a reason for hiding this comment

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

뒤로 가기 버튼에 대한 padding이 더 넓으면 좋을거 같아요!

Copy link
Member

@0JIEUN0 0JIEUN0 left a comment

Choose a reason for hiding this comment

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

common component 를 잘 만들어두셔서 GroupDetailVoteScreen, GroupDetailSurveyScreen, GroupDetailResultScreen 동작이 깔끔하네요! 대신 data 빼고는 동작이 같아서 앞으로도 그렇다면 하나의 컴포넌트로 해도 괜찮겠다는 생각이 조금 들었습니다. 지금도 좋습니다~~!! 수고하셨어요!!👍👍

<View style={styles.textContainer}>
<View style={{ marginRight: 28 }}>
<Text style={styles.font3}>소개</Text>
<Text style={styles.font3}>링크</Text>
Copy link
Member

Choose a reason for hiding this comment

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

회의 내용에 따라, 링크 부분은 삭제해주시면 될 것 같아요!

@Hongji0611
Copy link
Member Author

Hongji0611 commented Jan 20, 2022

common component 를 잘 만들어두셔서 GroupDetailVoteScreen, GroupDetailSurveyScreen, GroupDetailResultScreen 동작이 깔끔하네요! 대신 data 빼고는 동작이 같아서 앞으로도 그렇다면 하나의 컴포넌트로 해도 괜찮겠다는 생각이 조금 들었습니다. 지금도 좋습니다~~!! 수고하셨어요!!👍👍

저도 이게 약간 걸렸는데 하나의 화면으로하면 탭을 쓴 의미가 있을까? 싶어서 일단 나눠두긴 했어요

  1. 그대로 유지한다. (화면에 따라서 데이터를 다르게 가져옴)
  2. 탭을 버튼으로 바꾸고 하나의 컴포넌트로 둔다 (버튼에 따라서 데이터를 가져옴. 하지만 탭일때의 슬라이드 효과는 사용 못함)

둘중에 선택하면 될 것 같습니다!

@Hongji0611 Hongji0611 closed this Jan 20, 2022
@Hongji0611 Hongji0611 reopened this Jan 20, 2022
Copy link
Member

@0JIEUN0 0JIEUN0 left a comment

Choose a reason for hiding this comment

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

저는 지금도 좋습니다! 수고하셨어요 👍👍

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