Skip to content

Fix: square thumbnail in arist#294

Merged
rukamori merged 8 commits into
ArchiveTuneApp:devfrom
sang765:fix/square-thumbnail-in-arist
Feb 8, 2026
Merged

Fix: square thumbnail in arist#294
rukamori merged 8 commits into
ArchiveTuneApp:devfrom
sang765:fix/square-thumbnail-in-arist

Conversation

@sang765

@sang765 sang765 commented Feb 7, 2026

Copy link
Copy Markdown
Member

This pull request addresses a UI issue where square cropping was being incorrectly applied to non-square thumbnails, leading to visual distortions. By introducing an explicit aspect ratio check, the system now ensures that square cropping is only activated when the thumbnail naturally has a 1:1 ratio, thereby enhancing the visual consistency and correctness of thumbnail displays across the application.

Highlights

  • Square Crop Logic Refinement: The condition for applying square cropping to thumbnails has been updated to be more precise.
  • Aspect Ratio Enforcement: An additional check has been introduced to ensure that square cropping is only applied when the thumbnail's aspect ratio is exactly 1:1.
  • UI Bug Fix: This change prevents incorrect or distorted cropping on non-square thumbnails, improving the visual integrity of the UI.
Changelog
  • app/src/main/kotlin/moe/koiverse/archivetune/ui/component/Items.kt
    • Modified the shouldApplySquareCrop boolean logic within the ItemThumbnail composable to include a thumbnailRatio == 1f check.
    • Modified the shouldApplySquareCrop boolean logic within the LocalThumbnail composable to include a thumbnailRatio == 1f check.

Video

Screenrecorder-20260208-032048.mp4

sang765 and others added 8 commits February 8, 2026 02:22
Add an additional condition to ensure square cropping is only applied when the thumbnail has a 1:1 aspect ratio, preventing incorrect cropping on non-square thumbnails.
…s.kt

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…s.kt

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@rukamori

rukamori commented Feb 8, 2026

Copy link
Copy Markdown
Member

@sang765 thanks for the fixes👌

@rukamori rukamori merged commit 9924b30 into ArchiveTuneApp:dev Feb 8, 2026
1 check passed
@sang765

sang765 commented Feb 8, 2026

Copy link
Copy Markdown
Member Author

@sang765 thanks for the fixes👌

Yay 😉

@sang765 sang765 deleted the fix/square-thumbnail-in-arist branch February 8, 2026 02:12
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