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

Issue155 fixed. #191

Merged
merged 1 commit into from
Jan 16, 2014
Merged

Issue155 fixed. #191

merged 1 commit into from
Jan 16, 2014

Conversation

ilya-murzinov
Copy link
Contributor

The problem is that grid cell contains combobox, but current condition looks only for Text or CheckBox.
So I updated condition, thus combobox inside grid cell can be found.

By the way, this cell's text can't be accessed via Text property.

@ilya-murzinov ilya-murzinov mentioned this pull request Jan 6, 2014
@JakeGinnivan
Copy link
Member

It would be good to get a UI test for this (extend the current grid tests and put a combobox column in), would you be able to add a test?

JakeGinnivan added a commit that referenced this pull request Jan 16, 2014
@JakeGinnivan JakeGinnivan merged commit 592b6f3 into TestStack:master Jan 16, 2014
@ilya-murzinov
Copy link
Contributor Author

Yeah, I'll try to do it.
By the way, feel free to assign issues to me.

@JakeGinnivan
Copy link
Member

I prefer not assigning issues, pick things up as you feel like it.

The largest bit of work to do is actually getting the tests passing with the UIAComWrapper branch. This branch migrates White to the new COM version of UIA, which is faster and has a bunch of big fixes compared to the managed version white is currently on.

White will also be able to support winRT apps once that is done :)

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.

None yet

2 participants