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

SUPERCILEX / clipboard-history

Ringboard—the clipboard manager for Linux

Apache License 2.0

143 stars 5 forks source link

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

Closed ivan-toriya closed 4 weeks ago

ivan-toriya commented 4 weeks ago

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 commented 4 weeks ago

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 commented 3 weeks ago

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?

SUPERCILEX commented 3 weeks ago

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

ivan-toriya commented 3 weeks ago

Ah, yeah, it works, thanks!

SUPERCILEX commented 3 weeks ago

Yay!