LinusU / LinusU/decode-ico

Icons with 24bit images larger than 128x128 are incorrectly being detected as 32bit.

Open
#7 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
19
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Attached is an icon with 4-bit, 8-bit, and 24-bit entries. There are no 32-bit images yet the 24-bit `1024x1024`, `512x512`, and `256x256` images are being detected as 32-bit.
[c14a2a57ead18f3532a5a8949382c536.zip](https://github.com/LinusU/decode-ico/files/7114095/c14a2a57ead18f3532a5a8949382c536.zip)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the report with the attached c14a2a57ead18f3532a5a8949382c536.zip icon and trace how decode-ico determines bit depth for its 1024x1024, 512x512, and 256x256 entries. Done means those 24-bit images are detected as 24-bit while the existing 4-bit and 8-bit entries remain correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.