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

Is this the best we can do with the 16x16 icon? #7725

Closed
SpiralGalaxy opened this issue Nov 20, 2017 · 10 comments
Closed

Is this the best we can do with the 16x16 icon? #7725

SpiralGalaxy opened this issue Nov 20, 2017 · 10 comments

Comments

@SpiralGalaxy
Copy link

Only 1 icon is not like the others...

windowclipping 18

@junpet
Copy link

junpet commented Nov 20, 2017

Win7x64 1920x1080
qBittorrent v4.0.0
image

@SpiralGalaxy
Copy link
Author

Hell, that's even worse with the inconsistent white border.

@ghost
Copy link

ghost commented Nov 20, 2017

I noticed this, too. I can't believe they shipped icons that look that aliased.

@mzso
Copy link

mzso commented Nov 20, 2017

It looks pretty horrible here too, on win 8.1: QBT notification area icon

@mzso
Copy link

mzso commented Nov 20, 2017

By the way Can I donwload beta 2 somewhere? I wanted to make a comparison, but I can't see it anywhere anymore...
I finally managed it (old vs new): old-vs-new-qbt_icon

@mzso
Copy link

mzso commented Nov 20, 2017

@SpiralGalaxy commented on 2017. nov. 20. 05:51 CET:

Only 1 icon is not like the others...

windowclipping 18

I advise yout to rename the issue to "The QBT notification area icon with 4.0.0 is very aliased" or such, because the current one doesn't convey any information...

@sledgehammer999
Copy link
Member

sledgehammer999 commented Nov 20, 2017

This is issue was present with the older icon too. It is a Qt bug. I have already reported it there and they have a fix for Qt 5.9.4. (qt bug: https://bugreports.qt.io/browse/QTBUG-63447)

The bug is that Qt decides to use a 32x32 icon and Windows downscales it badly to 16x16. (And yes we include a 16x16 icon too).

This wasn't so obvious with the previous icon because of the way the icon looks already.
Older version of qBittorrent based on Qt 5.6 don't show the problem.

In any case, this will be fixed once Qt makes a new release available.

@Chocobo1
Copy link
Member

In any case, this will be fixed once Qt makes a new release available.

@sledgehammer999
It seems the just released Qt 5.9.3 fixed it already.

@sledgehammer999
Copy link
Member

It seems the just released Qt 5.9.3 fixed it already.

Yes. I confirm that it seems to work.
I have rebuild my toolchain for 64 bit based on 5.9.3, so our next release will use it.

@ghost ghost mentioned this issue May 5, 2018
@expiorer
Copy link

expiorer commented Jun 4, 2018

The icon data looks ok.
But the icon in tray sucks.
image

@qbittorrent qbittorrent locked and limited conversation to collaborators Feb 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants