Preview available at https://egui-pr-preview.github.io/pr/7081-clamp-cursors
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

Did you find other issues that require clamping? The reason I added the Weak<Galley> to TextArea state is to prevent clamp_cursor from running every frame, since it's O(N) to the number of rows. But maybe that's okay?

It's what the previous code did, so it's probably okay. It's possible we don't need to call cursor_from_layout and layout_from_cursor and can just clamp the cursor's byte range directly; I can test that out later.

Yeah that makes sense. Would be nice to do a quick sanity check to see how much it’s getting called.

By the way, my reasoning for clamping using cursor -> layout -> cursor, instead of the byte range was to maintain the cursor position as close as possible to the old one. For example, if the string is modified by unindenting it, I’d like the cursor to stay in the same line, instead of being moved downwards to the corresponding byte offset. I don’t think I tested that particular case though.