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

Write selection text to hidden textarea on macOS #156717

Merged
merged 4 commits into from
Sep 12, 2022

Conversation

pingren
Copy link
Contributor

@pingren pingren commented Jul 30, 2022

Fixes #27799

Explanation

Before

macOS services are not available when editor.accessibilitySupport setting is off or auto

After (hidden textarea is made visible for illustration)

Code-OSS-30Jul22_23-10-19.mp4

@bpasero bpasero assigned hediet and alexdima and unassigned isidorn Jul 31, 2022
hediet
hediet previously approved these changes Aug 2, 2022
@alexdima alexdima self-requested a review August 2, 2022 12:54
Copy link
Member

@bpasero bpasero left a comment

Choose a reason for hiding this comment

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

Pushed a change to use getValueLengthInRange to avoid large allocations.

@bpasero bpasero requested a review from hediet September 9, 2022 11:10
@bpasero bpasero enabled auto-merge (squash) September 9, 2022 11:10
@bpasero bpasero added this to the September 2022 milestone Sep 9, 2022
@pingren
Copy link
Contributor Author

pingren commented Sep 9, 2022

I have read comments, but I am too busy to pick this up. Thank you for the improvement! Couldn't agree more!

@bpasero bpasero self-assigned this Sep 12, 2022
@bpasero bpasero merged commit c7c1717 into microsoft:main Sep 12, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Oct 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

macOS: OS services cannot pick up selected text from editor
5 participants