8000 Wrote documentation for 6 element endpoints by dpgraham · Pull Request #9281 · appium/appium · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Wrote documentation for 6 element endpoints #9281

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

Merged
merged 3 commits into from
Oct 2, 2017

Conversation

dpgraham
Copy link
Contributor
@dpgraham dpgraham commented Sep 22, 2017

Documented the following element endpoints

  • /active
  • /elements
  • /element
  • /value
  • /submit
  • /text

* /active
* /elements
* /element
* /value
* /submit
* /text
Copy link
Member
@jlipps jlipps left a comment

Choose a reason for hiding this comment

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

comments made!

short_description: Gets the active element of the current session
description:
>
Get the element on the page that currently has focus. The element will be returned as a MobileElement JSON object.
Copy link
Member

Choose a reason for hiding this comment

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

i'm not sure MobileElement is a thing. i think it's just a term the java client uses? just the first sentence of this is probably fine.

csharp: "https://github.com/appium/appium-dotnet-driver/" # TODO

# Driver support by platform
driver_support:
Copy link
Member

Choose a reason for hiding this comment

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

this is actually supported on all these platforms? i'm kind of surprised

>
Located element will be returned as a MobileElement JSON object.

The locator strategy returns the first element it finds. #TODO: Let's make a document with the locator strategies that this links to
Copy link
Member

Choose a reason for hiding this comment

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

we talk about locator strategies in the new getting started doc, could link to that?

Copy link
Member

Choose a reason for hiding this comment

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

we could also put the list of standard locator strategies here, and mention that drivers may support additional strategies?

Copy link
Member

Choose a reason for hiding this comment

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

or if this is the one-stop shop for all drivers, we should list them in a table here I guess

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I think we should have a full table listing all of the strategies and a description for them and the support levels.

@@ -0,0 +1,84 @@
---
name: Find Element
short_description: Click on an element
Copy link
Member

Choose a reason for hiding this comment

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

this does not click on an element

short_description: Search for multiple elements
description:
>
The located elements will be returned as a list of MobileElement JSON objects. #TODO: Let's make a document with the locator strategies that this links to
Copy link
Member

Choose a reason for hiding this comment

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

maybe we should reference the find element doc

csharp: "https://github.com/appium/appium-dotnet-driver/" # TODO Dotnet documentation link

# Driver support by platform
driver_support:
Copy link
Member

Choose a reason for hiding this comment

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

i don't think this is supported unless you're in a webview context

- name: session_id
description: ID of the session to route the command to
- name: element_id
description: ID of the element to route the command to
Copy link
Member

Choose a reason for hiding this comment

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

i wouldn't say this command is being routed to an element. i'd just say it's the id of the element whose text we want.

@@ -0,0 +1,111 @@
# Find Element
Copy link
Member

Choose a reason for hiding this comment

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

why do we have both element.md and find-element.md? likewise with the multiple variant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was a leftover artifact. The generation wasn't cleaning the folder. I'll add a rimraf to that.

docs/toc.js Outdated
["Click", "click.md"]
["Click", "click.md"],
["Active Element", "active.md"],
["Find Element", "find-element.md"],
Copy link
Member

Choose a reason for hiding this comment

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

I would put the finds at the very top

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I think I'll add subcategories to elements as well. There's too many for one menu.

@@ -0,0 +1,70 @@
---
name: Active Element
Copy link
Member

Choose a reason for hiding this comment

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

"Active Element" doesn't sound like a command to me. maybe "Get Active Element"? or "Find Active Element"?

* rimraf document commands
* Fixed wording of some pages
driver.setValue("~SomeAccessibilityId");
ruby:
|
@driver.find_element(:accessibility_id, "SomeAccessibilityID").send_keys("Hello World!"")

Choose a reason for hiding this comment

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

I think there's an extra " in "Hello World!""

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good eye. Thanks!

@dpgraham dpgraham added this to the Robot XVII (Sprint 17) milestone Sep 27, 2017
@dpgraham dpgraham merged commit c0b921b into master Oct 2, 2017
@dpgraham dpgraham deleted the dpgraham-element_commands_01_to_05 branch October 2, 2017 16:21
dpgraham added a commit that referenced this pull request Oct 2, 2017
* /active
* /elements
* /element
* /value
* /submit
* /text
imurchie pushed a commit that referenced this pull request Nov 15, 2017
* /active
* /elements
* /element
* /value
* /submit
* /text
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.

4 participants
0