Changeset 1511 for trunk/src/org/expeditee/gui/FrameUtils.java

Timestamp:

02/19/20 10:44:38 (4 years ago)

Author:

bnemhaus

Message:

Frame::Parse has been updated to include a new boolean parameter. When true, widgets that are created as a result of the parse send not only notify the widget framework that they have been added, but are also visible. When false, they only notify they have been added.

The widget framework now distinguishes between added and visible widgets, this fixes a bug. Bug: when programmatically adding a widget to not the current frame, it never gets properly removed and therefore still catches click events from users. By distinguishing between adding and making visible this is avoided.

---

Another bug has been fixed. Bug: When setting a text item to have a right anchor, and then subsequently reducing the size of the window, this text item would get a width of zero assigned. This was caused by some issues with the logic of how right margins for items were calculated.

File:

• trunk/src/org/expeditee/gui/FrameUtils.java (modified) (11 diffs)

trunk/src/org/expeditee/gui/FrameUtils.java

@@ -71 +71 @@
 import org.expeditee.items.Item;
 import org.expeditee.items.Item.HighlightMode;
+import org.expeditee.items.ItemParentStateChangedEvent;
 import org.expeditee.items.ItemUtils;
 import org.expeditee.items.JSItem;
@@ -680 +681 @@
          * @param txt
          *            Text to create the widget from. Must not be null.
+         * @param userEdit True if createWidget is being called because a user has made a edit, false otherwise.
          * 
          * @return True if created/added. False if could not create.
@@ -685 +687 @@
          * @author Brook Novak
          */
-        private static boolean createWidget(Frame frame, Text txt, ItemsList list) {
+        private static boolean createWidget(Frame frame, Text txt, ItemsList list, boolean userEdit) {
 
                 if (frame == null) {
@@ -723 +725 @@
 
                 frame.addAllItems(iw.getItems(), list);
+                
+                if (userEdit) {
+                        iw.onParentStateChanged(new ItemParentStateChangedEvent(frame, ItemParentStateChangedEvent.EVENT_TYPE_SHOWN));
+                }
 
                 return true;
@@ -728 +734 @@
         
         private static boolean createWidgetInBody(Frame frame, Text txt) {
-                return createWidget(frame, txt, frame.getBody(false));
+                return createWidget(frame, txt, frame.getBody(false), false);
         }
 
@@ -889 +895 @@
         }
         
-        private static void transformOutOfPlaceItems(Frame toParse, ItemsList toTransform) {
+        private static void transformOutOfPlaceItems(Frame toParse, ItemsList toTransform, boolean sendWidgetVisible) {
                 // Get all items from toTransform that have not been marked as deleted.
                 List<Item> items = toParse.getItems(false, toTransform);
@@ -988 +994 @@
                                                         // Check for interactive widgets
                                                 } else if (ItemUtils.startsWithTag(i, ItemUtils.TAG_IWIDGET)) {
-                                                        createWidget(toParse, (Text) i, toTransform);
+                                                        createWidget(toParse, (Text) i, toTransform, sendWidgetVisible);
                                                 }
 
@@ -1149 +1155 @@
          */
         public static void Parse(Frame toParse, boolean firstParse) {
-                Parse(toParse, firstParse, false);
+                Parse(toParse, firstParse, false, false);
         }
 
@@ -1160 +1166 @@
          *            an ao tag linked to a frame with an frameImage of a frame which
          *            also has an ao tag on it.
+         * @param userEdit TODO
          */     
-        public static void Parse(Frame toParse, boolean firstParse, boolean ignoreAnnotations) { 
+        public static void Parse(Frame toParse, boolean firstParse, boolean ignoreAnnotations, boolean userEdit) { 
                 List<String> accessList = Label.getAccessibleLabelsNames(toParse.getPrimaryBody());
                 
@@ -1167 +1174 @@
                 ItemsList surrogates = toParse.getSurrogateBody();
                 
-                transformOutOfPlaceItems(toParse, primaries);
-                transformOutOfPlaceItems(toParse, surrogates);
+                transformOutOfPlaceItems(toParse, primaries, userEdit);
+                transformOutOfPlaceItems(toParse, surrogates, userEdit);
                 
                 toParse.getInteractableItems().clear();
@@ -1801 +1808 @@
                 if (!DisplayController.isXRayMode()) {
                         if (toReparse != null) {
-                                Parse(toReparse, false, false);
+                                Parse(toReparse, false, false, true);
                         } else {
                                 if (toRecalculate != null) {