Fixed issue with EntityListView handling of invalidate(). For some reason revalidateLater() is insufficient in this case as, if the component is being scrolled it could be painted in an inconsistent state. This may point to a problem with revalidateLater() which I will investigate, but for now fixing it by changing call to revalidate().

Author: shannah

bin/CodeRAD.cn1lib

@@ -6,6 +6,7 @@
 package com.codename1.rad.controllers;
 
 import com.codename1.ui.Button;
+import com.codename1.ui.CN;
 import com.codename1.ui.Command;
 import com.codename1.ui.Component;
 import com.codename1.ui.ComponentSelector;
@@ -269,12 +270,24 @@ public Form getView() {
     /**
      * Goes back to previous form.  The previous form is always the parent controller's
      * form, if the parent controller is a FormController.
+     * 
+     * If the current controller is an instance of {@link AutoCloseable},
+     * then this will first attempt to call {@link AutoCloseable#close() } on the current form first.  If it throws an exception,
+     * then that exception will be swallowed, but the showBack() action will be cancelled.
      */
     public void back() {
         Controller parent = getParent();
         if (parent != null) {
             FormController fc = parent.getFormController();
             if (fc != null) {
+                if (this instanceof AutoCloseable) {
+                    AutoCloseable ac = (AutoCloseable)this;
+                    try {
+                        ac.close();
+                    } catch (Exception ex) {
+                        return;
+                    }
+                }
                 fc.getView().showBack();
             }
         }
@@ -284,7 +297,25 @@ public void show() {
         getView().show();
     }
 
+    /**
+     * Shows this controller's form using the "back" animation.  If the current controller is an instance of {@link AutoCloseable},
+     * then this will first attempt to call {@link AutoCloseable#close() } on the current form first.  If it throws an exception,
+     * then that exception will be swallowed, but the showBack() action will be cancelled.
+     */
     public void showBack() {
+        Form currForm = CN.getCurrentForm();
+        if (currForm != null) {
+            ViewController currentController = getViewController(currForm);
+            if (currentController instanceof AutoCloseable) {
+                AutoCloseable ac = (AutoCloseable)currentController;
+                try {
+                    ac.close();
+                } catch (Exception ex) {
+                    return;
+                }
+            }
+        }
+        
         getView().showBack();
     }
 

src/com/codename1/rad/controllers/FormController.java

@@ -16,6 +16,7 @@
 import com.codename1.rad.models.PropertySelector;
 import com.codename1.rad.models.Tag;
 import com.codename1.rad.models.TextFormatterAttribute;
+import com.codename1.rad.text.TextFormatter;
 import com.codename1.rad.ui.UI;
 import com.codename1.rad.ui.image.PropertyImageRenderer;
 import com.codename1.rad.ui.image.RoundImageRenderer;
@@ -108,7 +109,7 @@ private String _getText() {
 
     protected String getText() {
         String out = _getText();
-        TextFormatterAttribute formatter = (TextFormatterAttribute)getField().findAttribute(TextFormatterAttribute.class);
+        TextFormatterAttribute formatter = (TextFormatterAttribute)getField().findInheritedAttribute(TextFormatterAttribute.class);
         if (formatter != null) {
             return formatter.getValue().format(out);
         }
@@ -118,6 +119,7 @@ protected String getText() {
     @Override
     public void update() {
         String oldVal = getComponent().getText();
+        
         String newVal = iconOnly ? "" : getText();
 
         if (!Objects.equals(oldVal, newVal)) {

src/com/codename1/rad/propertyviews/LabelPropertyView.java

@@ -297,7 +297,10 @@ private void handleTransactionEvent(EntityList.TransactionEvent te) {
             update();
             Form f = getComponentForm();
             if (f != null) {
-                revalidateLater();
+                // Important: We seem to need revalidate() here rather than revalidateLater()
+                // because if an animation is in progress, the component may just disappear for a 
+                // moment.
+                revalidate();
             }
             return;
         }