Implement changes from code reviews

masonry/src/passes/update.rs

@@ -205,6 +205,11 @@ pub(crate) fn run_update_disabled_pass(root: &mut RenderRoot) {
 
 // ----------------
 
+// This pass will update scroll positions in cases where a widget has requested to be
+// scrolled into view (usually a textbox getting text events).
+// Each parent that implements scrolling will update its scroll position to ensure the
+// child is visible. (If the target area is larger than the parent, the parent will try
+// to show the top left of that area.)
 pub(crate) fn run_update_scroll_pass(root: &mut RenderRoot) {
     let _span = info_span!("update_scroll").entered();
 

masonry/src/widget/portal.rs

@@ -149,12 +149,7 @@ impl<W: Widget> Portal<W> {
     // Note - Rect is in child coordinates
     // TODO - Merge with pan_viewport_to
     // Right now these functions are just different enough to be a pain to merge.
-    pub fn pan_viewport_to_raw(
-        &mut self,
-        portal_size: Size,
-        content_size: Size,
-        target: Rect,
-    ) -> bool {
+    fn pan_viewport_to_raw(&mut self, portal_size: Size, content_size: Size, target: Rect) -> bool {
         let viewport = Rect::from_origin_size(self.viewport_pos, portal_size);
 
         let new_pos_x = compute_pan_range(

masonry/src/widget/textbox.rs

@@ -13,6 +13,7 @@ use vello::{
     peniko::{BlendMode, Color},
     Scene,
 };
+use winit::event::Ime;
 
 use crate::widget::{LineBreaking, WidgetMut};
 use crate::{
@@ -205,10 +206,13 @@ impl Widget for Textbox {
 
     fn on_text_event(&mut self, ctx: &mut EventCtx, event: &TextEvent) {
         let result = self.editor.text_event(ctx, event);
-        // If focused on a link and enter pressed, follow it?
         if result.is_handled() {
-            // TODO - Use request_scroll_to with cursor rect
-            ctx.request_scroll_to_this();
+            // Some platforms will send a lot of spurious Preedit events.
+            // We only want to request a scroll on user input.
+            if !matches!(event, TextEvent::Ime(Ime::Preedit(..))) {
+                // TODO - Use request_scroll_to with cursor rect
+                ctx.request_scroll_to_this();
+            }
             ctx.set_handled();
             // TODO: only some handlers need this repaint
             ctx.request_layout();