feat: 로그인 화면 UI 리팩토링 및 구조 개선 - #18
Open
kiho-0 wants to merge 1 commit into
Open
Conversation
chiyyy
requested changes
Jul 22, 2026
chiyyy
left a comment
chiyyy
left a comment
Contributor
There was a problem hiding this comment.
전체적으로 의아한 부분들이 많은데 개발하신 부분 캡쳐해서 보내주시면 감사하겠습니다
Comment on lines
-200
to
205
| <View className="relative mt-3 overflow-hidden rounded-card border border-border"> | ||
| <NaverMapView | ||
| style={{ height: 160 }} | ||
| initialCamera={{ | ||
| latitude: MOCK_WARD_LOCATION.latitude, | ||
| longitude: MOCK_WARD_LOCATION.longitude, | ||
| zoom: 15, | ||
| }} | ||
| isScrollGesturesEnabled={false} | ||
| isZoomGesturesEnabled={false} | ||
| isTiltGesturesEnabled={false} | ||
| isRotateGesturesEnabled={false} | ||
| isStopGesturesEnabled={false} | ||
| isShowLocationButton={false}> | ||
| <NaverMapMarkerOverlay | ||
| latitude={MOCK_WARD_LOCATION.latitude} | ||
| longitude={MOCK_WARD_LOCATION.longitude} | ||
| /> | ||
| </NaverMapView> | ||
| <Pressable onPress={onPressMore} className="absolute inset-0" /> | ||
| </View> | ||
| {/* TODO: features/location 실제 지도(react-native-maps 등) 컴포넌트로 교체 */} | ||
| <Pressable | ||
| onPress={onPressMore} | ||
| className="mt-3 items-center justify-center rounded-card border border-border bg-surface py-16"> | ||
| <Text className="text-sm text-text-muted">지도 (준비 중, 눌러서 위치 확인으로 이동)</Text> | ||
| </Pressable> | ||
| </SectionCard> | ||
| ); |
Contributor
There was a problem hiding this comment.
이미 구현 완료된 섹션을 왜 다시 준비중으로 바꾸신건지 모르겠습니다 수정해주세요
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.
Login.types.ts (타입 정의)
역할: 로그인 화면에서 사용되는 상태값(state)과 이벤트 핸들러 함수들의 타입을 인터페이스로 정의합니다.
useLogin.ts (커스텀 훅 - 비즈니스 로직)
역할: 아이디(id), 비밀번호(password) 입력 상태 관리 및 로그인 버튼 클릭, 회원가입, 아이디/비밀번호 찾기 등의 이벤트 핸들러 함수 로직을 독립된 훅으로 분리합니다.
Login.styles.ts (스타일)
역할: React Native의 StyleSheet.create를 사용해 로그인 화면의 모든 컴포넌트 스타일(컨테이너, 인라인 입력창, 버튼, 타이틀, 소셜 로그인 레이아웃 등)을 관리합니다.
LoginView.tsx
역할: useLogin 훅으로부터 전달받은 상태와 핸들러를 화면에 그려주는 역할(JSX 렌더링)만 담당합니다.
LoginScreen.tsx
역할: useLogin 훅을 호출하여 필요한 데이터와 함수를 가져온 뒤, 이를 LoginView 컴포넌트에 props로 전달해 주는 컨테이너 역할을 합니다.