On Thursday 30 November 2000 23:30, Herbert Voss wrote:
> in lyx 1.1.5 i could toggle the "use package geometry"-button,
> with or without any values in the four fields for the margins.
> in 1.1.6.pre2 you have to clear all four fields, otherwise
> you can't change the use of the geometry package.
>
> is there any sense for this new behaviour?
>
> Herbert

No sense at all. Here's the patch.

Please apply,
Angus


Index: ChangeLog
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/ChangeLog,v
retrieving revision 1.720
diff -u -r1.720 ChangeLog
--- ChangeLog   2000/12/07 15:30:52     1.720
+++ ChangeLog   2000/12/07 18:49:52
@@ -1,5 +1,10 @@
 2000-12-07  Angus Leeming <[EMAIL PROTECTED]>
 
+       * src/frontends/xforms/FormDocument.C (checkMarginValues): you don't
+       need to empty the fields to turn off use of the geometry package!
+
+2000-12-07  Angus Leeming <[EMAIL PROTECTED]>
+
        * src/lyxparagraph.h, src/paragraph.C (CopyIntoMinibuffer): pass a
        (Buffer const &), not a (BufferParams const &) and so fix a crash
        caused by using current_view before it had been initialised. Not
Index: src/frontends/xforms/FormDocument.C
===================================================================
RCS file: 
/usr/local/lyx/cvsroot/lyx-devel/src/frontends/xforms/FormDocument.C,v
retrieving revision 1.31
diff -u -r1.31 FormDocument.C
--- src/frontends/xforms/FormDocument.C 2000/11/29 15:06:42     1.31
+++ src/frontends/xforms/FormDocument.C 2000/12/07 18:49:53
@@ -842,8 +842,8 @@
                !strlen(fl_get_input(paper_->input_foot_skip)) &&
                !strlen(fl_get_input(paper_->input_custom_width)) &&
                !strlen(fl_get_input(paper_->input_custom_height)));
-    if (!allEmpty)
-       fl_set_button(paper_->push_use_geometry, 1);
+    //if (!allEmpty)
+    //fl_set_button(paper_->push_use_geometry, 1);
 }
 
 bool FormDocument::CheckDocumentInput(FL_OBJECT * ob, long)

Reply via email to