mdn / content

The content behind MDN Web Docs
https://developer.mozilla.org
Other
9.2k stars 22.48k forks source link

[Other] WebDriver firefoxOptions activities/intents #4315

Closed chrisdavidmills closed 3 years ago

chrisdavidmills commented 3 years ago

Acceptance criteria

    • [ ] The listed features are documented sufficiently on MDN
    • [ ] BCD is updated
    • [ ] Interactive example and data repos are updated if appropriate
    • [ ] The content has been reviewed as needed

Features to document

  1. WebDriver firefoxOptions activities/intents

Related Gecko bugs

For folks helping with Firefox-related documentation features — make sure above AC have been done, but also:

whimboo commented 3 years ago

@chrisdavidmills I've set the dev-doc-needed flag when I thought that we would add this information on MDN. But then we actually decided against that and now have this default set compiled into geckodriver itself.

Do you think it would still helpful for users to see these defaults on MDN?

chrisdavidmills commented 3 years ago

@chrisdavidmills I've set the dev-doc-needed flag when I thought that we would add this information on MDN. But then we actually decided against that and now have this default set compiled into geckodriver itself.

Do you think it would still helpful for users to see these defaults on MDN?

I'm happy with whatever you think is useful for WebDriver users. My main thought is that it might be useful for people to know what the defaults are, and then be given the alternative options, so they can change them if they want something else. But I don't know a great deal about WebDriver.

hamishwillee commented 3 years ago

FWIW I think it is important to have one canonical source for information. So if everything is documented somewhere in webdriver itself then all you would have on MDN would be an overview of what you can do/what it is for, and then instructions on how to get any other information you need. What isn't clear is whether you're just documenting part of what is needed elsewhere or everything.

If it is everything, can we get what is on MDN removed?

hamishwillee commented 3 years ago

@whimboo @chrisdavidmills So is there a resolution on what we do with this? (just looking at unassigned items to see which I should take).

As per https://github.com/mdn/content/issues/4315#issuecomment-826594183 it isn't clear to me what the docs story is for web driver.

chrisdavidmills commented 3 years ago

@whimboo can you clarify for us — are the WebDriver docs on MDN helpful, or should we just provide a single overview page and link to the in tree docs? I'm happy either way, but if it would be better to just link to the in tree docs, we should probably close this issue, and open a new issue for cleaning up what is on MDN. Thanks!

whimboo commented 3 years ago

Hey and sorry for the delay with my response. So this issue has been actually extended to all topics regarding WebDriver, compared to the initial creation for just the Android activities and intents.

Let me start with the Android topic. Originally we wanted to show a list of default androidActivity, and androidIntentArguments on MDN so that users can correctly start Firefox (including Nightly, debug builds, Klar/Focus,...) on Android. But as we finally agreed on all the defaults are now part of geckodriver itself. It means that consumers of geckodriver don't have to specify any of these capabilities by default. As such what we already have on MDN regarding Android should be totally sufficient.

Regarding the overall approach to document WebDriver commands we should definitely add more content to MDN. While the WebDriver specification itself is targeted to those who actually implement the browser specific drivers (geckodriver, chromedriver, safaridriver, edgedriver,...), the documentation on MDN should give a resource for clients using these drivers. So topics like which commands are available, what is their HTTP end-point, which HTTP method and payload do they need, and what's the response. But all this should be moved to a new issue for further discussion.

As such should we close this issue, and create a new one for the general WebDriver documentation on MDN?

chrisdavidmills commented 3 years ago

As such should we close this issue, and create a new one for the general WebDriver documentation on MDN?

Makes perfect sense, thanks for the clarification, and yes, I agree entirely. I'll close this and create a new opportunity assessment issue for the general WebDriver work