On Fri, 2005-09-30 at 11:27 +0200, Christian Neumair wrote:
> While we treat the margins of the icon container in canvas units, we set
> them in pixel units. Use a big panel and you'll note that it overlaps
> the icons on the top, as reported by bug 149847 [1].
>
> Proposed patch attached.
Thats not right. Some places treat the margins as canvas units already,
for instance:
/* Lay out icons a line at a time. */
canvas_width = (GTK_WIDGET (container)->allocation.width
- container->details->left_margin
- container->details->right_margin)
/ EEL_CANVAS (container)->pixels_per_unit;
canvas_height = (GTK_WIDGET (container)->allocation.height
- container->details->top_margin
- container->details->bottom_margin)
/ EEL_CANVAS (container)->pixels_per_unit;
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.
A better fix would be to always correctly treat ->*_margin as in pixel
coordinates.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Alexander Larsson Red Hat, Inc
[EMAIL PROTECTED] [EMAIL PROTECTED]
He's a suave devious waffle chef on the hunt for the last specimen of a great
and near-mythical creature. She's an elegant gold-digging college professor
descended from a line of powerful witches. They fight crime!
--
nautilus-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/nautilus-list
