[EMAIL PROTECTED] writes: | Author: uwestoehr | Date: Fri Mar 2 23:20:32 2007 | New Revision: 17395 | | URL: http://www.lyx.org/trac/changeset/17395 | Log: | - fix bug 2458 | http://bugzilla.lyx.org/show_bug.cgi?id=2458 | - whitespace | | Modified: | lyx-devel/trunk/src/frontends/qt4/QGraphicsDialog.C | lyx-devel/trunk/src/frontends/qt4/QPrefsDialog.C | lyx-devel/trunk/src/insets/insetgraphicsParams.C | | Modified: lyx-devel/trunk/src/frontends/qt4/QGraphicsDialog.C | URL: http://www.lyx.org/trac/file/lyx-devel/trunk/src/frontends/qt4/QGraphicsDialog.C?rev=17395 | ============================================================================== | --- lyx-devel/trunk/src/frontends/qt4/QGraphicsDialog.C (original) | +++ lyx-devel/trunk/src/frontends/qt4/QGraphicsDialog.C Fri Mar 2 23:20:32 2007 | @@ -89,7 +89,7 @@ | this, SLOT( change_adaptor() ) ); | connect(getPB, SIGNAL( clicked() ), | this, SLOT( change_adaptor() ) ); | - connect(scaleCB, SIGNAL(clicked()), | + connect(scaleCB, SIGNAL(clicked() ), | this, SLOT(change_adaptor()) ); | connect(Scale, SIGNAL( textChanged(const QString&) ), | this, SLOT( change_adaptor() ) ); |
Rather remove the spaces instead of adding new ones... we do not use this kind of spacing elsewhere. (But if you really want this, you should at least make it consistent with the other...) | Modified: lyx-devel/trunk/src/insets/insetgraphicsParams.C | URL: http://www.lyx.org/trac/file/lyx-devel/trunk/src/insets/insetgraphicsParams.C?rev=17395 | ============================================================================== | --- lyx-devel/trunk/src/insets/insetgraphicsParams.C (original) | +++ lyx-devel/trunk/src/insets/insetgraphicsParams.C Fri Mar 2 23:20:32 2007 | @@ -68,7 +68,7 @@ | filename.erase(); | lyxscale = 100; // lyx scaling in percentage | display = graphics::DefaultDisplay; // display mode; see preferences | - scale = string(); // output scaling in percentage | + scale = string("100"); // output scaling in percentage | width = LyXLength(); | height = LyXLength(); | keepAspectRatio = false; // for LaTeX output This this hunk the actual bug fix? I think it is better to do this kind of commits as two commits. -- Lgb
