Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: 변경된 밸런스게임 조회 api 파라미터 수정 #264

Merged
merged 47 commits into from
Dec 21, 2024
Merged
Show file tree
Hide file tree
Changes from 46 commits
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
0e89460
[style] : mobile typo 정의 및 Header mobile svg 추가
alwubin Dec 7, 2024
5785e4f
[feat] : viewport 메타 태그 추가 및 미디어 쿼리 기본 스타일 적용
alwubin Dec 8, 2024
7889a18
[feat] : LandingPage에 사용되는 모바일 svg 추가
alwubin Dec 8, 2024
bf378ee
[feat] : 모바일 Button 컴포넌트 구현
alwubin Dec 8, 2024
9e1cf04
[style] : 모바일 typo 추가
alwubin Dec 8, 2024
9b93298
[feat] : BalanceGameCategory 관련 모바일 스타일 추가
alwubin Dec 8, 2024
2e34267
[feat] : CategoryBox 관련 모바일 스타일 추가
alwubin Dec 8, 2024
6d7999c
[feat] : SearchBar 모바일 스타일 추가
alwubin Dec 8, 2024
9cbf5fa
[feat] : ContentsButton 모바일 스타일 추가
alwubin Dec 8, 2024
c79b191
[feat] : TopBanner 모바일 스타일 추가
alwubin Dec 8, 2024
d253605
[feat] : 모바일 layout 정의
alwubin Dec 8, 2024
9e04430
[feat] : Mobile 컴포넌트 관련 svg 파일 추가
alwubin Dec 8, 2024
5a64243
[feat] : LandingPage 속 새로운 mobile 컴포넌트 UI 및 스토리북 구현
alwubin Dec 8, 2024
640137f
[feat] : BalanceGameList 관련 모바일 스타일 추가
alwubin Dec 8, 2024
7c40e26
[style] MobileCreateButton hover style 및 배경 색상 추가
alwubin Dec 9, 2024
4df0893
[feat] useIsMobile 훅 구현 및 반응형 적용
alwubin Dec 9, 2024
da1caea
[refactor] : 모바일 UI 컴포넌트 폴더 구조 변경 및 적용
alwubin Dec 9, 2024
f9dfd3f
[feat] : DraftSaveButton 컴포넌트 UI 및 스토리북 구현
alwubin Dec 9, 2024
11e0e37
Merge branch 'dev' into feat/249-landing-mobile-ui
alwubin Dec 10, 2024
5f794bc
[chore] : 모바일 UI 관련 폴더 구조 정리 및 파일 이동
alwubin Dec 10, 2024
7f838d2
[refactor] : 스토리북 통일을 위해 Default/All로 수정
alwubin Dec 11, 2024
f7f8bd1
[refactor] : 모바일 UI 스토리북 폴더 구조 수정
alwubin Dec 11, 2024
1ca657c
[refactor] : TopBanner 모바일 dotStyling 수정
alwubin Dec 11, 2024
def69e0
[refactor] : 레이아웃 공백 요소의 불필요한 ARIA role 제거 및 접근성 개선
alwubin Dec 11, 2024
376bc7b
[refactor] : 접근성 개선을 위한 aria-hidden 추가
alwubin Dec 11, 2024
683110a
Update src/components/mobile/molecules/MobileCreateDropdown/MobileCre…
alwubin Dec 12, 2024
4bd8535
[refactor] accessToken 타입 처리 개선
alwubin Dec 12, 2024
0c7ea14
[refactor] : 로그인 핸들러 중복 로직 제거 및 개선
alwubin Dec 12, 2024
a87da2f
[style] typo 누락 스타일 추가
alwubin Dec 12, 2024
9aa65cb
[refactor] : 컴포넌트 네이밍 수정
alwubin Dec 12, 2024
9d33708
[refactor] : 스토리북 속 porps 일부 action 적용
alwubin Dec 12, 2024
408850c
[refactor] : 컴포넌트 네이밍 수정 및 적용
alwubin Dec 12, 2024
d62a7fc
[refactor] : 로그인 핸들러 중첩 함수 오류 수정
alwubin Dec 12, 2024
27311d2
[refactor] : 사용자 에이전트 문자열 검사 로직 개선
alwubin Dec 12, 2024
ca6ad6e
[refactor] : 사용자 에이전트 문자열 검사 로직 수정
alwubin Dec 12, 2024
d3c5a97
[refactor] : 모바일 토글 defaultItem을 통해 종속성 제거
alwubin Dec 13, 2024
d2f2cda
[refactor] : 태그 제거 및 cosole 함수 action 기능으로 수정
alwubin Dec 13, 2024
5e33858
[refactor] : 인코딩 처리 명시
alwubin Dec 16, 2024
1f6d5d5
[feat] : storybook custom viewport를 통해 모바일 UI 스토리북 구현
alwubin Dec 16, 2024
738de7e
[refactor] : 수정된 props 반영
alwubin Dec 16, 2024
7b02866
[refactor] : 밸런스게임 조회 파라미터 '인기' 태그에 대한 수정
alwubin Dec 16, 2024
c4a8688
[refactor] : 밸런스게임 태그 상태값 수정 및 적용
alwubin Dec 16, 2024
f27e036
[refactor] : 밸런스 게임에 대한 인기순/최신순 api 동시 호출 분리
alwubin Dec 16, 2024
e59e9a3
[refactor] : activeTab의 기본값을 빈 문자열에서 '인기'로 변경하고 tagName 처리 로직 수정
alwubin Dec 18, 2024
cce8f0e
[refactor] : params 타입을 GameParams로 정의 및 조건 수정
alwubin Dec 18, 2024
d689948
[refactor] : enabled 옵션을 통한 조건부 데이터 패칭 로직 수정
alwubin Dec 20, 2024
f0b8e84
Merge branch 'dev' into feat/263/fix-game-tag-state
alwubin Dec 21, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .storybook/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ const config: StorybookConfig = {
'@storybook/addon-essentials',
'@chromatic-com/storybook',
'@storybook/addon-interactions',
'@storybook/addon-viewport',
],
framework: {
name: '@storybook/react-webpack5',
Expand Down
21 changes: 20 additions & 1 deletion .storybook/preview.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,19 @@
import type { Preview } from '@storybook/react';
import { INITIAL_VIEWPORTS } from '@storybook/addon-viewport';
import '../src/styles/reset.css';

const customViewports = {
mobile: {
name: 'mobile',
styles: {
width: '430px',
height: '400px',
},
type: 'mobile',
},
};
Copy link
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

모바일 뷰포트 설정 개선 필요

현재 구성은 단일 모바일 해상도만 지원하고 있습니다. 다양한 모바일 기기를 지원하기 위해 추가 뷰포트 설정이 필요합니다.

다음과 같이 개선하는 것을 추천드립니다:

 const customViewports = {
   mobile: {
     name: 'mobile',
     styles: {
       width: '430px',
-      height: '400px',
+      height: '932px',
     },
     type: 'mobile',
   },
+  mobileSM: {
+    name: 'mobile-small',
+    styles: {
+      width: '360px',
+      height: '780px',
+    },
+    type: 'mobile',
+  },
+  mobileXL: {
+    name: 'mobile-large',
+    styles: {
+      width: '480px',
+      height: '1040px',
+    },
+    type: 'mobile',
+  },
 };
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
const customViewports = {
mobile: {
name: 'mobile',
styles: {
width: '430px',
height: '400px',
},
type: 'mobile',
},
};
const customViewports = {
mobile: {
name: 'mobile',
styles: {
width: '430px',
height: '932px',
},
type: 'mobile',
},
mobileSM: {
name: 'mobile-small',
styles: {
width: '360px',
height: '780px',
},
type: 'mobile',
},
mobileXL: {
name: 'mobile-large',
styles: {
width: '480px',
height: '1040px',
},
type: 'mobile',
},
};



const preview: Preview = {
parameters: {
controls: {
Expand All @@ -9,7 +22,13 @@ const preview: Preview = {
date: /Date$/i,
},
},
viewport: {
viewports: {
...INITIAL_VIEWPORTS,
...customViewports
},
},
},
};

export default preview;
export default preview;
Loading