Skip to content

Comments

fix: qa-button#178

Merged
dasosann merged 2 commits intomainfrom
fix/rending-qa2
Feb 15, 2026
Merged

fix: qa-button#178
dasosann merged 2 commits intomainfrom
fix/rending-qa2

Conversation

@dasosann
Copy link
Contributor

요약

구현 사항

  • [ ]
  • [ ]
  • [ ]
  • [ ]

📸 스크린샷

Need Review

  • ~ 부분 이렇게 구현했어요, 피드백 부탁해요!

Reference

📜 리뷰 규칙

Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.

  • P1: 꼭 반영해주세요 (Comment)
  • P2: 적극적으로 고려해주세요 (Comment)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@vercel
Copy link

vercel bot commented Feb 15, 2026

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

Project Deployment Actions Updated (UTC)
fe Ready Ready Preview, Comment Feb 15, 2026 3:57pm

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @dasosann, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 렌딩 페이지의 사용자 인터페이스를 개선하는 데 중점을 둡니다. 캐러셀 콘텐츠의 텍스트를 간결하게 다듬고, 인디케이터의 동작 방식을 고정하여 사용자 경험을 향상시켰습니다. 또한, 모바일 환경에서의 레이아웃 안정성을 위해 뷰포트 높이 단위를 조정했습니다.

Highlights

  • 렌딩 페이지 캐러셀 텍스트 간결화: 캐러셀 슬라이드의 제목에서 '이벤트마다'라는 반복적인 문구를 제거하여 텍스트를 더 간결하게 만들었습니다.
  • 캐러셀 인디케이터 고정: 캐러셀 인디케이터(점들)가 더 이상 캐러셀 내용과 함께 스와이프되지 않고, 화면 하단에 고정되어 표시되도록 변경했습니다.
  • 동적 뷰포트 높이 적용: 루트 컨테이너의 높이를 h-screen에서 h-dvh로 변경하여 모바일 환경에서 뷰포트 높이 계산의 정확성을 개선했습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • src/feature/root/components/RendingBody.tsx
    • 캐러셀 슬라이드 제목에서 '이벤트마다' 문구를 제거했습니다.
    • 캐러셀 인디케이터를 스와이프 가능한 영역에서 분리하여 고정된 위치에 렌더링하도록 구조를 변경했습니다.
    • 인디케이터 관련 주석을 업데이트했습니다.
  • src/feature/root/components/ScreenNewRoot.tsx
    • 루트 컨테이너의 높이 설정을 h-screen에서 h-dvh로 변경했습니다.
Activity
  • 활동 내역이 제공되지 않았습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

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

Choose a reason for hiding this comment

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

Code Review

이번 PR에서는 랜딩 페이지의 UI 구조를 개선하고 모바일 뷰포트 대응을 강화하셨네요. 주요 변경 사항인 ScreenNewRoot.tsxh-dvh 도입은 모바일 브라우저의 동적 뷰포트 이슈를 해결하는 좋은 접근입니다. 또한 RendingBody.tsx에서 인디케이터를 캐러셀 외부로 분리하여 사용자 경험을 개선한 점도 훌륭합니다. 다만, 웹 접근성(Accessibility) 향상을 위해 클릭 가능한 요소들에 button 태그와 type='button' 속성을 명시적으로 사용하는 것을 권장하며, h-dvh 적용에 따른 하위 컴포넌트들의 높이 설정 일관성도 함께 확인해 주시면 더욱 좋을 것 같습니다.

Comment on lines +168 to +170
<button
key={idx}
onClick={() => api?.scrollTo(idx)}
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

button 요소에는 type='button' 속성을 명시하는 것이 좋습니다. 이는 폼(form) 내부에 버튼이 위치할 경우 의도치 않게 서브밋(submit)되는 것을 방지하며, 브라우저가 요소의 역할을 명확히 인지하도록 돕습니다.

            <button
              type='button'
              key={idx}
              onClick={() => api?.scrollTo(idx)}

@dasosann dasosann merged commit b66d489 into main Feb 15, 2026
3 of 4 checks passed
@dasosann dasosann deleted the fix/rending-qa2 branch February 15, 2026 15:56
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.

1 participant