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 149847 - Desktop icons not moving correctly when top dock resizes
Desktop icons not moving correctly when top dock resizes
Status: RESOLVED OBSOLETE
Product: nautilus
Classification: Core
Component: Desktop
2.13.x
Other All
: High normal
: 2.14.x
Assigned To: Nautilus Maintainers
Nautilus Maintainers
AP3
Depends on:
Blocks:
 
 
Reported: 2004-08-10 21:47 UTC by korn
Modified: 2010-07-10 12:41 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
Proposed patch (1.20 KB, patch)
2005-09-30 09:29 UTC, Christian Neumair
needs-work Details | Review

Description korn 2004-08-10 21:47:29 UTC
1. Launch GOK
2. Configure the GOK window to be a top dock
3. Note that the desktop icons have moved down some to get out of the way of the
GOK window, but not enough out of the way (the top of the "This Computer" icon
is cut off)
4. Select the Compose keyboard in GOK
5. Note that the desktop icons have moved further down to get out of the way of
the expanded GOK window, but now both "This Computer" and also "Documents" are
complely obscured.
6. Select the Launch keyboard in GOK (only two rows by default)
7. Note that the desktop icons have now moved up a bit, and only the very top of
the "This Computer" icon is obscured.
8. Bring up an application (e.g. gedit) and place its window just underneath the
GOK window.  Go through a bunch of GOK keyboards (e.g. "Activate", "Launch"),
and note that this applications' window accurately tracks the resizing.  This
suggest the bug is (mostly) in Nautilus and not GOK.  Turns out it doesn't track
correctly with "Compose".  Will file a GOK but for that...
Comment 1 Calum Benson 2004-10-21 16:39:52 UTC
Apologies for spam-- ensuring Sun a11y team are cc'ed on all current a11y bugs.
 Filter on "SUN A11Y SPAM" to ignore.
Comment 2 Christian Neumair 2005-08-31 09:55:05 UTC
Is this still an issue for recent Nautilus/GOK combinations?

If so, I suppose that there are bugs wrt GOK/"_NET_WORKAREA". Nautiluses uses
the workarea to determine its margins. If you have some time to debug Nautilus,
you may dive into src/file-manager/fm-desktop-icon-view.c and check whether
con_container_set_workarea behaves as expected.
Comment 3 Christian Neumair 2005-09-30 09:28:00 UTC
Quiet a dumb issue. Confirming.
Comment 4 Christian Neumair 2005-09-30 09:29:09 UTC
Created attachment 52848 [details] [review]
Proposed patch

I've also submitted this patch to nautilus-list [1] for review.

[1] http://mail.gnome.org/archives/nautilus-list/2005-September/msg00211.html
Comment 5 Martin Wehner 2005-12-18 01:43:53 UTC
Comment on attachment 52848 [details] [review]
Proposed patch

Alex review and Mannys response from the list:

> Thats not right. Some places treat the margins as canvas units already

I wasn't aware of these horrible inconsistencies, sorry.

> Furthermore, the margins should be in pixels, since they are in pixels.
> When the zoom changes the margin in pixels doesn't change, but the
> margin in canvas units does.

Oh, that's what these obscure canvas units are all about :D. I didn't
closely investigate it and thought this would just be some obscure
factor like the PANGO 1024 #define.

> A better fix would be to always correctly treat ->*_margin as in pixel
> coordinates.

Surprisingly, that doesn't fix it. The icons are now layed out extremely
tight to the canvas top-left edge.
The whole icon container code is quiet confusing, and a mixture of
#defines and externally set margins. The layout code for instance does
not do x+=left_margin and y+=top_margin before laying out, resulting in
a top-left layout with double right/bottom margins. Also,
lay_down_icons_tblr is extremely tied to the desktop, which is reflected
by DESKTOP_PAD_HORIZONTAL and DESKTOP_PAD_VERTICAL defines it contains.
The bitrotten icon container really deserves an in-depth investigation.
Comment 6 Christian Neumair 2006-02-25 21:53:42 UTC
Updating version to 2.13. Chances are quite good that we get a patch similar to [1] into 2.14. Milestoning to 2.14.

[1] http://mail.gnome.org/archives/nautilus-list/2006-February/msg00098.html
Comment 7 Calum Benson 2006-04-26 16:41:00 UTC
Apologies for spam-- marking AP3 to reflect accessibility impact.
Comment 8 Calum Benson 2006-04-26 17:14:38 UTC
Apologies for spam... ensuring Sun a11y folks are cc'ed on all current accessibility bugs.
Comment 9 Marcus Carlson 2010-07-09 23:50:22 UTC
I may be at deep water here but resizing the gnome-panel moves the icons on the desktop correctly (in nautilus 2.30). Is that what this bug is about or have I missed something?
Comment 10 Cosimo Cecchi 2010-07-10 12:41:28 UTC
It seems Alexander committed a fix for this:

http://mail.gnome.org/archives/nautilus-list/2006-February/msg00130.html

"I commited the attached patch. I verified that it at lest fixed the top
icon moving wrongly when using != 100% default zoom and changing the
size of the top panel."

So, let's close this as OBSOLETE. Feel free to reopen the bug if it's still an issue.