Skip to content

release/1.3.2 - #7

Merged
ienground merged 4 commits into
mainfrom
release/1.3.2
Jun 9, 2026
Merged

release/1.3.2#7
ienground merged 4 commits into
mainfrom
release/1.3.2

Conversation

@ienground

Copy link
Copy Markdown
Owner

No description provided.

ienground added 2 commits June 9, 2026 19:40
- libs.versions.toml 파일의 라이브러리 버전을 1.3.1에서 1.3.2로 업데이트
- TapTarget.kt 파일의 코드 구조를 개선하여 컴포지션 로직을 최적화
- bringIntoView 기능을 위한 Offset 계산 로직을 개선하여 UX 향상
- TapTarget.kt 파일에서 Compose 상태 관련 import 정리 및 변경
- App.kt 파일에서 화면 레이아웃 및 탭 타겟 설정 코드 개선
- 코드 구조 개선을 위한 파일 수정 및 정리

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

이번 풀요청은 탭 타겟을 화면에 표시할 때 수직 오프셋을 지정할 수 있는 bringIntoViewVerticalOffset 기능을 추가하고 관련 라이브러리 버전을 1.3.2로 업데이트합니다. 리뷰어는 bringIntoViewVerticalOffset이 동적으로 변경될 때 tapTargetScope가 올바르게 갱신되도록 remember 키에 추가할 것과, 레이아웃 계층에 연결되지 않은 상태에서 발생할 수 있는 예외를 방지하기 위해 target.coordinates.isAttached 여부를 확인하는 방어 코드를 추가할 것을 제안했습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread taptarget/src/commonMain/kotlin/zone/ien/taptargetcmp/TapTarget.kt Outdated
Comment thread taptarget/src/commonMain/kotlin/zone/ien/taptargetcmp/TapTarget.kt Outdated
ienground and others added 2 commits June 9, 2026 19:49
…t.kt

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
- TapTargetCoordinator의 API 문서화 및 설명 추가
- README.md에 bringIntoView 기능 사용법 및 예제 코드 추가
- 타겟 요소의 자동 스크롤 기능에 대한 설명 및 파라미터 설명 업데이트
@ienground
ienground merged commit 4010756 into main Jun 9, 2026
2 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