Deprecate scroll_click() - All clicks now scroll first

mdmintz · mdmintz · commit 4f125e4bb4bf · 2017-10-05T23:17:35.000-04:00
diff --git a/help_docs/method_summary.md b/help_docs/method_summary.md
@@ -75,7 +75,7 @@ self.scroll_to(selector, by=By.CSS_SELECTOR)
 
 self.slow_scroll_to(selector, by=By.CSS_SELECTOR)
 
-self.scroll_click(selector, by=By.CSS_SELECTOR)
+self.scroll_click(selector, by=By.CSS_SELECTOR)  # DEPRECATED
 
 self.click_xpath(xpath)
 
diff --git a/seleniumbase/fixtures/base_case.py b/seleniumbase/fixtures/base_case.py
@@ -624,7 +624,8 @@ def slow_scroll_to(self, selector, by=By.CSS_SELECTOR,
         self._slow_scroll_to_element(element)
 
     def scroll_click(self, selector, by=By.CSS_SELECTOR):
-        self.scroll_to(selector, by=by)
+        # DEPRECATED - self.click() now scrolls to the element before clicking
+        # self.scroll_to(selector, by=by)
         self.click(selector, by=by)
 
     def click_xpath(self, xpath):
