Skip to content

🔀::MainMissionView UI 작성#31

Open
gangho0922 wants to merge 62 commits intodevelopfrom
feat/student-main
Open

🔀::MainMissionView UI 작성#31
gangho0922 wants to merge 62 commits intodevelopfrom
feat/student-main

Conversation

@gangho0922
Copy link
Collaborator

@gangho0922 gangho0922 commented Jul 5, 2024

  1. MainMissionView의 UI를 작성하였습니다.
  2. 추가되지 않았던 등수표시할 때 넣을 Medal의 Color들을 추가였습니다.
  3. 오류가 있는 banner 이미지를 수정하였습니다.
  4. Extension/Image 부분의 오타들을 수정하였습니다.

image

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-07-12.at.16.08.22.mp4

이 동영상은 popupView UI 동작 장면입니다.

Asset 파일 추가
Asset 파일 수정
오타 수정
Color DesignSystem 추가
LogoNavigationBar 추가
MainMissionScrollView 추가
MainRankingView 추가
Color extension 추가
StudentMainView UI 작성
@gangho0922 gangho0922 requested review from MelBIP and uuuunseo July 5, 2024 02:53
@gangho0922 gangho0922 self-assigned this Jul 5, 2024
@gangho0922 gangho0922 linked an issue Jul 5, 2024 that may be closed by this pull request
컨벤션에 따른 수정 및 몇몇 UI 컴포넌트화
SignupListPopupView 코드 수정
SignupListPopupView 위치 수정
rankingID 추가
rankingID 추가
MainMissionScrollView UI 수정
MainMissionScrollView UI 수정
RankingDataList 추가
StudentMainView 코드 수정
추후에 작업할 코드 추가
파일 이름 TeacehrTabBar -> TeacherTabBar로 수정
MainRankingScrollView 코드 수정
MainMissionScrollView 코드 수정
StudentMainView 코드 수정
MainRankingScrollView 필요없는 코드 수정
MainMissionScrollView 필요없는 코드 수정
StudentMainModel 수정
StudentMainModelStateProtocol 수정
StudentMainView 필요없는 코드 수정
SignupListPopupView 필요없는 container 삭제
StudentMainView 코드 수정
struct StudentMainView: View {
@StateObject var studentMainContainer: MVIContainer<StudentMainIntent, StudentMainModelStateProtocol>

@State private var showPopup = false
Copy link
Collaborator

Choose a reason for hiding this comment

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

View에서 이렇게 변수를 관리하는 건 MVI랑 맞지 않는 거 같아요.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

그럼 혹시 저 변수들을 따로 빼내야되는건가요?? 제가 MVI에 대해 잘 몰라가지고 여쭤봅니다~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아 그리고 저 showPopup은 StudentMainView에다가 Popup을 띄워야되는줄 알고 만들었던거라 저건 지우도록 하겠습니다~

VStack(alignment: .leading) {
Text("* 문제는 12:30 ~ 19:30분까지 풀수 있습니다.")
.skFont(.pr12)
.frame(maxWidth: .infinity, alignment: .leading)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 코드가 왼쪽정렬을 위한 코드라면 VStack에서 leading을 줘서 필요 없을 것 같은데 삭제하면 왼쪽 정렬이 안되나요? 아니면 다른 이유가 있는 건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아뇨 다른 이유는 없고 SwiftUI에서 UI 관련해서 여러 테스트를 해보다가 제거 하는걸 깜빡했던 거 같습니다~ 바로 지울게용


Text("-")

Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요
Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요

주석이 있는데 무슨 뜻인지 이해를 못하겠어요...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

TeacherMainView에 container가 만들어지고 나서 저 Text의 "" 자리에 걸맞는 걸 삽입한다는 뜻이었습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저러한 의도 였다면 주석을 어떤식으로 다는게 좋을까요?? 의견 여쭤보고 싶습니다~

Copy link
Collaborator

Choose a reason for hiding this comment

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

PopupView는 container가 필요 없는 CustomView 아닌가요? TeacherMain의 Model이 만들어져야 하는 이유가 뭔지 궁금합니다. 삽입 한다는 말이 조금 애매하기도 해서 CustomView면 ""보다는 상수나 변수를 선언해서 넣어두는 편이 더 좋을 것 같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저는 그냥 CustomView 각각마다 container가 들어가야되는줄 알았습니다~ 그럼 그렇게 수정해서 다시 올리겠습니다~

MainMissionScrollView 코드 단축
StudentMainView 필요없는 코드 삭제
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.

Student- Main페이지 퍼블리싱

3 participants