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 663398 - Native Window Placement improvement: icon overlay
Native Window Placement improvement: icon overlay
Status: RESOLVED WONTFIX
Product: gnome-shell
Classification: Core
Component: extensions
3.2.x
Other Linux
: Normal major
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-11-04 13:44 UTC by Donato
Modified: 2013-12-24 13:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to remove tabs (4.87 KB, patch)
2012-06-19 03:42 UTC, Robert Schroll
none Details | Review
Patch to add icons to overlay (7.24 KB, patch)
2012-06-19 03:45 UTC, Robert Schroll
none Details | Review
Patch to change injection method (5.08 KB, patch)
2012-06-19 03:48 UTC, Robert Schroll
none Details | Review

Description Donato 2011-11-04 13:44:08 UTC
Sometime different windows seems equal at first look (e.g. browser with empty page, terminal, document, etc).
I think that adding over the window the application icon will improve the usability. I've seen this capability in compiz and in kde.

Thanks.

Donato
Comment 1 Robert Schroll 2012-06-19 03:41:14 UTC
I started hacking this a bit to add icons, working off of the Overlay Icons extension (https://extensions.gnome.org/extension/60/overlay-icons/).  The code is rather ugly at the moment.  Notably:
- It will only work when the title is at the top, since I just attached code to the existing framework.
- There are several different injection methods sitting around.
If you're interested in including this in the extension, let me know and I'll start cleaning things up.
Comment 2 Robert Schroll 2012-06-19 03:42:37 UTC
Created attachment 216715 [details] [review]
Patch to remove tabs

This doesn't actually have anything to do with this bug, but my OCD didn't let me start until this was fixed.
Comment 3 Robert Schroll 2012-06-19 03:45:30 UTC
Created attachment 216716 [details] [review]
Patch to add icons to overlay

This patch has most of the changes.  It's rather ugly, in large part because I don't know when we can overwrite the existing functions, and when we have to carefully inject our code.  All the injections are done "by hand", which made sense when I was adding the first function from Overlay Icons.  By the fifth, it was a stupid decision.
Comment 4 Robert Schroll 2012-06-19 03:48:37 UTC
Created attachment 216717 [details] [review]
Patch to change injection method

I realized that, since we were attaching these functions to the relevant signals, we didn't have to inject -- we could just attach a separate function that would call the original.  But for reasons I don't understand, this doesn't work with the show and hide functions.
Comment 5 Baptiste Mille-Mathias 2013-12-24 13:07:07 UTC
Hi there,

I'm going to close this bug as there was a previous bug report about such report with code and it was rejected, and such functionnality goes into an extension.

Regards.