Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

egui ctrl+n using 0-based index instead of 1-based #26

Closed
ivan-toriya opened this issue Aug 14, 2024 · 5 comments
Closed

egui ctrl+n using 0-based index instead of 1-based #26

ivan-toriya opened this issue Aug 14, 2024 · 5 comments

Comments

@ivan-toriya
Copy link

I think it would make more sense to change it to 1-based index when using Ctrl + N to paste the Nth entry.

Also, another feature request is to see this index in the GUI. What do you think?

@SUPERCILEX
Copy link
Owner

The number overlay is a great idea, done. I think it renders the question of indexing obsolete because you can easily see which entry to select, but please let me know if you still think we should be 1 indexing after the update.

@ivan-toriya
Copy link
Author

I've updated using the cargo install-update -a with nightly rust. Restarted. Can't see the indexing. I'm super new to rust, what am I doing wrong?
image

@SUPERCILEX
Copy link
Owner

Oh it only shows up if you hit Ctrl. Does that work?

@ivan-toriya
Copy link
Author

Ah, yeah, it works, thanks!

@SUPERCILEX
Copy link
Owner

Yay!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants