Insensitive (disabled) toolbar icons in GTK3 are not dimmed

I have bisected the GTK code to find what commit introduced the change in behavior:

commit 99c4f2dd3947c290e3ea19153202be3e43b22add
Date: Wed Mar 18 15:22:09 2015 +0100

    render: Make image effect not depend on state

    Instead rely on -gtk-image-effect only. Adwaita should already work this
    way.

    Relying on state was a leftover feature from the GTK 2 days.

Since this change, it is required to use "-gtk-image-effect: dim;" for insensitive elements. Also, in GTK 3.20+, the attribute was renamed to "-gtk-icon-effect", see https://mail.gnome.org/archives/commits-list/2015-December/msg00151.html

So I suggest to add both gtk-image-effect and gtk-icon-effect to fix the issue in 16.04 (GTK 3.18) and for 16.10 (GTK 3.20?). I will post a patch to achieve that.

Eclipse bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=492354

Actually, gtk-icon-effect shouldn't be used until GTK 3.20 otherwise it spews a warning.

I pushed a branch with the required theme changes but I'm not sure I made all the necessary debian package changes (changelog, etc?). Please let me know if I can do anything else to help.

Can someone help getting this fix in? This is a big usability issue for a lot of Eclipse users because it looks like all the buttons can be pressed at all time.

Hi Marc-Andre, I was running into the same problem on Ubuntu 16.04.1 with Eclipse Neon (4.6.0).
Your fix works fine for me, thank you.

Thank you for the patch! Please Canonical integrate it ASAP :-(

The problem can also be seen in Remmina remote desktop and this patch fixes the problem there too.

In the screen shot I just attached, the Copy, Edit and Delete buttons should be dimmed because nothing is selected but instead they always look "enabled" and bright.

Same bug in Wireshark-gtk. The toolbar should have the "Stop live session" button dimmed but it looks enabled. This patch also fixes this problem.

Thank you for your bug report. What GTK version do you use on xenial? I can't confirm the issue here on an updatode xenial installation... (the patch still should be reviewed and probably makes sense, just trying to understand why the issue is not consistent, do you use a ppa with a newer gtk or something?)

I can reproduce the issue with an updated xenial (save button not disabled when running Eclipse). Here are my gtk versions:

libgtk2.0-0:
  Installé : 2.24.30-1ubuntu1
  Candidat : 2.24.30-1ubuntu1
 Table de version :
 *** 2.24.30-1ubuntu1 500
        500 http://fr.archive.ubuntu.com/ubuntu xenial/main amd64 Packages
        100 /var/lib/dpkg/status
libgtk-3-0:
  Installé : 3.18.9-1ubuntu3.1
  Candidat : 3.18.9-1ubuntu3.1
 Table de version :
 *** 3.18.9-1ubuntu3.1 500
        500 http://fr.archive.ubuntu.com/ubuntu xenial-updates/main amd64 Packages
        100 /var/lib/dpkg/status
     3.18.9-1ubuntu3 500
        500 http://fr.archive.ubuntu.com/ubuntu xenial/main amd64 Packages

This issue can be observed in Xenial and Yakkety.

Install `gtk-3-examples` and run `gtk3-widget-factory`, look at the toolbar on Page 2, the last icon (tooltip 'Insert something') is disabled and should be dimmed, but is not.

I have also reproduced this issue on Xenial and Yakkety using:

  * Remmina (from the archive)
  * Wireshark (from the archive)
  * Eclipse (downloaded)

To reproduce with eclipse it has to be downloaded since the version in the archive uses GTK2+ for styling, and dims insensitive icons correctly.

Here is a merge proposal for Yakkety:

  * https://code.launchpad.net/~flexiondotorg/ubuntu-themes/yakkety-icon-dim-lp1578810/+merge/308480

Here is a merge proposal for Xenial:

  * https://code.launchpad.net/~flexiondotorg/ubuntu-themes/xenial-icon-dim-lp1578810/+merge/308481

I tested the proposed fix for Yakkety and it also works well, thanks!

Ping? :)

No need to ping - I was just away last week. The packages are building now and I will upload when that is done.

This bug was fixed in the package ubuntu-themes - 16.10+17.04.20161024-0ubuntu1

---------------
ubuntu-themes (16.10+17.04.20161024-0ubuntu1) zesty; urgency=medium

  [ Martin Wimpress ]
  * Dim images of insensitive widgets. Thanks to Marc-Andre Laperle.
    (LP: #1578810)

 -- <email address hidden> (<email address hidden>) Mon, 24 Oct 2016 13:03:20 +0000

Hello Marc-Andre, or anyone else affected,

Accepted ubuntu-themes into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-themes/16.10+16.10.20161024-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Hello Marc-Andre, or anyone else affected,

Accepted ubuntu-themes into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-themes/14.04+16.04.20161024-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

I have verified both 16.10+16.10.20161024-0ubuntu1 on yakkety and 14.04+16.04.20161024-0ubuntu1 on xenial. It worked properly.

This bug was fixed in the package ubuntu-themes - 14.04+16.04.20161024-0ubuntu1

---------------
ubuntu-themes (14.04+16.04.20161024-0ubuntu1) xenial; urgency=medium

  [ Iain Lane ]
  * Dim images of insensitive widgets. Thanks Marc-Andre Laperle. (LP:
    #1578810)

 -- <email address hidden> (<email address hidden>) Mon, 24 Oct 2016 16:25:22 +0000

The verification of the Stable Release Update for ubuntu-themes has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package ubuntu-themes - 16.10+16.10.20161024-0ubuntu1

---------------
ubuntu-themes (16.10+16.10.20161024-0ubuntu1) yakkety; urgency=medium

  [ Martin Wimpress ]
  * Dim images of insensitive widgets. Thanks to Marc-Andre Laperle.
    (LP: #1578810)

 -- <email address hidden> (<email address hidden>) Mon, 24 Oct 2016 13:03:52 +0000

Thanks a lot everyone for making this fix happen! :)