Error in Component Width after Screen Rotation

ewpatton · December 22, 2022, 6:43pm

I've never quite understood this decision, but there is a code-based reason for this:

mit-cml/appinventor-sources/blob/3d0ad63de092c6f41b96e9e794427ace4d151959/appinventor/components/src/com/google/appinventor/components/runtime/Form.java#L476


      
              }
            } else { // Portrait
              if (frameLayout.getHeight() >= frameLayout.getWidth()) {
                dispatchEventNow = true;
              }
            }
          }
          if (dispatchEventNow) {
            recomputeLayout();
            final FrameLayout savedLayout = frameLayout;
            androidUIHandler.postDelayed(new Runnable() {
                public void run() {
                  if (savedLayout != null) {
                    savedLayout.invalidate();
                  }
                }
              }, 100);          // Redraw the whole screen in 1/10 second
                                // we do this to avoid screen artifacts left
                                // left by the Android runtime.
            ScreenOrientationChanged();
          } else {

I'm not 100% sure that whatever reasons for the 100 ms delay is true any more. One thing I'm considering is moving the call to ScreenOrientationChanged into the delayed redraw operation so that the behavior is more consistent with people's expectations about what this event represents.