-
Notifications
You must be signed in to change notification settings - Fork 3
chore : ui 변경 #150
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
chore : ui 변경 #150
Conversation
Walkthrough이 변경 사항은 chat-page.html과 game-page.html 파일에 커스텀 스크롤바 스타일을 추가하여, 주요 UI 요소의 스크롤바 외관을 WebKit 기반 브라우저와 Firefox에서 일관성 있게 개선합니다. 기능적 동작에는 변화가 없으며, 시각적 스타일링만 적용되었습니다. Changes
Suggested reviewers
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed 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)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/main/resources/templates/chat-page.html (1)
91-109: 스크롤바 스타일 공통화 제안
chat-page.html와game-page.html모두 거의 동일한 커스텀 스크롤바 정의를 중복 작성하고 있습니다.
향후 색상/굵기 등을 한 번에 조정할 수 있도록, 공통 CSS(예:/css/common-scrollbar.css)로 분리한 뒤 두 템플릿에서link태그로 불러오는 편이 유지보수에 유리합니다.
또한 WebKit·Firefox 외의 브라우저(Edge/Chromium 기반)에서도 동일하게 동작하는지 한 번 더 확인해주세요.src/main/resources/templates/game-page.html (1)
120-147: 불필요한 대상 포함 및 스타일 불일치 확인 요청
.button-container는 현재overflow속성이 없어 스크롤바가 표시되지 않습니다.
스크롤바 커스텀 대상에서 제외하거나, 향후 스크롤 가능 영역으로 변경될 계획인지 명확히 해두면 혼란을 줄일 수 있습니다.스크롤바 굵기가 이 파일에서는
10px,chat-page.html에서는8px으로 서로 다릅니다. 의도한 차이가 아니라면 통일을 권장합니다.두 파일 모두 색상·두께 값을 상수화(변수, CSS custom properties 등)해 두면 다크모드/밝은모드 전환 시 대응이 수월합니다.
확인 후 필요하다면 수정 부탁드립니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/main/resources/templates/chat-page.html(1 hunks)src/main/resources/templates/game-page.html(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
Summary by CodeRabbit