After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783975 - Top panel: Transparency doesn't update on "Hide all normal windows" action.
Top panel: Transparency doesn't update on "Hide all normal windows" action.
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
3.25.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2017-06-19 23:06 UTC by franglais125
Modified: 2017-07-05 14:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
panel: Consider visibility changes for solid style (1.76 KB, patch)
2017-06-20 08:27 UTC, Florian Müllner
committed Details | Review

Description franglais125 2017-06-19 23:06:17 UTC
This is regarding the new top panel transparency when 'free floating'.

The panel's opacity is not properly updated when windows are hidden/shown with a shortcut.

Steps to reproduce:
1) Enable the "Hide all normal windows" shortcut (e.g. Super+d)
2) Maximize a window, so that the top panel becomes opaque.
3) Hit "Super+d", the desktop should be showing now, but the top panel remains opaque.

More symptoms:
4) Use Ctrl+Alt+down to move to the next workspace. Now the transparency is updated.
5) Move back to the previous workspace.
6) Hit "Super+d" again to restore the window. The panel remains transparent, while it should be opaque.

Solution:
Probably adding a new signal that updates on windows restacked/focus or similar might help with this.
Comment 1 Florian Müllner 2017-06-20 08:27:48 UTC
Created attachment 354088 [details] [review]
panel: Consider visibility changes for solid style

We currently assume that window state changes are accompanied by an
allocation change (triggered for example by the minimize animation).
However this misses the case where a window actor is simply hidden
without any transition, as is the case with the 'show-desktop' action
for instance, so start tracking plain visibility changes as well.
Comment 2 franglais125 2017-06-20 17:16:02 UTC
Thanks for the patch. It's working fine here.

For anyone interested, I 'backported' this as an extension.

https://github.com/franglais125/panel-transparency
Comment 3 franglais125 2017-06-30 21:29:22 UTC
What is the the way forward for this patch? Will it be included eventually in master?

It would be a shame for the patch to simply rot here :)
Comment 4 Florian Müllner 2017-06-30 22:50:04 UTC
(In reply to franglais125 from comment #3)
> What is the the way forward for this patch? Will it be included eventually
> in master?

Yes, but it should get reviewed first - the person most likely to do that is currently on vacation :-)
Comment 5 Rui Matos 2017-07-05 12:47:51 UTC
Review of attachment 354088 [details] [review]:

looks fine
Comment 6 Florian Müllner 2017-07-05 14:50:34 UTC
Attachment 354088 [details] pushed as 292ac41 - panel: Consider visibility changes for solid style