Merge branch 'qa-remove-click-body' into 'master'

Remove uses of click_body in e2e specs

See merge request gitlab-org/gitlab!24313

qa/qa/ee/page/group/epic/show.rb

@@ -35,7 +35,8 @@ module QA
             def add_issue_to_epic(issue_url)
               find_element(:issue_actions_split_button).find('button', text: 'Add an issue').click
               fill_element :add_issue_input, issue_url
-              click_body
+              # Clicking the title blurs the input
+              click_element :title
               click_element :add_issue_button
             end
 

qa/qa/page/base.rb

@@ -255,12 +255,6 @@ module QA
         click_link text
       end
 
-      def click_body
-        wait_for_requests
-
-        find('body').click
-      end
-
       def visit_link_in_element(name)
         visit find_element(name)['href']
       end

qa/qa/page/component/select2.rb

@@ -34,6 +34,10 @@ module QA
           has_css?('.select2-active', wait: 1)
           has_no_css?('.select2-active', wait: 30)
         end
+
+        def dropdown_open?
+          has_css?('.select2-input')
+        end
       end
     end
   end

qa/qa/page/project/issue/show.rb

@@ -124,7 +124,7 @@ module QA
               end
             end
 
-            click_body
+            click_element(:edit_link_labels)
 
             labels.each do |label|
               has_element?(:labels_block, text: label)

qa/qa/page/project/new.rb

@@ -32,8 +32,7 @@ module QA
 
         def choose_namespace(namespace)
           retry_on_exception do
-            click_body
-            click_element :project_namespace_select
+            click_element :project_namespace_select unless dropdown_open?
             search_and_select(namespace)
           end
         end