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

added partial solution #12

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

razvanrus2003
Copy link

@razvanrus2003 razvanrus2003 commented Jan 20, 2024

#5
-added support for utf8 characters for save and find commands.
-highlight bugs for fullwidth characters

@razvanrus2003
Copy link
Author

-fixed highlight bug by using unicode_width crate to determine String display width instead of the number of bits/characters

@razvanrus2003
Copy link
Author

razvanrus2003 commented Jan 20, 2024

-final solution

Copy link

@alexandruradovici alexandruradovici left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the build issues.

Copy link

@alexandruradovici alexandruradovici left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please send the pull request to upstream and put a link to it here.

@razvanrus2003
Copy link
Author

ilai-deutel#281
here

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

Successfully merging this pull request may close these issues.

3 participants