Merge pull request #157 from plone/wait_for_element

Makes ``Wait For Then Click Element`` keyword more robust
plone · May 1, 2024 · 15ac811 · 15ac811
2 parents 51caf70 + 2802561
commit 15ac811
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/news/157.bugfix b/news/157.bugfix
@@ -0,0 +1 @@
+Makes ``Wait For Then Click Element`` keyword more robust. @wesleybl
diff --git a/src/plone/app/robotframework/selenium.robot b/src/plone/app/robotframework/selenium.robot
@@ -101,8 +101,9 @@ Wait For Element
     ...              Element must match exactly one time.
     [Arguments]  ${element}
     Wait Until Page Contains Element  ${element}
-    Set Focus To Element  ${element}
     Wait Until Element Is Visible  ${element}
+    Wait Until Element Is Enabled  ${element}
+    Set Focus To Element  ${element}
     Sleep  0.1
     ${count} =  Get Element Count  ${element}
     Should Be Equal as Numbers  ${count}  1
@@ -112,8 +113,9 @@ Wait For Elements
     ...              Element may match more than once.
     [Arguments]  ${element}
     Wait Until Page Contains Element  ${element}
-    Set Focus To Element  ${element}
     Wait Until Element Is Visible  ${element}
+    Wait Until Element Is Enabled  ${element}
+    Set Focus To Element  ${element}
 
 Wait For Then Click Element
     [Documentation]  Can contain css=, jquery=, or any other element selector.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Makes ``Wait For Then Click Element`` keyword more robust. @wesleybl