App indicator icon size is too small

[PJSingh5000]

The Ubuntu appindicators extension scales the indicator icon to a smaller size than it should be. The icon should match the sizes of the other icons in the top-bar.

I tested this with package hplip-gui (version 3.17.7+repack0-3) and the HP icon is much smaller than it should be.

This specific icon is /usr/share/hplip/data/images/32x32/hp_logo.png.

I've tried replacing hp_logo.png with a 32x32 sized icon, a 22x22 sized icon, and a 600x600 sized icon by replacing. In all cases, the icon is always scaled to a specific size that is smaller and out of scale when compared to all of the other indicator icons (such as the Power, Network, and Sound indicator icons). Note that there is NO empty border in the image file that would cause the circle logo to appear smaller when resized.

I ~think~ the available space for the icon is obtained from Panel.PANEL_ICON_SIZE in indicatorStatusIcon.js. However, I suspect that Panel.PANEL_ICON_SIZE returns an incorrect value or needs to be adjusted in some way?

Description: Ubuntu Artful Aardvark (development branch)
Release: 17.10

gnome-shell-extension-appindicator:
  Installed: 17.10.1
  Candidate: 17.10.1
  Version table:
 *** 17.10.1 500
        500 http://us.archive.ubuntu.com/ubuntu artful/main amd64 Packages
        500 http://us.archive.ubuntu.com/ubuntu artful/main i386 Packages
        100 /var/lib/dpkg/status

ProblemType: Bug
DistroRelease: Ubuntu 17.10
Package: gnome-shell-extension-appindicator 17.10.1
ProcVersionSignature: Ubuntu 4.13.0-16.19-generic 4.13.4
Uname: Linux 4.13.0-16-generic x86_64
ApportVersion: 2.20.7-0ubuntu3
Architecture: amd64
CurrentDesktop: GNOME
Date: Sun Oct 15 21:08:24 2017
Dependencies:

InstallationDate: Installed on 2017-10-14 (1 days ago)
InstallationMedia: Ubuntu 17.10 "Artful Aardvark" - Beta amd64 (20171012)
PackageArchitecture: all
ProcEnviron:
 TERM=xterm-256color
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: gnome-shell-extension-appindicator
UpgradeStatus: No upgrade log present (probably fresh install)

[Andrea Azzarone (azzar1)]

Seems like gnome-shell is taking the icons-size value from css. We need to do the same in gnome-shell-extension-appindicator.

[Ragnar Dunbar (zaroff)]

This also affects the Dropbox indicator-icon, which additionally features no left or right-click menu. The behaviour is not consistent with other indicator-icons - the transmission status-icon, if enabled, appears correctly and features also a menu.

[Bart Willemsen (b-willemsen8)]

Also witnessing this with the JetBrains Toolbox application. The icon is super small, but seems to function correctly otherwise.

[Irakliy Kukava (irakliy01)]

appindicator also scales the telegram icon. The size is very small to see the number of unread messages

[Eugene Romanenko (eros2)]

Any workaround for this?
Icons for telegram, brightness-indicator, syspeek and classicmenu-indicator terrible small!
Transmission icon is ok.

[Norbert (nrbrtx)]

For me the best workaround is to switch to MATE DE.

[Carlos Pita (carlosjosepita)]

See my report https://bugs.launchpad.net/ubuntu/+source/gnome-shell-extension-appindicator/+bug/1832793

Indeed PANEL_ICON_SIZE is the wrong way to compute the icon size.

[Launchpad Janitor (janitor)]

This bug was fixed in the package gnome-shell-extension-appindicator - 33-1

---------------
gnome-shell-extension-appindicator (33-1) experimental; urgency=medium

  * Team upload.

  [ Marco Trevisan (Treviño) ]
  * New upstream version:
    - Completely refactored the icons handling, to use the same look and
      behavior of the default gnome-shell indicators and properly handle
      HiDPI cases.
    - appIndicator: Always use Gio.Icon as model and St.Icon ui (LP: #1723827,
      LP: #1817073, LP: #1832793, LP: #1825623)
    - appIndicator: Don't try to reload multiple times a pending icon
    - appIndicator: Lookup theme icons using scale factor
    - testTool: Add test to show a changing label
    - indicatorStatusIcon: Use Clutter actor alignment on 3.35+ versions
      (LP: #1866042)
    - appIndicator: Add a cancellable to stop the proxy on destruction
    - Util: Cancel previous properties refresh requests on new one
    - util: Don't emit signal changed if a property contents is equal
    - Util: add message structured logging as well
    - Util: Cleanup syntax of refreshPropertyOnProxy
    - appIndicator: Use simpler implementation for argbToRgba
    - appIndicator: Use warning level for not found icons

  [ Fini Jastrow ]
  * appIndicator: Fix runtime error when icon can not be loaded

 -- Marco Trevisan (Treviño) <email address hidden> Thu, 12 Mar 2020 22:36:05 +0100