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

Revert "V2 of UI Automation in Windows Console: fix setEndPoint/compareEndPoints" #10088

Conversation

codeofdusk
Copy link
Contributor

This PR introduces serious degradation of character and word review functionality in consoles.

Reverts #10057

@codeofdusk
Copy link
Contributor Author

Cc @feerrenrut @LeonarddeR

@LeonarddeR
Copy link
Collaborator

I think I know what went wrong in #10057

Furst, you had a property that was positively worded. self._hasText, so you did a not self._hasText check in compareEndPoints and setEndPoint.
Now, we are using _isCollapsed, which is the negated equivalent. However, you are checking not self._is Collapsed, which is exactly the opposite of what we want🤣

I'm closing this. Could you please open a new pr according to the instructions above?

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.

2 participants