-
Notifications
You must be signed in to change notification settings - Fork 470
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
AutomationPeer editor extension #332
base: master
Are you sure you want to change the base?
AutomationPeer editor extension #332
Conversation
Thank you very much for your pull request! The changes are looking good, just a few remarks:
If you don't have time to apply these changes, I can do that for you once we merge this. I will have to take a closer look and test how Narrator behaves now. |
I have renamed the property. Now I need to figure how to get rid of the "bad commits" |
This comment has been minimized.
This comment has been minimized.
4b05e8c
to
35ccb67
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…at represents IntelliSense suggesiton list and the currently selected suggestion - to allow screen narrators to read the suggestion.
35ccb67
to
962dffd
Compare
Hmmm, it seems, this change currently does not provide a out-of-the-box solution for users of AvalonEdit. I would have expected Narrator to properly work out-of-the-box with how we currently use/instantiate the AvalonEdit/ICSharpCode.AvalonEdit.Sample/Window1.xaml.cs Lines 124 to 154 in 395ef81
Seems like I will have to dig around to get this to work properly like in VS. Thanks anyway for the starting point. |
Add extension to text editor that allows to inject automation peer that represents IntelliSense suggestion list and the currently selected suggestion - to allow screen narrators to read the suggestion.