Skip to content

Video thumbnail#166

Open
Quffik wants to merge 2 commits into
ExteraApp:mainfrom
Quffik:Video-thumbnail
Open

Video thumbnail#166
Quffik wants to merge 2 commits into
ExteraApp:mainfrom
Quffik:Video-thumbnail

Conversation

@Quffik

@Quffik Quffik commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

What: Replaced the nested if-else widget structure with a clean ternary conditional expression to render MxcImage correctly, fixed the fallback BlurHash aspect ratio rendering formula, grouped the duration and file size inside a single styled corner badge, and updated the action button to a custom translucent IconButton.filledTonal.

Why: The fallback blurred layout suffered from a broken height computation, causing layout distortion. Additionally, combining metadata overlay information saves vertical bubble space and provides a polished design matching modern chat interfaces.

How:

  • Fixed the fallback BlurHash height definition from bubbleWidth * aspectRatio to bubbleWidth / aspectRatio.
  • Grouped the duration text and the conditional sizeInt?.sizeString into a unified string separated by a bullet (•), placed inside a single Positioned container with custom padding and background transparency.
  • Replaced the text-based button with a centered, translucent IconButton.filledTonal using a hardcoded Icons.play_arrow_outlined icon
изображение

Pull Request has been tested on:

  • Android
  • iOS
  • Browser (Chromium based)
  • Browser (Firefox based)
  • Browser (WebKit based)
  • Desktop Linux
  • Desktop Windows
  • Desktop macOS

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