Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

transparency of inlined PNG image #21

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 3 comments
Closed

transparency of inlined PNG image #21

GoogleCodeExporter opened this issue Apr 6, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. install mod_pagespeed
2. enable with CoreFilters et collapse_whitespace,elide_attributes
3. HTML page with little transparent PNG (900B ~)

What is the expected output? What do you see instead?

transparency of PNG image is broken, transparency is black.

What version of the product are you using (please check X-Mod-Pagespeed
header)?

X-Mod-Pagespeed:0.9.0.0-128


On what operating system?
Linux Debian 5.0.6

Which version of Apache?
2.2.9-10+lenny8

Which MPM?
apache2-mpm-worker

Please provide any additional information below, especially a URL or an
HTML file that exhibits the problem.

original: http://web.directnom.com/
with pagespeed (source copy): http://web.directnom.com/PAGESPEED/index.html

so it's now disabled.

Original issue reported on code.google.com by gwenael....@gmail.com on 5 Nov 2010 at 11:00

@GoogleCodeExporter
Copy link
Author

Thanks for the great example pages!

Original comment by sligocki@google.com on 5 Nov 2010 at 4:05

  • Added labels: Priority-High
  • Removed labels: Priority-Medium

@GoogleCodeExporter
Copy link
Author

Thanks indeed.  I was able to easily reproduce the misbehavior locally.  A fix 
may be tricky, as I suspect it points to shortcomings in our various 
image-handling libraries.  I'll see if we can at least bail out if we can't 
handle transparency gracefully.

Original comment by jmaes...@google.com on 5 Nov 2010 at 8:08

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

This is fixed in svn.  OpenCV was stripping the alpha channel; for the moment 
we simply avoid shrinking the image.  Please try out the svn (trunk checkout) 
if you can.

Original comment by jmaes...@google.com on 10 Nov 2010 at 6:46

  • Changed state: Fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant