seleniumhq.github.io icon indicating copy to clipboard operation
seleniumhq.github.io copied to clipboard

[🐛 Bug]: Document for "Locator strategies" does explain what is "By"

Open HGtz2222 opened this issue 2 years ago • 4 comments

What happened?

https://www.selenium.dev/documentation/webdriver/elements/locators/

In this page, the demo code show how to locator an element, such as:

driver.find_element(By.CLASS_NAME, "information")

But I cannot see what is the "By", and how can I import "By" in my code.

If I use Java, IDE can help me generate code to import "By" . But if I use Python, most IDE cannot do such thing. So I hope the document can add the import statement for these key identifiers such as "By".

Thanks very much!

What browsers and operating systems are you seeing the problem on?

uncorrelated

HGtz2222 avatar May 27 '23 15:05 HGtz2222

@HGtz2222, thank you for creating this issue. We will troubleshoot it as soon as we can.


Info for maintainers

Triage this issue by using labels.

If information is missing, add a helpful comment and then I-issue-template label.

If the issue is a question, add the I-question label.

If the issue is valid but there is no time to troubleshoot it, consider adding the help wanted label.

After troubleshooting the issue, please add the R-awaiting answer label.

Thank you!

github-actions[bot] avatar May 27 '23 15:05 github-actions[bot]

So, first, we are trying to move away from static code, and move all examples to our repo. But even in our repo we are trying to only import the lines of code that matter. At least this way there will be a big link to see the full example.

At the same time, I'm leaning towards us just using the full path for classes instead of doing imports to make it more obvious where it is coming from. e.g.:

webdriver.common.by.By.ID

titusfortner avatar Jun 17 '23 17:06 titusfortner

So do we thing this is a bug and need a fix or CR ?

@HGtz2222 I see for Python you can import By using

from selenium.webdriver.common.by import By

eaccmk avatar Sep 02 '23 05:09 eaccmk

Yes, we should either do the full path for it, or include the import line in a second reference.

titusfortner avatar Sep 02 '23 12:09 titusfortner

Closed via #1880

diemol avatar Aug 22 '24 08:08 diemol