Skip to content

Feature/windows fixes #32

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

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

Conversation

fwaheed02
Copy link

Following fixes are made:

  • support unicode for windows
  • fixed line wrapping
  • Improved drawing by removing cursor

Fatima Waheed added 2 commits January 21, 2022 15:47
- support unicode for windows
- fixed line wrapping
- Improved drawing by removing curso
starseeker added a commit to starseeker/cpp-linenoise that referenced this pull request Oct 29, 2024
This is an attempt at merging the changes from
yhirose#32
@starseeker
Copy link
Contributor

@yhirose What's the status of this PR?

@yhirose
Copy link
Owner

yhirose commented Mar 10, 2025

@starseeker sorry for the late reply. To be honest, I didn't take a look at it since the current version satisfies my projects. But you can benefit from this pull request, I would like you to review it for me. If it looks good to you, please let me know. Thanks!

@starseeker
Copy link
Contributor

@fwaheed02 If you're still available to discuss this PR, can you provide an example image or two showing the cursor hiding?

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