New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spriting replaces transparent color with arbitrary color #831

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 8 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Apr 6, 2015

What steps will reproduce the problem?
1. Enable spriteing
2. Create 2 or more transparent GIFs that use different colors for transparency
3. Look a the resulting sprite

(I'm assuming. I don't have a staging server appropriate for reproducing this 
problem in a clean-room environment.)

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

I expect to see the images appearing as they always do, properly transparent. 
Instead, all but one of the background colors is displayed. I assume that this 
is because only one color can be transparent. Seems to me that mod_pagespeed 
shouldn't combine sprite GIFs that have different background colors.

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

1.7.30.1-3609. (The problem started after updating to this version.)

On what operating system?

CentOS.

Which version of Apache?

2.2.17-1

Which MPM?

Oddly, httpd -l doesn't say.

URL of broken page:

N/A—I've disabled spriteing to solve this problem temporarily.

Original issue reported on code.google.com by wa...@jaquith.org on 13 Nov 2013 at 6:57

Attachments:

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by hui...@google.com on 13 Nov 2013 at 7:11

  • Changed state: Accepted
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Issue 834 has been merged into this issue.

Original comment by hui...@google.com on 20 Nov 2013 at 8:55

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by hui...@google.com on 20 Nov 2013 at 8:56

  • Changed state: Started
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by hui...@google.com on 20 Nov 2013 at 8:57

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Fixed in r3666.

Original comment by hui...@google.com on 9 Dec 2013 at 6:51

  • Changed state: Fixed
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by jefftk@google.com on 2 Jan 2014 at 5:40

  • Added labels: Milestone-v31
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by hui...@google.com on 3 Jan 2014 at 7:48

  • Changed title: Spriting replaces transparent color by arbitrary color
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by hui...@google.com on 6 Jan 2014 at 6:27

  • Changed title: Spriting replaces transparent color with arbitrary color
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment