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

make area use default cursor writer #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

russell
Copy link

@russell russell commented Oct 6, 2023

Make the area use the default cursor writer so that if the default cursor is set, then its write is used within the area writer.

I'm pretty sure this is the correct behaviour that should be happening, but let me also describe the problem i'm facing.

I'm writing tests that interact with the CLI elements, and i have been struggling to redirect the output of the interactive widgets away from stdout, I'm using this library via pterm, and i found a function in pterm pterm.SetDefaultOutput which could redirect output, i found that cursor had cursor.SetTarget but default areas were hard coded to stdout.

I could add a way to explicitly set the default area writer, but i'm not familiar with the usecases of setting a default writer to know if you would ever want the cursor to have a different writer to the area.

Make the area use the default area writer so that if the default
cursor is set, then its write is used within the area writer.
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.

1 participant