We have moved to Github. Please open tickets there.

Opened 11 months ago

Closed 11 months ago

Last modified 11 months ago

#4577 closed Defect (fixed)

Transparencies in PNG uploads are turned black

Reported by: mrclay Owned by: mrclay
Priority: normal Milestone: Elgg 1.8.6
Component: File Version: 1.6
Severity: minor Keywords:
Cc: brett@… Difficulty:

Description

I patched this in our 1.6 install, will try to submit for 1.8 ASAP

Change History (5)

comment:1 Changed 11 months ago by mrclay

  • Owner set to mrclay
  • Status changed from new to assigned

Since get_resized_image_from_existing_file() must return a JPEG, the transparency has to filled with some color, and by default it's black. This PR changes it to white.
https://github.com/Elgg/Elgg/pull/270

comment:2 Changed 11 months ago by cash

  • Milestone changed from Needs Review to Elgg 1.8.6

comment:3 Changed 11 months ago by Steve Clay

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixes #4577: Transparency converted to white instead of black

Changeset: 9607776966d5c0f8a132dd4275841298619907ea

comment:4 Changed 11 months ago by Steve Clay

Fixes #4577: Transparency converted to white instead of black

Changeset: 9607776966d5c0f8a132dd4275841298619907ea

comment:5 Changed 11 months ago by Cash Costello

Merge pull request #270 from mrclay/4577-png-transparency

Fixes #4577: Transparency converted to white instead of black

Changeset: 15c0a0106be33711fe6b11e2c6acd2812d6e9bbe

Note: See TracTickets for help on using tickets.