Skip to content

Conversation

@aonnikov
Copy link
Member

@aonnikov aonnikov commented Sep 5, 2025

Fixes unsupported format issues for bmp and heic images

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
@aonnikov aonnikov requested review from Copilot and haiodo September 5, 2025 11:03
@huly-github-staging
Copy link

Connected to Huly®: UBERF-13512

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for previewing HEIC and BMP image formats by converting them to PNG. The implementation introduces conversion utilities for both formats and integrates them into the existing image preview provider.

  • Adds HEIC to PNG conversion using the heic-decode library
  • Adds BMP to PNG conversion using the bmp-ts library with manual RGBA channel reordering
  • Updates the ImageProvider to handle HEIC/HEIF and BMP content types

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

File Description
pods/preview/src/utils/heic.ts New utility for converting HEIC files to PNG format
pods/preview/src/utils/bmp.ts New utility for converting BMP files to PNG with RGBA channel correction
pods/preview/src/providers/image.ts Updated image provider to handle HEIC/HEIF and BMP conversions
pods/preview/package.json Added dependencies for heic-decode and bmp-ts libraries
Files not reviewed (1)
  • common/config/rush/pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
@haiodo haiodo merged commit 70805d2 into develop Sep 5, 2025
15 checks passed
@haiodo haiodo deleted the uberf-10382-heic-bmp branch September 5, 2025 16:34
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.

3 participants