Skip to content
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

Image transparency replaced with checkerboard on zoom #8745

Open
shidarin opened this issue Mar 18, 2018 · 14 comments
Open

Image transparency replaced with checkerboard on zoom #8745

shidarin opened this issue Mar 18, 2018 · 14 comments

Comments

@shidarin
Copy link

A PNG image with transparency is respected on the preview image within the chat stream:

screen shot 2018-03-18 at 4 02 30 pm

But not in the zoomed view:

screen shot 2018-03-18 at 4 03 40 pm

Where the transparency is replaced with a checkerboard.

Because of the checkerboard background, I figure there's a chance this was already discussed, and is intentional behavior. If so, this issue can be closed.

@shidarin
Copy link
Author

Apparently this is only the case when Pan & Zoom is disabled. Turning Pan & Zoom to enabled causes the transparency to be respected.

@zulipbot
Copy link
Member

Hello @zulip/server-misc members, this issue was labeled with the area: uploads label, so you may want to check it out!

@Anupam-dagar
Copy link
Collaborator

@zulipbot claim

@Anupam-dagar
Copy link
Collaborator

@shidarin what should be the expected behaviour?
is it to show background color instead of checkerboard?

@shidarin
Copy link
Author

That was my expected behavior- display the image over the background color while respecting transparency.

I initially thought that the checkerboard was a concious design decision, but it goes away when you click the pan & zoom button. That makes me think it's a bug.

@Anupam-dagar
Copy link
Collaborator

so what change should be done?

@shidarin
Copy link
Author

I would determine what's causing the checkerboard to appear, then adjust that to be the background color instead.

@ghost
Copy link

ghost commented Mar 22, 2018

Discussed this a bit with @timabbott in #issues:

Some pertinent bits:

Tim Abbott: It seems like it's clearly a bug that they are different between the two views. How we want to resolve it is a bit ambiguous. I think we originally did the checkboard thing so that it's clear if you have a transparent image that it is transparent (otherwise, you might think it has a blue background, which could be confusing).

Tim Abbott: So I suspect we'd be happier to flip it to always due the blue background, but I'd be interested in others' thoughts.

So, anyone have any thoughts on checkboard vs the blue background?

@Anupam-dagar
Copy link
Collaborator

@zulipbot abandon

@aero31aero
Copy link
Member

My concern here would be in cases where a transparent png is made up of mainly blue color. The checkerboard design helps in eliminating most of such cases due to its frequent and consistent changes, so worst case, it is like looking at an image through a net, but the image is still visible.

@timabbott
Copy link
Sponsor Member

I think the current thinking is we will eventually want to make the background controllable by an option.

@Anupam-dagar
Copy link
Collaborator

@zulipbot claim

@zulipbot
Copy link
Member

zulipbot commented Apr 5, 2018

Hello @Anupam-dagar, you have been unassigned from this issue because you have not updated this issue or any referenced pull requests for over 14 days.

You can reclaim this issue or claim any other issue by commenting @zulipbot claim on that issue.

Thanks for your contributions, and hope to see you again soon!

@shidarin
Copy link
Author

shidarin commented May 7, 2018

Posted an update for the wrong issue. Ignore this.

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

No branches or pull requests

6 participants