Skip to content

feat: Enter opens a tree file fullscreen instead of expanding - #8

Merged
code0xff merged 6 commits into
code0xff:devfrom
whackur:dev
Jul 31, 2026
Merged

feat: Enter opens a tree file fullscreen instead of expanding#8
code0xff merged 6 commits into
code0xff:devfrom
whackur:dev

Conversation

@whackur

@whackur whackur commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

What

Tree 뷰에서 Enter의 의미를 바꾼다.

  • Tree 뷰 (Focus::FileList)Enter는 더 이상 디렉터리를 확장/접지 않는다. 확장은 / 전용이다. 파일 행에서 Enter는 파일을 프리뷰 패널에 로드하고 그 패널을 fullscreen으로 키우며 포커스를 옮긴다. 디렉터리 행에서는 아무 일도 하지 않는다.
  • Diff/file 뷰어 (Focus::DiffViewer)Enter가 fullscreen을 토글한다 (<prefix> f와 동일). 들어간 키로 그대로 나올 수 있다.

확장에 이미 방향키가 있으므로 Enter는 파일 열기에 쓰는 편이 낫다는 판단.

How

  • focus.rsset_diff_fullscreen(on)을 분리했다. "diff fullscreen 진입 시 terminal/list fullscreen 해제 + 포커스 이동" 규칙을 토글과 강제 진입이 한 곳에서 공유한다.
  • tree_toggletree_open_selected (다른 호출자 없음).
  • hint bar, README 단축키 표·트리 설명, docs/architecture.md의 "트리 뷰는 Enter도 확장한다" 문장을 갱신했다.

검색 입력 중의 Enterhandle_diff_search_key / handle_tree_search_keyhandle_unmapped_upper_key보다 먼저 가로채므로 기존 동작(검색 확정, 매치 reveal)이 그대로다.

Tests

  • tree_open_on_directory_row_does_not_change_expansion
  • tree_open_on_file_row_loads_file_view_and_goes_fullscreen
  • tree_open_on_file_row_clears_competing_fullscreens
  • enter_in_diff_viewer_toggles_diff_fullscreen

cargo test 1328 passed / 0 failed, cargo clippy --all-targets --all-features -- -D warnings clean.

참고

이 PR은 fork의 dev 브랜치를 통째로 올린 것이라, 위 변경 외에 아직 upstream에 없던 커밋 3개(9ee66f7, b2d0138, 684d676 — 테스트 환경 관련 수정과 설정 정리)도 함께 들어 있다.

@whackur
whackur requested a review from code0xff July 31, 2026 04:43
@code0xff
code0xff merged commit 3bcae30 into code0xff:dev Jul 31, 2026
5 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.

2 participants