Skip to content

Comments

[style] 모바일 검색창 크기 키우기#361

Merged
seongwon030 merged 11 commits intodevelop-fefrom
style/#359-mobile-searchbar-ui-update-FE-99
May 5, 2025
Merged

[style] 모바일 검색창 크기 키우기#361
seongwon030 merged 11 commits intodevelop-fefrom
style/#359-mobile-searchbar-ui-update-FE-99

Conversation

@seongwon030
Copy link
Member

@seongwon030 seongwon030 commented May 1, 2025

#️⃣연관된 이슈

ex) #359

📝작업 내용

  1. SearchBoxContainer의 justify-content 속성 space-between에서 center로 변경
  2. margin 확보 위해 input 태그 flex: 1width: calc(100%-32px)로 설정

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

Summary by CodeRabbit

  • Style
    • 모바일 헤더와 아이콘 크기가 전체적으로 커졌으며, 모바일 환경에서 헤더와 카테고리 버튼의 위치가 상향 조정되었습니다.
    • 검색창의 가로 폭과 정렬, 패딩, 버튼 크기 등이 모바일 환경에 맞게 개선되었습니다.
    • 배너의 상단 여백이 모바일 화면에서 더 넓어졌습니다.

@seongwon030 seongwon030 added 🎨 Design 마크업 & 스타일링 💻 FE Frontend labels May 1, 2025
@seongwon030 seongwon030 requested a review from oesnuj May 1, 2025 05:39
@seongwon030 seongwon030 self-assigned this May 1, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented May 1, 2025

"""

Walkthrough

이 변경사항은 모바일 환경에서의 헤더, 검색창, 배너, 카테고리 버튼 리스트의 스타일을 조정하는 내용입니다. 구체적으로, 모바일 헤더와 아이콘의 크기가 커졌고, 검색창의 너비와 높이, 정렬, 패딩 등이 조정되었습니다. 배너와 카테고리 버튼 컨테이너의 상단 마진 및 위치 오프셋도 모바일 화면에서 증가되었습니다. 모든 변경은 스타일 속성 값의 수정에 한정되며, 컴포넌트의 공개 인터페이스나 로직에는 영향을 주지 않습니다.

Changes

파일 그룹 변경 요약
frontend/src/components/common/Header/Header.styles.ts 모바일 헤더 및 아이콘 크기 증가, 내부 이미지 크기 및 버튼 크기 조정
frontend/src/components/common/SearchBox/SearchBox.styles.ts 검색창 너비·높이·정렬·패딩 조정, 모바일용 폰트 크기 및 버튼 크기 조정, margin-top 제거
frontend/src/pages/MainPage/components/Banner/Banner.styles.ts 모바일(≤500px)에서 배너 상단 마진 42px→56px로 증가
frontend/src/pages/MainPage/components/CategoryButtonList/CategoryButtonList.styles.ts 모바일(≤500px)에서 카테고리 버튼 컨테이너 top 오프셋 46px→56px로 증가

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant MobileHeader
    participant SearchBox
    participant Banner
    participant CategoryButtonList

    User->>MobileHeader: 모바일 화면 진입
    MobileHeader-->>User: 확대된 헤더 및 아이콘 표시

    User->>SearchBox: 검색창 접근
    SearchBox-->>User: 확대된 검색창 UI 제공

    User->>Banner: 메인 배너 확인
    Banner-->>User: 상단 마진이 늘어난 배너 표시

    User->>CategoryButtonList: 카테고리 버튼 확인
    CategoryButtonList-->>User: 상단 오프셋이 증가된 버튼 리스트 표시
Loading

Assessment against linked issues

Objective Addressed Explanation
모바일 검색창 크기 키우기 (FE-99)

Suggested reviewers

  • oesnuj
    """

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6fdf603 and 37d772f.

📒 Files selected for processing (1)
  • frontend/src/components/common/Header/Header.styles.ts (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • frontend/src/components/common/Header/Header.styles.ts
✨ Finishing Touches
  • 📝 Generate Docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@netlify
Copy link

netlify bot commented May 1, 2025

Deploy Preview for moadong ready!

Name Link
🔨 Latest commit 37d772f
🔍 Latest deploy log https://app.netlify.com/sites/moadong/deploys/6818df32f69ffb00070d9e13
😎 Deploy Preview https://deploy-preview-361--moadong.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 40
Accessibility: 87
Best Practices: 92
SEO: 92
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0ac19ac and 756aadd.

📒 Files selected for processing (4)
  • frontend/src/components/common/Header/Header.styles.ts (2 hunks)
  • frontend/src/components/common/SearchBox/SearchBox.styles.ts (3 hunks)
  • frontend/src/pages/MainPage/components/Banner/Banner.styles.ts (1 hunks)
  • frontend/src/pages/MainPage/components/CategoryButtonList/CategoryButtonList.styles.ts (1 hunks)
🔇 Additional comments (11)
frontend/src/pages/MainPage/components/Banner/Banner.styles.ts (1)

16-17: 모바일 화면에서 배너 상단 여백 증가 적용이 적절합니다.

모바일 화면(max-width: 500px)에서 배너 컨테이너의 상단 여백을 42px에서 56px로 증가시킨 변경은 다른 컴포넌트(헤더, 카테고리 버튼 리스트)와의 일관된 수직 간격을 유지하는 데 도움이 됩니다. 헤더 높이도 56px로 조정되었기 때문에 이러한 일관성은 모바일 UI에서 중요합니다.

frontend/src/components/common/SearchBox/SearchBox.styles.ts (4)

5-7: 검색창 정렬 및 너비 조정이 사용성을 향상시킵니다.

justify-content를 space-between에서 center로 변경하고 너비를 300px에서 345px로 확장한 것은 모바일 환경에서 검색창의 가시성과 사용성을 향상시킵니다. 중앙 정렬은 더 균형 잡힌 디자인을 제공합니다.


8-9: 패딩 조정이 검색창의 내부 공간을 최적화합니다.

수직 패딩을 10px에서 3px로 줄인 것은 검색창 내부의 공간을 효율적으로 사용하면서도 시각적으로 충분한 공간을 제공합니다.


43-46: 모바일 환경에서 폰트 크기 일관성 확보가 좋습니다.

max-width 500px 미디어 쿼리에서 폰트 크기를 14px로 설정한 것은 좋은 선택입니다. 이는 max-width 550px에서 설정한 10px보다 크며, 더 작은 화면에서도 가독성을 유지하는 데 도움이 됩니다.


74-77: 검색 버튼 크기 조정이 터치 경험을 개선합니다.

모바일 환경에서 검색 버튼의 크기를 명시적으로 설정한 것은 터치 상호작용을 위한 적절한 타겟 영역을 제공합니다. 이는 모바일 사용성 향상에 기여합니다.

frontend/src/pages/MainPage/components/CategoryButtonList/CategoryButtonList.styles.ts (1)

13-14: 카테고리 버튼 컨테이너의 위치 조정이 UI 일관성을 유지합니다.

sticky 포지션의 top 값을 46px에서 56px로 증가시킨 것은 모바일 헤더 높이 증가(56px)에 맞춰 조정한 것으로 보입니다. 이렇게 하면 스크롤 시 카테고리 버튼이 헤더와 겹치지 않고 적절한 간격을 유지할 수 있습니다.

frontend/src/components/common/Header/Header.styles.ts (5)

96-97: 모바일 헤더 높이 증가가 터치 영역을 개선합니다.

모바일 헤더 컨테이너의 높이를 46px에서 56px로 증가시킨 것은 더 넓은 터치 영역을 제공하고, 다른 컴포넌트들(배너, 카테고리 버튼 리스트)의 간격 조정과 일관성을 유지합니다.


112-115: 메인 아이콘 크기 증가가 가시성을 향상시킵니다.

모바일 메인 아이콘의 크기를 증가시키고 명시적인 aspect-ratio를 추가한 것은 아이콘의 가시성을 향상시키고 비율을 일관되게 유지하는 데 도움이 됩니다.


120-121: 이미지 크기 조정이 버튼 크기와 일치합니다.

메인 아이콘 내부 이미지의 너비를 버튼 크기와 일치하도록 조정한 것은 일관된 시각적 표현을 위해 중요합니다.


127-129: 메뉴 버튼 크기 증가가 사용성을 개선합니다.

메뉴 버튼의 크기를 증가시킨 것은 모바일 환경에서 터치 정확도를 향상시키고 사용자 경험을 개선합니다.


134-135: 이미지 크기 조정이 버튼 크기와 일치합니다.

메뉴 버튼 내부 이미지의 너비를 버튼 크기와 일치하도록 조정한 것은 일관된 시각적 표현을 유지합니다.

@seongwon030 seongwon030 linked an issue May 1, 2025 that may be closed by this pull request
2 tasks
Copy link
Member

@oesnuj oesnuj left a comment

Choose a reason for hiding this comment

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

반영 전후 같이 띄워서 보니까 차이가 확실하네요
훨씬 누르기 편할 것 같습니다.

@seongwon030 seongwon030 merged commit 531be86 into develop-fe May 5, 2025
5 checks passed
@seongwon030 seongwon030 deleted the style/#359-mobile-searchbar-ui-update-FE-99 branch May 5, 2025 16:00
@coderabbitai coderabbitai bot mentioned this pull request Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 Design 마크업 & 스타일링 💻 FE Frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[style] FE-99 모바일 검색창 크기 키우기

2 participants