Re: [PATCH] add graph window



On Fr, 2005-11-18 at 00:51 -0500, Jody Goldberg wrote:

+                (w->zoom_factor > 1.0 ?
+                 w->height < w->width * w->aspect_ratio :
+                 w->height > w->width * w->aspect_ratio))) {
What are you trying to achieve here ?

Thanks for your review!

This is meant for adapting the graph size to always fit into the window
at 100%, while for zoom factors > 100% we don't want to have "borders"
between the graph and the window in one dimension, i.e. the whole
visible area should always be covered by the graph. Thus we align to the
relatively shorter dimension for 100% zoom (shorter when considering the
aspect ratio), while aligning to the relatively longer dimension (again
considering the aspect ratio).

-- 
Christian Neumair <chris gnome-de org>

Attachment: signature.asc
Description: This is a digitally signed message part



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]