Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PuyL1VJDKFV2CL9XK86onP
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PuyL1VJDKFV2CL9XK86onP
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #32
무엇을 바꿨나
위젯을 아예 숨길 수 있고, 마지막 모습을
settings키에 저장한다. 영상별video:저장과 분리해서 영상을 옮겨도 같은 모습으로 뜬다. 숨기고 되돌리는 버튼은 패널 헤더에 있다.저장하는 값은
floatingHidden과floatingExpanded둘이다. 보임과 크기는 서로 다른 버튼이 다루므로 값도 나눈다. 하나로 합치면 한쪽을 바꿀 때 다른 쪽이 함께 덮여서, 펼쳐둔 채 숨긴 위젯이 접힌 모습으로 되돌아온다.상태의 주인이
floating.js의 모듈 변수에서wiring.js의state로 옮겨졌다. 저장되는 값이니 state에 있어야 하고, 덤으로 모듈 전역 둘(isExpanded,lastView)이 사라졌다.설명이 필요한 선택 둘:
✕를 더하면 접기(⌄)와 나란히 놓여 무엇이 무엇인지 헷갈리고, 숨긴 위젯을 되돌릴 자리도 결국 패널이어야 한다.storage.js가 아니라ui/floatingState.js가 따진다. 무엇이 올바른 값인지 아는 쪽이 ui다. 참이라고 분명히 적힌 것만 참으로 보고, 나머지는 기본값이 된다.확인
npm test137개 통과,npm run test:e2e24개 통과.미확인
픽스처는 셀렉터가 낡은 것을 잡지 못한다. 실제 유튜브에서 확인할 것:
위젯 숨기기를 누른다 → 위젯이 사라지고 버튼이위젯 보이기로 바뀐다.위젯 보이기를 누르면 돌아온다.🤖 Generated with Claude Code
https://claude.ai/code/session_01PuyL1VJDKFV2CL9XK86onP