You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2009/10/12 16:08:17 UTC

svn commit: r824343 - /directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java

Author: seelmann
Date: Mon Oct 12 14:08:17 2009
New Revision: 824343

URL: http://svn.apache.org/viewvc?rev=824343&view=rev
Log:
Refresh instead of set input, to avoid flickering

Modified:
    directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java

Modified: directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java?rev=824343&r1=824342&r2=824343&view=diff
==============================================================================
--- directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java (original)
+++ directory/studio/trunk/ldapbrowser-ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditor.java Mon Oct 12 14:08:17 2009
@@ -387,10 +387,16 @@
      */
     public void workingCopyModified( Object source )
     {
-        setEntryEditorWidgetInput( getEntryEditorInput() );
+        if ( mainWidget != null && !mainWidget.getViewer().isCellEditorActive() )
+        {
+            ISelection selection = mainWidget.getViewer().getSelection();
+            mainWidget.getViewer().refresh();
+            mainWidget.getViewer().setSelection( selection );
+        }
+
         if ( !isAutoSave() )
         {
-            // mark as dirty only
+            // mark as dirty
             firePropertyChange( PROP_DIRTY );
         }
     }
@@ -403,13 +409,9 @@
      */
     private void setEntryEditorWidgetInput( EntryEditorInput eei )
     {
-        // don't update input if an cell editor is active
-        if ( mainWidget != null && !mainWidget.getViewer().isCellEditorActive() )
+        if ( mainWidget != null )
         {
-            // set input, remember old selection and set it afterwards
-            ISelection selection = mainWidget.getViewer().getSelection();
             universalListener.setInput( eei.getSharedWorkingCopy( this ) );
-            mainWidget.getViewer().setSelection( selection );
         }
     }