-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[🐛 Bug]: Missing support for RelativeBy in annotations #14881
Comments
@gryznar, thank you for creating this issue. We will troubleshoot it as soon as we can. Info for maintainersTriage this issue by using labels.
If information is missing, add a helpful comment and then
If the issue is a question, add the
If the issue is valid but there is no time to troubleshoot it, consider adding the
If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C),
add the applicable
After troubleshooting the issue, please add the Thank you! |
RelativeBy
in annotations in various placesRelativeBy
in annotations
RelativeBy
in annotations
Thank you for raising this issue @gryznar. If you could provide a PR for this, the team would be happy to take a look. |
Just plain changes? Or sth more is required? That's why I didn't created pr |
I'm referring to creating a PR with your suggested changes to see if it works/enhances as you suggest. I'm not sure if more is required, so I'm asking you to give it a shot if you have the time. You can refer to our Contribution guide for instructions on how to do so. Regardless, thank you for taking the time to raise this issue - we are always grateful to hear back from the community. |
What happened?
Despite the code will handle that properly, annotations are unadjusted to
RelativeBy
. First of all:find_element()
andfind_elements()
:None of them has annotation for
by
. In both cases, it should be:Union[ByType, RelativeBy]
:Another case are conditions in
expected_conditions.py
:Here annotations for locator are:
Tuple[str, str]
, whereas it should be:Union[Tuple[ByType, str], Tuple[RelativeBy, None]]
. Same applies for other conditions.How can we reproduce the issue?
Relevant log output
Operating System
n/a
Selenium version
4.24.0 for Python
What are the browser(s) and version(s) where you see this issue?
n/a
What are the browser driver(s) and version(s) where you see this issue?
n/a
Are you using Selenium Grid?
n/a
The text was updated successfully, but these errors were encountered: