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

Cannot load compressed .ico files #310

Open
James-E-A opened this issue May 23, 2021 · 7 comments
Open

Cannot load compressed .ico files #310

James-E-A opened this issue May 23, 2021 · 7 comments

Comments

@James-E-A
Copy link

James-E-A commented May 23, 2021

Expected behaviour

The image should display.

Actual behaviour

Could not load image …
Compressed icons are not supported.

Steps to reproduce the behaviour

Click this link and open the resulting file in Eye of Mate

MATE general version

1.24.2

Package version

eom-1.24.2-1.fc32

Linux Distribution

Fedora 32

Link to bugreport of your Distribution (requirement)

https://bugzilla.redhat.com/show_bug.cgi?id=1963750

@raveit65
Copy link
Member

I don't see that eom is crashing. This icon is simply not supported.

@James-E-A
Copy link
Author

I did some further testing and it appears the issue is with PNG-compressed .ico files, rather than anything to do with size

Example outputs from FILE(1):

  • MS Windows icon resource - 3 icons, 48x48, 32 bits/pixel, 32x32, 32 bits/pixel - Supported
  • MS Windows icon resource - 1 icon, 16x16 with PNG image data, 16 x 16, 8-bit/color RGBA, non-interlaced, 32 bits/pixel - Not supported

@James-E-A James-E-A changed the title Crashes on .ico images over 64×64px Cannot load compressed .ico files May 24, 2021
@Shoxx98
Copy link

Shoxx98 commented Mar 1, 2024

please implement a way to view compressed icon files

@lukefromdc
Copy link
Member

Note that we are a very small team, and it could take quite a while to get to something like this on our own. I myself have done very little with eom other than testing PR's and bugfixes so will have to leave this for someone on the team that knows what they are doing. As always, if someone outside the team submits a PR of their own for this we will be happy to try it out.

@Shoxx98
Copy link

Shoxx98 commented Mar 6, 2024

please implement a way to view compressed icon files

I am sorry for posting this, I searched for the problem for another program (mentioned that one in the google search), clicked a github link to an already existing issue that sounded exactly like it, and didnt look at the project name again. Don't feel pressured into implementing a feature that no one really asks for

@lukefromdc
Copy link
Member

Not to worry: feature requests give developers (and new developers) a popularity contest for potential new features and can help guide the roadmap for a program. Not all will be used in any program popular enough to attract them, but some will be and widely requested items can tell devs what they may have missed. Example: if a new archive manager written by someone using only Debian could unpack .deb files but not .rpm files that would be an oversight and someone on Fedora would probably ask for .rpm support or even submit a PR for it if they have the skills.

Also note that a feature request from one person may attract a PR submitted by another with similar needs.

In this case, lots of people working on linux/unix systems will never encounter microsoft compressed icons, but that could change.

Suppose Microsoft decided to lock all versions of MS windows to the Microsoft Store and disable all installation of software from outside it with no unpaid option to switch away from that. This would effectively ban Steam, which is exactly why discussion of such things a decade ago forced Steam to experiment with Linux support as both an insurance policy and a shot across Microsoft's bow. Had that run to completion, lots of devs from the Windows world might have shown up in the Linux world, making a feature request like this one important and probably bringing a PR from one of them.

@cwendling
Copy link
Member

This should be implemented in GdkPixbuf I believe, and there seems to be some unfinished work for it. I guess if anybody want to take that one it could be merged there :)

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

5 participants