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 141900 - Activate Show Desktop Button on DND hover
Activate Show Desktop Button on DND hover
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: show-desktop-button
git master
Other All
: Normal enhancement
: ---
Assigned To: Panel Maintainers
Panel Maintainers
: 164407 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-05-05 12:06 UTC by Maciej Katafiasz
Modified: 2005-01-18 06:40 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Proposed patch (2.01 KB, patch)
2004-05-16 20:14 UTC, Maciej Katafiasz
none Details | Review
New patch without compilation warnings (2.45 KB, patch)
2004-11-28 20:09 UTC, Maciej Katafiasz
committed Details | Review

Description Maciej Katafiasz 2004-05-05 12:06:40 UTC
Hovering pointer over Show Desktop Button during DND should activate it,
similarly to the way it does with task list now. This idea could be probably
extended on other UI elements, like Metacity "minimize" button, or maybe desktop
switcher, dunno if it's sensible though.
Comment 1 Mark McLoughlin 2004-05-13 12:32:25 UTC
I doubt its a good idea for random sites (which aren't even drop sites) to do
random things as you drag something around ...

Usability people: ?
Comment 2 Bryan W Clark 2004-05-13 15:27:13 UTC
Interesting idea, however I think Mark's remarks bring up a good point.  This
behaviour could get out of control when extended to lots of other UI elements. 

On just the 'Show Desktop' button I don't think this would get out of hand, the
behaviour wouldn't complicate the button much and would provide users with a new
way of getting to the desktop.  Having this as the single point of action for
this could be a solution to the 'I have lots of windows open and want drop this
item in a folder off the Desktop (or the Desktop itself)' problem.  I think we'd
really have to test it out to know.

Maciej, I'd love to work with you on a patch to the 'Show Desktop' button; I
don't think it would be a difficult patch for you to write, maybe Mark could
help you out with any snags you run into in the gnome-panel.
Comment 3 Maciej Katafiasz 2004-05-13 18:18:09 UTC
OK, I may try to do this. It'd be in wnck-applet, right? I agree that it may get
out of hand, OTOH some judiciously chosen points could speed operations a lot.
Particularly desktop switcher, I've seen many times new people assume that it's
possible to use it for DnD (after they learned about switcher concept) and be
suprised it's not actually the case. I'd also like it to be DnD sensitive, ATM
it requires moving windows between desktops prior to dragging which is cumbersome.
Comment 4 Calum Benson 2004-05-14 15:43:21 UTC
I suppose this is harmless enough, but I'd be surprised if anyone ever discovers
it exists (and if they do, it might give them unrealistic expectations about
where else they can drag things, and then they'll be frustrated when they can't).

IMHO it would be more useful (and discoverable) if a 'minimize all' happened
while I was hovering over an empty area on the desktop-- but I guess that's
basically just considering the desktop to be a spring loaded folder, so we can't
do that :)
Comment 5 Maciej Katafiasz 2004-05-16 20:14:15 UTC
Created attachment 27761 [details] [review]
Proposed patch
Comment 6 Maciej Katafiasz 2004-05-17 08:49:44 UTC
Please see also http://bugzilla.gnome.org/show_bug.cgi?id=142649
Comment 7 Maciej Katafiasz 2004-07-16 01:45:39 UTC
What about this patch? Is it good enough to be accepted? Adding PATCH keyword,
hopefully it will attract someone to review it :)
Comment 8 Josh Lee 2004-09-09 01:54:12 UTC
Microsoft Windows activates 'show desktop' if you drag something to an empty
spot on the taskbar and hold it there. This is a pretty useful behavior and it
probably won't be activated accidentally. I say include it.
Comment 9 Christian Neumair 2004-10-21 13:58:48 UTC
Maciej: The patch looks clean code-wise. The rest is up to the maintainers.
Thanks for your efforts!
Comment 10 Vincent Untz 2004-10-27 20:27:27 UTC
Maciej: does it work? I tried it, but it did nothing.
Comment 11 Maciej Katafiasz 2004-11-28 19:43:25 UTC
(Sorry for the delay, but for long time my build was broken, and I had some
issues with getting it back to work)

Yes, applies cleanly and works fine for me, although I noticed small warning
during compile, do you compile with -Wall -Werror? I'm actually quite surprised
it's not enabled by default with jhbuild. I'll set it on and take care of
warnings, cleaned up patch should follow shortly
Comment 12 Maciej Katafiasz 2004-11-28 20:09:51 UTC
Created attachment 34249 [details] [review]
New patch without compilation warnings

It's the same as previous one, with additional #include only to fix compiler
warning
Comment 13 Vincent Untz 2005-01-10 18:47:34 UTC
Comment on attachment 34249 [details] [review]
New patch without compilation warnings

I committed a slightly modified version.
Comment 14 Vincent Untz 2005-01-10 18:48:02 UTC
Thanks
Comment 15 Vincent Untz 2005-01-18 06:40:59 UTC
*** Bug 164407 has been marked as a duplicate of this bug. ***