Skip to content

FLYW-207 fix: 좌석 선택 & 예약 페이지 UI 변경 (#248)#253

Merged
cl-o-lc merged 1 commit intodevfrom
FLYW-207-등급이-다르면-좌석-줄바꿈
Feb 3, 2026

Hidden character warning

The head ref may contain hidden characters: "FLYW-207-\ub4f1\uae09\uc774-\ub2e4\ub974\uba74-\uc88c\uc11d-\uc904\ubc14\uafc8"
Merged

FLYW-207 fix: 좌석 선택 & 예약 페이지 UI 변경 (#248)#253
cl-o-lc merged 1 commit intodevfrom
FLYW-207-등급이-다르면-좌석-줄바꿈

Conversation

@kkhhmm3103
Copy link
Contributor

@kkhhmm3103 kkhhmm3103 commented Feb 3, 2026

📌 PR 설명


UI 수정 ## ✅ 완료한 기능 명세
  • 좌석 선택 내역 색상 변경
  • 좌석 선택 버튼 이미지 삭제

📸 스크린샷


💭 고민과 해결과정


🔗 관련 이슈

Closes #248


Summary by CodeRabbit

스타일

  • 좌석 선택 버튼의 아이콘을 제거했습니다.
  • 좌석 정보 카드의 글꼴 크기와 색상을 조정하여 가독성을 개선했습니다.
  • 미니맵의 비행기 시각 요소를 수정했습니다.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

개요

좌석 예약 UI의 시각적 요소들을 조정하는 변경사항입니다. 버튼의 SVG 아이콘을 제거하고, 좌석 요약 카드와 행의 폰트 크기를 증가시키며, 색상을 변경하고, 미니맵에서 노즈콘을 제거하고 조종실 위치를 조정했습니다.

변경사항

집합 / 파일(들) 요약
예약 페이지
src/main/webapp/WEB-INF/views/reservations/booking.jsp
"선택하기" 버튼에서 인라인 SVG 아이콘 제거. 버튼의 기능에는 변화 없음.
좌석 컴포넌트 스타일
src/main/webapp/resources/seat/css/component.css
좌석 요약 카드와 행의 폰트 크기 증가 (15px→17px, 14px→16px, 13px→14px 등). 날짜/시간 색상을 #3b82f6에서 #64748b로 변경. 활성 상태 행의 이름 색상 스타일 제거. 활성 버튼 이름 색상을 #1d4ed8에서 #1e293b로 변경.
미니맵 스타일
src/main/webapp/resources/seat/css/minimap.css
노즈콘(노즈) 요소의 CSS 블록 완전 제거. 조종실 창 위치를 top: -18px에서 top: 20px로 변경하여 하향 이동.

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~10분

관련 PR

추천 레이블

FIX, ready-for-review

추천 리뷰어

  • ochanhyeok
  • minseokim0113

시의 축배

🐰 아이콘 사라지고, 폰트는 커지니
색은 또 다른 빛으로 물들어
노즈콘은 조종실 아래로 숨겨지고
좌석 예약의 얼굴이 다시 그려진다
소소한 손길로 완성되는 아름다움 ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive 연결된 이슈 #248은 구체적인 구현 요구사항 없이 작업 항목이 비어있으며, PR의 UI 변경사항과의 명확한 연결고리가 불명확합니다. 이슈 #248의 '등급이 다르면 좌석 줄바꿈' 요구사항과 현재 변경사항(색상, 이미지 삭제, 폰트 크기 조정)의 구체적인 연관성을 명시해주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 주요 변경 사항인 UI 수정(색상 변경, 이미지 삭제, CSS 스타일 조정)을 명확히 설명하고 있습니다.
Out of Scope Changes check ✅ Passed CSS 폰트 크기 및 색상 변경, SVG 이미지 삭제, 미니맵 노즈 요소 제거 등의 변경사항들이 모두 UI 수정 범위에 포함되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch FLYW-207-등급이-다르면-좌석-줄바꿈

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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.

@kkhhmm3103 kkhhmm3103 added FIX 버그 수정 ready-for-review PR 리뷰 요청 labels Feb 3, 2026
Copy link
Contributor

@ochanhyeok ochanhyeok left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

Copy link
Contributor

@cl-o-lc cl-o-lc left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@cl-o-lc cl-o-lc merged commit 94da316 into dev Feb 3, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FIX 버그 수정 ready-for-review PR 리뷰 요청

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FLYW-207] 등급이 다르면 좌석 줄바꿈

3 participants