Merged
Conversation
- toggle top/bottom bars via tap using detectTapGestures - animate bars with fade + slide transitions - replace magic bottom padding with navigationBars insets - rename showTopBar → showBars for clearer semantics - improve error retry tap handling
- Box background: pure black - Top bar with semi-transparent background, icons and text use white color on this page - Pill-style page indicator perf Removed WindowInsets.navigationBars calculation at the bottom, centralized all UI control elements at the top to reduce visual clutter. Used fontWeight = FontWeight.Bold to enhance number readability
… pinch zoom and pan gestures for images in ImagePreviewPage. Add double-tap zoom centered on the tap position and clamp offsets to image bounds. When the image is zoomed in, HorizontalPager scrolling is temporarily disabled to avoid gesture conflicts. Pager scrolling is restored when scale returns to 1f. Also allow tapping the image to toggle the top bar visibility. - Implement pinch zoom and pan gestures for images in ImagePreviewPage. Add double-tap zoom centered on the tap position and clamp offsets to image bounds. - When the image is zoomed in, HorizontalPager scrolling is temporarily disabled to avoid gesture conflicts. Pager scrolling is restored when scale returns to 1f.
Member
可以的 |
Member
|
hello 还在吗，我计划在几天后发布测试版，你如果写好了可以提交上来 |
Contributor
Author
在改，有些小瑕疵 |
- deps: add me.saket.telephoto:zoomable (0.18.0) to version catalog and app module.
- refactor(ImagePreviewPage):
- Normalize single/multiple URI inputs into a unified `previewUris` list.
- Implement immersive system bar handling using WindowInsetsController (hide on enter, restore on dispose).
- Replace manual scale/offset logic with `Modifier.zoomable()` for smoother pinch-to-zoom and double-tap gestures.
- Retain `AsyncImagePainter.State` branching to maintain existing Loading/Error UI feedback.
- Integrate `ZoomableContentLocation` with `intrinsicSize` to ensure precise content boundary detection and pager gesture co-operation.
- 预加载改成最多 2 个并发，整组图片会分批预取，不再一次性全冲出去。 - 去掉了 `newBuilder().build()` 的临时 loader，图片显示和预取都改成走 `LocalContext.current.imageLoader`，阶段监听挂到 `ImageRequest` 上。 - 规则组图片标题改成 `uri -> List<title>` 聚合，同一张图被多个子规则复用时，副标题会合并展示，不再只保留第一个命中的归属。 - 清除preKeys标题生成的多余强转
Contributor
Author
这回应该是没问题了
实机测试Screenrecording_20260403_011446_2026.04.03-01.21.00.mp4 |
Contributor
Author
|
麻烦大佬辛苦看下👍 |
lisonge
reviewed
Apr 13, 2026
cjy0812
commented
Apr 13, 2026
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
2 participants
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.
目前图片查看加阴影还是存在图片色与字色相似难以分辨问题
如图
具体做了
目前问题
Telephoto组件库以减少代码量，方便后期维护目前效果
Screenrecording_20260329_085526.mp4
TODO