🔀::MainMissionView UI 작성#31
Open
gangho0922 wants to merge 62 commits into
Open
Conversation
Asset 파일 추가
Asset 파일 수정
Color DesignSystem 추가
LogoNavigationBar 추가
MainMissionScrollView 추가
MainRankingView 추가
Color extension 추가
StudentMainView UI 작성
uuuunseo
reviewed
Jul 5, 2024
uuuunseo
reviewed
Jul 5, 2024
uuuunseo
reviewed
Jul 5, 2024
MelBIP
approved these changes
Jul 5, 2024
StudentMainModel 작성
StudentMainModelProtocol 작성
StudentMainView 코드 수정
…e/Stack-Knowledge-iOS-v2 into feat/student-main
불필요 파일 삭제
StudentMainIntent 파일 생성
…owledge-iOS-v2 into feat/student-main
…owledge-iOS-v2 into feat/student-main
AppComponent+Order
Main관련 파일 삭제
StudentMain DI 수정
StudentMainModel 작성
컨벤션에 따른 수정 및 몇몇 UI 컴포넌트화
SignupListPopupView 코드 수정
uuuunseo
reviewed
Jul 13, 2024
uuuunseo
reviewed
Jul 13, 2024
uuuunseo
reviewed
Jul 13, 2024
SignupListPopupView 위치 수정
rankingID 추가
rankingID 추가
MainMissionScrollView UI 수정
MainMissionScrollView UI 수정
RankingDataList 추가
StudentMainView 코드 수정
추후에 작업할 코드 추가
…owledge-iOS-v2 into feat/student-main
파일 이름 TeacehrTabBar -> TeacherTabBar로 수정
MainRankingScrollView 코드 수정
MainMissionScrollView 코드 수정
StudentMainView 코드 수정
uuuunseo
reviewed
Jul 18, 2024
MainRankingScrollView 필요없는 코드 수정
MainMissionScrollView 필요없는 코드 수정
StudentMainModel 수정
StudentMainModelStateProtocol 수정
StudentMainView 필요없는 코드 수정
SignupListPopupView 필요없는 container 삭제
StudentMainView 코드 수정
uuuunseo
reviewed
Jul 18, 2024
| struct StudentMainView: View { | ||
| @StateObject var studentMainContainer: MVIContainer<StudentMainIntent, StudentMainModelStateProtocol> | ||
|
|
||
| @State private var showPopup = false |
Collaborator
There was a problem hiding this comment.
View에서 이렇게 변수를 관리하는 건 MVI랑 맞지 않는 거 같아요.
Collaborator
Author
There was a problem hiding this comment.
그럼 혹시 저 변수들을 따로 빼내야되는건가요?? 제가 MVI에 대해 잘 몰라가지고 여쭤봅니다~
Collaborator
Author
There was a problem hiding this comment.
아 그리고 저 showPopup은 StudentMainView에다가 Popup을 띄워야되는줄 알고 만들었던거라 저건 지우도록 하겠습니다~
| VStack(alignment: .leading) { | ||
| Text("* 문제는 12:30 ~ 19:30분까지 풀수 있습니다.") | ||
| .skFont(.pr12) | ||
| .frame(maxWidth: .infinity, alignment: .leading) |
Collaborator
There was a problem hiding this comment.
이 코드가 왼쪽정렬을 위한 코드라면 VStack에서 leading을 줘서 필요 없을 것 같은데 삭제하면 왼쪽 정렬이 안되나요? 아니면 다른 이유가 있는 건가요?
Collaborator
Author
There was a problem hiding this comment.
아뇨 다른 이유는 없고 SwiftUI에서 UI 관련해서 여러 테스트를 해보다가 제거 하는걸 깜빡했던 거 같습니다~ 바로 지울게용
|
|
||
| Text("-") | ||
|
|
||
| Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요 |
Collaborator
There was a problem hiding this comment.
Suggested change
| Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요 | |
| Text("") // TeacherMain페이지 model 완성 후 그에 맞는 데이터 삽입 필요 |
주석이 있는데 무슨 뜻인지 이해를 못하겠어요...
Collaborator
Author
There was a problem hiding this comment.
TeacherMainView에 container가 만들어지고 나서 저 Text의 "" 자리에 걸맞는 걸 삽입한다는 뜻이었습니다!
Collaborator
Author
There was a problem hiding this comment.
저러한 의도 였다면 주석을 어떤식으로 다는게 좋을까요?? 의견 여쭤보고 싶습니다~
Collaborator
There was a problem hiding this comment.
PopupView는 container가 필요 없는 CustomView 아닌가요? TeacherMain의 Model이 만들어져야 하는 이유가 뭔지 궁금합니다. 삽입 한다는 말이 조금 애매하기도 해서 CustomView면 ""보다는 상수나 변수를 선언해서 넣어두는 편이 더 좋을 것 같아요
Collaborator
Author
There was a problem hiding this comment.
저는 그냥 CustomView 각각마다 container가 들어가야되는줄 알았습니다~ 그럼 그렇게 수정해서 다시 올리겠습니다~
MainMissionScrollView 코드 단축
StudentMainView 필요없는 코드 삭제
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-07-12.at.16.08.22.mp4
이 동영상은 popupView UI 동작 장면입니다.