Skip to content

Make Galley::pos_from_layout_cursor pub#7864

Merged
emilk merged 1 commit intoemilk:mainfrom
dionb:expose-pos_from_layout_cursor
Mar 3, 2026
Merged

Make Galley::pos_from_layout_cursor pub#7864
emilk merged 1 commit intoemilk:mainfrom
dionb:expose-pos_from_layout_cursor

Conversation

@dionb
Copy link
Contributor

@dionb dionb commented Jan 26, 2026

  • I have followed the instructions in the PR template

This PR just exposes the pos_from_layout_cursor function as public.

Hi, I'm trying to make a git gui with a merge editor, and for this I need a much more efficient and flexible text editor than the one currently in egui. So I'm working on building a more suitable one, which I intend to contribute back once it's working, but for now I would like to not need to fork the entirety of egui. By exposing this one function I (and others) can much more easily reuse Galleys.
I suggest also exposing end_pos, but I'm not currently using that. Let me know if I should update this PR to do so.

Thanks for the otherwise awesome tool :)

@github-actions
Copy link

Preview is being built...

Preview will be available at https://egui-pr-preview.github.io/pr/7864-expose-pos_from_layout_cursor

View snapshot changes at kitdiff

@emilk emilk changed the title expose pos_from_layout_cursor Make Galley::pos_from_layout_cursor pub Mar 3, 2026
@emilk emilk merged commit 20f3cb5 into emilk:main Mar 3, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants