Changeset 1511 for trunk/src/org/expeditee/items/Text.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/items/Text.java (modified) (4 diffs)

trunk/src/org/expeditee/items/Text.java

@@ -393 +393 @@
         @Override
         public void setWidth(Integer width) {
+                System.err.println("Text::setWidth::Text content=" + getText() + ", new width=" + width);
                 invalidateAll();
 
@@ -400 +401 @@
                         return;
                 }
-
+                
+                if (width == 0) {
+                        System.err.println("Width of Zero: " + getText());
+                }
+                
                 _width = width;
                 rebuild(true);
@@ -1783 +1788 @@
 
                 if (this.getMinWidth() != null && getTextLayouts().size() == 0 && this.getFont() != null) {
+                        // could use any text, 'p' is used simply to get some text in correct font.
                         getTextLayouts().add(TextLayout.getManager().layoutStringSimple("p", this.getFont()));
                         isFakeLayout = true;
@@ -2910 +2916 @@
                         if (anchor != null) {
                                 setX(DisplayController.getFramePaintAreaWidth() - anchor - getBoundsWidth() + getLeftMargin());
+                                //System.err.println("Text::setAnchorRight::boundsWidth=" + getBoundsWidth());
                         }
                         return;