Skip to content

Commit

Permalink
egui_web: constrain the IME text agent to the canvas (#830)
Browse files Browse the repository at this point in the history
limit the position of the text agent to the height of the client area
  • Loading branch information
sumibi-yakitori authored Nov 2, 2021
1 parent eda1d91 commit b1716be
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion egui_web/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1242,7 +1242,10 @@ fn move_text_cursor(cursor: &Option<egui::Pos2>, canvas_id: &str) -> Option<()>
if is_mobile() == Some(false) {
cursor.as_ref().and_then(|&egui::Pos2 { x, y }| {
let canvas = canvas_element(canvas_id)?;
let y = y + (canvas.scroll_top() + canvas.offset_top()) as f32;
let y = (y + (canvas.scroll_top() + canvas.offset_top()) as f32).min(
canvas.client_height() as f32
- text_agent().get_bounding_client_rect().height() as f32,
);
let x = x + (canvas.scroll_left() + canvas.offset_left()) as f32;
// Canvas is translated 50% horizontally in html.
let x = x - canvas.offset_width() as f32 / 2.0;
Expand Down

0 comments on commit b1716be

Please sign in to comment.