Regarding the Close () of QueryWidget and the two lines I think we don't
need any more in the ctor of MainWindow.cs.
543 query_widget.HandleChanged (query);
Added 2006 by Gabriel Burt. This is not necessary any more as this is
called implicitely by UpdateFindByTagMenu (), which is called in
MainWindow's ctor before.
544 query_widget.Close ();
Added by Maxxer, "Correctly set Find > Show/Hide find bar menu entry on
startup". I don't think this is necessary as the Close () is called
before several times. The mentioned UpdateFindByTagMenu () is calling
the Close (), too, as the QueryWidget is visible by default on creation.
As far as I can tell it was not needed then, as it was called before, too.
Nevertheless, this will remove two lines of code and save a little bit
of startup processor work, but won't us help much getting rid of the
MainWindow.Toplevel dependency.
A Patch: https://bugzilla.gnome.org/show_bug.cgi?id=601673
Regards
Paul.
_______________________________________________
F-spot-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/f-spot-list
