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

gEDA-bug: [ geda-Bugs-1699959 ] Zoom extents takes multiple repeats to "converge"



Bugs item #1699959, was opened at 2007-04-13 12:58
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=818426&aid=1699959&group_id=161080

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: gschem
Group: CVS
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Peter Clifton (petercjclifton)
Assigned to: Nobody/Anonymous (nobody)
Summary: Zoom extents takes multiple repeats to "converge"

Initial Comment:
I've noticed that new pages opened (blank except for a title-block) don't zoom to the same extents which they do if you manually run zoom-extents again afterwards.

This didn't happen for the first window opened by gschem, and it turns out that there is an extra re-draw in there. In addition, the new page hook appears to be called more times than needed.

(Filing a bug to poke me into extracting the work I did looking into this from my "PLAY" tree).

I've done a little debugging on the zoom issue, and it appears that it is related to the page->{left,top,right,bottom} variables being initialised to toplevel->init_{left,top,right,bottom} rather than the actual window size.

I was able to make it work better with:

+  set_window (toplevel, page,
+              toplevel->init_left, toplevel->init_left + toplevel->win_width,
+              toplevel->init_top,  toplevel->init_top + toplevel->win_height);
+

before the a_zoom_extents(...) in x_window_open_untitled_page().

I'm not convinced I know enough about how these _should_ be initialised to state this is the correct way to fix things.

Comments?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=818426&aid=1699959&group_id=161080


_______________________________________________
geda-bug mailing list
geda-bug@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-bug