Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2310 closed bug (fixed)

Qgis icon in dock on OSX is low resolution

Reported by: jctull Owned by: nobody
Priority: minor: annoyance Milestone: Version 1.4.0
Component: GUI Version: Trunk
Keywords: icon dock application bundle OSX Cc:
Must Fix for Release: No Platform: OS X
Platform Version: Awaiting user input: no

Description

For some reason, the application icon displayed in the OS X dock and application switcher (command-tab) is very low resolution. The finder icon is of proper resolution. It appears that the qgis.icns file does not have a higer-resolution icon image.

Change History (5)

comment:1 by kyngchaos, 14 years ago

Bizarre... I think the icon is OK, if you watch it closely it starts out full resolution in the Dock, then switches to a lores icon. Somehow the icon is getting switched out during startup. I haven't paid close attention to recent builds to notice when this changed, but it's OK in the 1.3 release.

Which OSX version for you? I on Snow Leopard.

Cmake or Xcode build? (Xcode here)

I'll try a Leopard build to see if the OSX version affects it.

Which Qt version? (4.5.3 release here)

comment:2 by kyngchaos, 14 years ago

Happens on Leopard also.

Another odd thing - when quitting, the Dock icon briefly switches back to full res before disappearing.

comment:3 by jctull, 14 years ago

It happens for both xcode and cmake builds. It appears to have cropped up in the last 2-3 weeks. I am also using qt-4.5.3, but another user on irc with (I believe) qt-4.6.0 said it is happening for him as well.

comment:4 by kyngchaos, 14 years ago

Resolution: fixed
Status: newclosed

Could this be it: r12424 -- Macs should also be using icon from "resource", ie the icns file. Though on a closer look it seems to just be reordering when the setWindowIcon() happens (which would happen on OSX in either case).

... yep, revert that change and the icon stays full res.

So, r12583 should fix this (Mac more like Windows in this case ;). Apparently the setWindowIcon() call pre-r12424 didn't really happen on OSX, even though it was not conditionalized out for OSX.

comment:5 by jctull, 14 years ago

Yes, that fixes it for the cmake build also. Good work.

Note: See TracTickets for help on using tickets.