Report cursor size to input method

Author: dcz

core/src/input_method.rs

@@ -1,5 +1,5 @@
 //! Listen to input method events.
-use crate::{Pixels, Point};
+use crate::{Pixels, Rectangle};
 
 use std::ops::Range;
 
@@ -11,7 +11,7 @@ pub enum InputMethod<T = String> {
     /// Input method is enabled.
     Enabled {
         /// The position at which the input method dialog should be placed.
-        position: Point,
+        cursor_position: Rectangle,
         /// The [`Purpose`] of the input method.
         purpose: Purpose,
         /// The preedit to overlay on top of the input method dialog, if needed.
@@ -130,11 +130,11 @@ impl<T> InputMethod<T> {
         match self {
             Self::Disabled => InputMethod::Disabled,
             Self::Enabled {
-                position,
+                cursor_position,
                 purpose,
                 preedit,
             } => InputMethod::Enabled {
-                position: *position,
+                cursor_position: *cursor_position,
                 purpose: *purpose,
                 preedit: preedit.as_ref().map(Preedit::to_owned),
             },

widget/src/scrollable.rs

@@ -750,10 +750,10 @@ where
                 );
 
                 if !had_input_method {
-                    if let InputMethod::Enabled { position, .. } =
+                    if let InputMethod::Enabled { cursor_position, .. } =
                         shell.input_method_mut()
                     {
-                        *position = *position - translation;
+                        *cursor_position = *cursor_position - translation;
                     }
                 }
             };

widget/src/text_editor.rs

@@ -356,11 +356,10 @@ where
             self.text_size.unwrap_or_else(|| renderer.default_size()),
         );
 
-        let position =
-            cursor + translation + Vector::new(0.0, f32::from(line_height));
+        let position = cursor + translation;
 
         InputMethod::Enabled {
-            position,
+            cursor_position: Rectangle::new(position, Size::new(0.0, f32::from(line_height))),
             purpose: input_method::Purpose::Normal,
             preedit: state.preedit.as_ref().map(input_method::Preedit::as_ref),
         }

widget/src/text_input.rs

@@ -430,7 +430,10 @@ where
             + alignment_offset;
 
         InputMethod::Enabled {
-            position: Point::new(x, text_bounds.y + text_bounds.height),
+            cursor_position: Rectangle::new(
+                Point::new(x, text_bounds.y),
+                Size::new(0.0, text_bounds.height),
+            ),
             purpose: if self.is_secure {
                 input_method::Purpose::Secure
             } else {

winit/src/window.rs

@@ -172,7 +172,7 @@ where
     pub renderer: P::Renderer,
     pub redraw_at: Option<Instant>,
     preedit: Option<Preedit<P::Renderer>>,
-    ime_state: Option<(Point, input_method::Purpose)>,
+    ime_state: Option<(Rectangle, input_method::Purpose)>,
 }
 
 impl<P, C> Window<P, C>
@@ -217,11 +217,11 @@ where
                 self.disable_ime();
             }
             InputMethod::Enabled {
-                position,
+                cursor_position,
                 purpose,
                 preedit,
             } => {
-                self.enable_ime(position, purpose);
+                self.enable_ime(cursor_position, purpose);
 
                 if let Some(preedit) = preedit {
                     if preedit.content.is_empty() {
@@ -231,7 +231,7 @@ where
                             self.preedit.take().unwrap_or_else(Preedit::new);
 
                         overlay.update(
-                            position,
+                            cursor_position.position(),
                             &preedit,
                             self.state.background_color(),
                             &self.renderer,
@@ -260,19 +260,19 @@ where
         }
     }
 
-    fn enable_ime(&mut self, position: Point, purpose: input_method::Purpose) {
+    fn enable_ime(&mut self, cursor: Rectangle, purpose: input_method::Purpose) {
         if self.ime_state.is_none() {
             self.raw.set_ime_allowed(true);
         }
 
-        if self.ime_state != Some((position, purpose)) {
+        if self.ime_state != Some((cursor, purpose)) {
             self.raw.set_ime_cursor_area(
-                LogicalPosition::new(position.x, position.y),
-                LogicalSize::new(10, 10), // TODO?
+                LogicalPosition::new(cursor.x, cursor.y),
+                LogicalSize::new(cursor.width, cursor.height),
             );
             self.raw.set_ime_purpose(conversion::ime_purpose(purpose));
 
-            self.ime_state = Some((position, purpose));
+            self.ime_state = Some((cursor, purpose));
         }
     }