Bug 123577

Cannot reproduce this.

Also a problem with the picture object

Created attachment 15119 [details]
Proposed patch

Makes behaviour of label and picture more consistent with other view objects.
The view objects are displayed immediately. Hitting Apply when the dialog is
first shown after the object creation has a visible effect.

If I'm not mistaken, that backs out:

------------------------------------------------------------------------
r490161 | staikos | 2005-12-20 18:30:23 -0500 (Tue, 20 Dec 2005) | 6 lines

only act as a new dialog if it really is a new dialog.  new variable created
to indicate that this is a 'new' object
backs out some of 490135 to keep legends consistent with labels, since it's
really the same bug.
BUG: 118755



Maybe I misunderstood what the bug is that you're reporting.  Are you saying 
that the problem is that the object does not appear until clicking Apply?  If 
so, the fix should be, in "New" mode, to disable Apply since Apply can't 
"apply" to something that doesn't exist yet.

The problem is that clicking Apply has no effect because the object is not visible. The object should be visible when the dialog is launched.

While the proposed patch may (I haven't investigated) conflict with some changes you made for BUG:118755 it in no way conflicts with the original bug - which had to do with an entirely different issue. 

   I don't think that we should be creating objects at this stage.  They're 
not finalized, and creating them is simply a hack around the fact that Apply 
doesn't really make sense yet.

As far as I can see, this is all that's needed:

Index: ksteditviewobjectdialog_i.cpp
===================================================================
--- ksteditviewobjectdialog_i.cpp       (revision 518714)
+++ ksteditviewobjectdialog_i.cpp       (working copy)
@@ -63,6 +63,7 @@

 void KstEditViewObjectDialogI::setNew() {
   _isNew = true;
+  _apply->setEnabled(false);
 }


(or _apply->hide() if you prefer)

That is an option, but the wrong one I think.

SVN commit 520475 by staikos:

In any case, Apply should be disabled for a new object.  It's only for object
edits.  Basically invalidates this bug report.
BUG: 123577



 M  +1 -0      ksteditviewobjectdialog_i.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/ksteditviewobjectdialog_i.cpp #520474:520475
@@ -63,6 +63,7 @@
 
 void KstEditViewObjectDialogI::setNew() {
   _isNew = true;
+  _apply->setEnabled(false);
 }