Icons blurry on HiDPI

[peteruithoven]

I noticed that the icons's are a bit blurry. Could be caused by my HiDPI screen?
For comparison screenshot with icon on snapcraft site on the left:

List:

Tested with latest master, 1.0.0 tagged version.

[peteruithoven]

It might not be Snaptastics fault though, the Snap icon is also blurry in the Applications menu:

[unbelievableflavour]

Ik could maybe at even higher pixeled images if found... Would that help? Because the size of the icon is actually the same now as the size of the AsyncImage.

[peteruithoven]

as the size of the AsyncImage

Which one?

Application menu doesn't seem to include any icon scale factor correction logic::
https://github.com/elementary/applications-menu/blob/master/src/Widgets/AppEntry.vala#L70
https://github.com/elementary/applications-menu/blob/master/src/Widgets/AppEntry.vala#L105

AsyncImage automatically seems to correct for the scale factor:
https://github.com/elementary/granite/blob/master/lib/Widgets/AsyncImage.vala

[unbelievableflavour]

In this function https://github.com/bartzaalberg/snaptastic/blob/master/src/IconHandler.vala#L47 I get the Icon from file. Most of the time 64x64 is the result there. If not it will fallback to 48x48 and so on. In the same file you have the var 'icon_size' which is 32. This variable is used in the AsyncImage for the lists. In https://github.com/bartzaalberg/snaptastic/blob/master/src/Components/DetailViewBanner.vala you find the icon for the detailview. In the reload_view function in that file it is set to icon_size 64x64 for the asyncImage.