Skip to content

[Style] 자잘한 스타일 수정#547

Merged
namdaeun merged 3 commits into
developfrom
style/#546_qa
Jun 1, 2026
Merged

[Style] 자잘한 스타일 수정#547
namdaeun merged 3 commits into
developfrom
style/#546_qa

Conversation

@namdaeun

Copy link
Copy Markdown
Member

Summary

  • border radius 수정
  • 섹션 padding값 수정

Screenshot

Comment

@namdaeun namdaeun self-assigned this May 30, 2026
@vercel

vercel Bot commented May 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sopt-org-frontend Ready Ready Preview, Comment Jun 1, 2026 10:01am

@coderabbitai

coderabbitai Bot commented May 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d8714996-6726-4459-afae-c833dbe69813

📥 Commits

Reviewing files that changed from the base of the PR and between 73db8da and 028c2d9.

📒 Files selected for processing (1)
  • src/views/MainPage/components/IntroSection/CoreValueSection/CoreValueCard/style.ts
✅ Files skipped from review due to trivial changes (1)
  • src/views/MainPage/components/IntroSection/CoreValueSection/CoreValueCard/style.ts

Summary by CodeRabbit

  • Style
    • 코어 밸류 아이템 컴포넌트의 스타일 선언을 정리해 반응형에서의 간격 및 모서리 처리 일관성을 개선했습니다.
    • 최근 뉴스 컴포넌트의 하단 여백을 제거했습니다.
    • 코어 밸류 카드의 태블릿 뷰 패딩을 줄여 카드 내 여백을 조정했습니다.

Walkthrough

이 PR은 CoreValue 관련 스타일의 선언 순서와 반응형 블록 정리를 수행하고, RecentNews의 하단 패딩을 0으로 변경하며 CoreValueCard의 태블릿 패딩을 24px → 20px으로 조정합니다.

Changes

스타일 업데이트

Layer / File(s) Summary
CoreValue Item 속성 재정렬
src/views/AboutPage/components/CoreValue/Item/style.ts
ItemContainergapborder-radius 선언 순서를 변경하고, media.tablet/media.mobile 블록 내 border-radius 규칙을 재배치. BackgroundBlur의 반응형 블록 구성도 정리됨.
RecentNews 하단 패딩 제거
src/views/MainPage/components/RecentNews/style.ts
상단 padding 432px은 유지하며 하단 padding을 372px0으로 변경하여 하단 여백 제거.
CoreValueCard 태블릿 패딩 조정
src/views/MainPage/components/IntroSection/CoreValueSection/CoreValueCard/style.ts
CardContentmedia.tablet 내 padding을 24px20px로 수정.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly Related Issues

  • #546: 스타일 속성 재정렬과 미디어 쿼리 관련 변경이 일치합니다.

Poem

🐰 속성 순서를 정리하니,
패딩도 살짝 줄였네,
반응형 모서리 둥글게,
레이아웃이 조용히 웃고,
작은 변경에 빛이 나네 ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 실제 변경 사항과 관련이 있습니다. '자잘한 스타일 수정'은 border-radius와 padding 값 수정 같은 스타일 변경을 포함하는 변경 집합을 반영합니다.
Description check ✅ Passed PR 설명이 변경 집합과 관련이 있습니다. border radius 수정과 섹션 padding값 수정 항목이 실제 파일 변경과 일치합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch style/#546_qa

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adjusts styling properties, including reordering and adding border-radius values for tablet and mobile layouts in AboutPage, and updating the padding of RecentNews in MainPage. The review feedback suggests removing redundant border-radius declarations under media.mobile since they are already covered by media.tablet cascading styles.

Comment on lines 49 to 51
gap: 8px;
border-radius: 14px;
background-size: 134px 134px;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

media.tablet에서 이미 border-radius: 14px;가 정의되어 있고, media.tablet이 모바일 화면 크기까지 커버한다면 media.mobile에서의 중복 선언은 생략할 수 있습니다.

    gap: 8px;
    background-size: 134px 134px;

Comment on lines +73 to +78
${media.tablet} {
border-radius: 14px;
}
${media.mobile} {
border-radius: 14px;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

일반적으로 media.tablet은 모바일 기기의 화면 크기까지 포함하는 max-width 미디어 쿼리로 정의됩니다. 만약 그렇다면, media.tablet에서 설정한 border-radius: 14px; 스타일이 모바일 기기에도 자연스럽게 적용(cascade)되므로, media.mobile 블록을 중복해서 작성할 필요가 없습니다.

중복 코드를 제거하여 스타일 시트를 더 간결하고 유지보수하기 쉽게 개선하는 것을 제안합니다.

  ${media.tablet} {
    border-radius: 14px;
  }

@github-actions

Copy link
Copy Markdown

Preview URL 🚀 : https://soptorg-frontend-e5o3zbi2v-daeuns-projects-025386f4.vercel.app

@jeonghoon11 jeonghoon11 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!!

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

Preview URL 🚀 : https://soptorg-frontend-ielia8rbk-daeuns-projects-025386f4.vercel.app

@namdaeun namdaeun merged commit cfea127 into develop Jun 1, 2026
6 checks passed
@namdaeun namdaeun deleted the style/#546_qa branch June 1, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants