Skip to content

fix: 업데이트 배너 클릭과 위치 이동 문제 수정 - #61

Merged
jeongph merged 2 commits into
mainfrom
feature/fix-update-button-unresponsive
Oct 9, 2026
Merged

jeongph merged 2 commits into
mainfrom
feature/fix-update-button-unresponsive

Conversation

@jeongph

@jeongph jeongph commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

빈 화면에 업데이트 배너가 겹칠 때 버튼 입력이 창 드래그로 처리될 여지를 제거하고, 배너가 등장하면서 가로로 튀는 문제를 수정합니다. 창 이동은 상단 바로 제한하고 업데이트 배너의 다운로드·재시작·닫기 버튼에 no-drag를 직접 지정했습니다.

배너는 중앙 토스트의 translateX(-50%) 애니메이션 대신 투명도만 바꾸는 전용 애니메이션을 사용합니다. 표시 위치와 클릭 영역이 움직이지 않습니다. 이 수정을 배포할 수 있도록 package.json과 lockfile 버전을 0.11.1로 올렸습니다.

검증:

  • npm run build: 타입 검사와 프로덕션 빌드 통과
  • npm run lint, Prettier, git diff --check 통과
  • npm test: 137개 테스트 통과
  • 실제 빌드한 renderer를 띄운 Electron 창에서 모의 업데이트로 다운로드·재시작·닫기 클릭 확인
  • 다운로드 버튼과 빈 본문 위 포인터 드래그 시 창 위치 유지, 배너 애니메이션 전후 위치 동일 확인

실제 업데이트 파일의 다운로드·설치는 모의 핸들러로 대체했으며, 릴리즈 파일의 빌드·서명·공증은 태그 푸시 후 기존 release 워크플로로 검증합니다.

Summary by CodeRabbit

  • 버그 수정
    • 업데이트 배너가 투명도 애니메이션과 함께 표시됩니다.
    • 배너 버튼을 클릭할 수 있도록 조정했습니다.
    • 대화가 비어 있을 때 본문 영역을 창 드래그 영역으로 사용하지 않도록 변경했습니다.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: followingseas/wake/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f8487136-b76d-4c91-b5bd-83c38b59bb89

📥 Commits

Reviewing files that changed from the base of the PR and between e99bcaa and daac46f.


⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (2)
  • package.json
  • src/renderer/src/styles/app.css

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

패키지 버전을 0.11.0에서 0.11.1로 변경했습니다. 빈 대화 화면의 창 드래그 선언을 제거하고, 업데이트 배너에 투명도 전용 등장 애니메이션을 적용했습니다. 배너 버튼은 드래그 제외 영역으로 지정했습니다.

Changes

업데이트 배너

Layer / File(s) Summary
배너 표시 및 버전 갱신
src/renderer/src/styles/app.css, package.json
빈 대화 화면에서 창 드래그 선언을 제거했습니다. 업데이트 배너에 투명도 전용 등장 애니메이션을 적용하고 버튼을 드래그 제외 영역으로 지정했습니다. 패키지 버전을 0.11.1로 변경했습니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix


Merge Risk: ⚪ Minimal · up to daac4

No actionable merge-blocking issue is identified in the described changes.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed PR 제목은 업데이트 배너의 클릭 문제와 위치 이동 문제를 명확히 설명합니다. 변경 내용의 주요 목적과 일치합니다.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.


✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@jeongph
jeongph merged commit 8f61c5c into main Oct 9, 2026
3 checks passed
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