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

Twix CompletionEdit #1478

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Twix CompletionEdit #1478

wants to merge 11 commits into from

Conversation

oleflb
Copy link
Contributor

@oleflb oleflb commented Oct 27, 2024

Why? What?

Fix twix completion edit since its broken on main.
I like the idea of trying to implement generally usable widgets in a separate crate from twix so that they can be used in other tools as well. Therefore I rewrote the completion edit to be less dependent on what is currently searched.
The fuzzy finder is replaced by the one from helix since it is way faster, besides that, the matching algorithm now is started only when the user input changed, not every frame.

The SegmentedControl widget is another generally usable widget I implemented some time ago for another tool and now I wanted to be available for twix as well.

Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

  • Thoroughly test the completion edits since it is essential for working with twix

@oleflb oleflb added tools:Debug Tooling Related to twix et.al. is:Bugfix labels Oct 27, 2024
@oleflb oleflb enabled auto-merge October 27, 2024 15:25
@oleflb oleflb requested a review from rmburg October 30, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:Bugfix tools:Debug Tooling Related to twix et.al.
Projects
Status: Request for Review
Development

Successfully merging this pull request may close these issues.

2 participants