feat: 홈 화면 오늘의 스케줄 섹션 구현 - 2#185
Merged
Virtuso1225 merged 4 commits intofeat/home-clubsfrom Mar 1, 2025
Merged
Conversation
halionaz
approved these changes
Feb 26, 2025
Member
halionaz
left a comment
halionaz
left a comment
There was a problem hiding this comment.
굿굿
코멘트만 가볍게 한번씩 읽어주시면 감사하겠슴다~
Member
There was a problem hiding this comment.
제쪽에서 아직 heroicon 사용하고 있는 부분이 있을 수도 있어서 머지할때 한번 체크 필요할듯 합니당
|
|
||
| const HomeContentsBase = () => { | ||
| const { authState } = useAuth() | ||
| const showSchedule = authState |
Member
There was a problem hiding this comment.
p5;
취향 차이일 거 같긴 한데, 이 경우에는 isLogin으로 이름지어주는게
ui 논리 플로우를 읽는 데 더 명확할 거 같다는 갠적인 생각이 있습니당
Suggested change
| const showSchedule = authState | |
| const isLogin = authState |
src/ui/Typography/index.tsx
Outdated
Comment on lines
23
to
25
| const typographyVariant = variant ?? (isMobile ? 'mobile' : 'desktop') | ||
| const typographyStyle = | ||
| variant === 'desktop' | ||
| typographyVariant === 'desktop' |
* feat: 스케줄 모바일 뷰 적용 * feat: 강의평 아이템 모바일 뷰 반영 * feat: fake timetable 사이즈 조정 * feat: 홈 화면 고려대 대표 기관 모바일 대응 - 4 (#187) * feat: 홈 화면 고려대 대표 기관 모바일 대응 * chore: 다른 컴포넌트 다시 호출하기 * feat: 홈 화면 간격 조정 * feat: 모바일 뷰에서 로그인 박스 없애기 * feat: 홈 화면 동아리 섹션 모바일 대응 - 5 (#188) * feat: 홈 화면 동아리 섹션 모바일 대응 * feat: mionr 한 디자인 변경사항 반영 * feat: 현재 배너 indicator mobile 대응
Virtuso1225
added a commit
that referenced
this pull request
Mar 1, 2025
* feat: 동아리 섹션 * feat: 홈 화면 상단 영역에서 로그인 하지 않은 상태의 UI 완성 * feat: 홈 화면 오늘의 스케줄 섹션 구현 - 2 (#185) * feat: 홈 화면 스케줄 구현 * feat: typo 에 자동 mobile, desktop 변환 넣기 * feat: 스케줄 모바일 뷰 적용 - 3 (#186) * feat: 스케줄 모바일 뷰 적용 * feat: 강의평 아이템 모바일 뷰 반영 * feat: fake timetable 사이즈 조정 * feat: 홈 화면 고려대 대표 기관 모바일 대응 - 4 (#187) * feat: 홈 화면 고려대 대표 기관 모바일 대응 * chore: 다른 컴포넌트 다시 호출하기 * feat: 홈 화면 간격 조정 * feat: 모바일 뷰에서 로그인 박스 없애기 * feat: 홈 화면 동아리 섹션 모바일 대응 - 5 (#188) * feat: 홈 화면 동아리 섹션 모바일 대응 * feat: mionr 한 디자인 변경사항 반영 * feat: 현재 배너 indicator mobile 대응 * feat: PR 리뷰 반영
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
2 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.
📌 내용
홈 화면의 스케줄 부분의 작업을 했어요
대부분 디자인 구현입니당.
☑️ 체크 사항
❗ Related Issues