Fine tuning

aaltat · aaltat · commit 916714eb0982 · 2019-09-15T00:10:20.000+03:00
diff --git a/src/SeleniumLibrary/__init__.py b/src/SeleniumLibrary/__init__.py
@@ -138,7 +138,7 @@ class SeleniumLibrary(DynamicCore):
     Different locator strategies have different pros and cons. Using ids,
     either explicitly like ``id:foo`` or by using the `default locator
     strategy` simply like ``foo``, is recommended when possible, because
-    the syntax is simple and locating elements by id is fast for browsers.
+    the syntax is simple and locating elements by an id is fast for browsers.
     If an element does not have an id or the id is not stable, other
     solutions need to be used. If an element has a unique tag name or class,
     using ``tag``, ``class`` or ``css`` strategy like ``tag:h1``,
@@ -281,12 +281,12 @@ class SeleniumLibrary(DynamicCore):
     https://robocon.io/, https://github.com/robotframework/'
     and 'https://github.com/.
 
-    = Timeouts, waits, and delays =
+    = Timeouts, waits and delays =
 
     This section discusses different ways how to wait for elements to
     appear on web pages and to slow down execution speed otherwise.
     It also explains the `time format` that can be used when setting various
-    timeouts, waits, and delays.
+    timeouts, waits and delays.
 
     == Timeout ==
 
@@ -385,11 +385,11 @@ class SeleniumLibrary(DynamicCore):
 
     = Thread support =
 
-    SeleniumLibrary is not thread-safe. This is mainly due because the underlying
+    SeleniumLibrary is not thread safe. This is mainly due because the underlying
     [https://github.com/SeleniumHQ/selenium/wiki/Frequently-Asked-Questions#q-is-webdriver-thread-safe|
-    Selenium tool is not thread-safe] within one browser/driver instance.
+    Selenium tool is not thread safe] within one browser/driver instance.
     Because of the limitation in the Selenium side, the keywords or the
-    API provided by the SeleniumLibrary is not thread-safe.
+    API provided by the SeleniumLibrary is not thread safe.
 
     = Plugins =
 
diff --git a/src/SeleniumLibrary/keywords/alert.py b/src/SeleniumLibrary/keywords/alert.py
@@ -75,7 +75,7 @@ def alert_should_not_be_present(self, action=ACCEPT, timeout=0):
         `Handle Alert` keyword.
 
         ``timeout`` specifies how long to wait for the alert to appear.
-        By default, is not waited for the alert at all, but a custom time can
+        By default, the alert is not waited at all, but a custom time can
         be given if alert may be delayed. See the `time format` section
         for information about the syntax.
 
diff --git a/src/SeleniumLibrary/keywords/element.py b/src/SeleniumLibrary/keywords/element.py
@@ -512,7 +512,7 @@ def click_button(self, locator, modifier=False):
 
         See the `Locating elements` section for details about the locator
         syntax. When using the default locator strategy, buttons are
-        searched using ``id``, ``name``, and ``value``.
+        searched using ``id``, ``name`` and ``value``.
 
         See the `Click Element` keyword for details about the
         ``modifier`` argument.
diff --git a/src/SeleniumLibrary/keywords/formelement.py b/src/SeleniumLibrary/keywords/formelement.py
@@ -385,7 +385,7 @@ def page_should_not_contain_button(self, locator, message=None, loglevel='TRACE'
 
         See the `Locating elements` section for details about the locator
         syntax. When using the default locator strategy, buttons are
-        searched using ``id``, ``name``, and ``value``.
+        searched using ``id``, ``name`` and ``value``.
         """
         self.assert_page_not_contains(locator, 'button', message, loglevel)
         self.assert_page_not_contains(locator, 'input', message, loglevel)
diff --git a/src/SeleniumLibrary/keywords/window.py b/src/SeleniumLibrary/keywords/window.py
@@ -56,7 +56,7 @@ def switch_window(self, locator='MAIN', timeout=None, browser='CURRENT'):
         - The ``locator`` can specify an explicit strategy by using the format
           ``strategy:value`` (recommended) or ``strategy=value``. Supported
           strategies are ``name``, ``title``, and ``url``. These matches windows
-          using their name, title, or URL, respectively. Additionally, ``default``
+          using their name, title or URL, respectively. Additionally, ``default``
           can be used to explicitly use the default strategy explained above.
 
         - If the ``locator`` is ``NEW`` (case-insensitive), the latest
@@ -221,14 +221,14 @@ def set_window_size(self, width, height, inner=False):
 
         If ``inner`` parameter is set to True, keyword sets the necessary
         window width and height to have the desired HTML DOM _window.innerWidth_
-        and _window.innerHeight_. See `Boolean arguments` for more details on how to set boolean
-        arguments.
-
-        The ``inner`` argument is new since SeleniumLibrary 4.0.
+        and _window.innerHeight_. See `Boolean arguments` for more details on
+        how to set boolean arguments.
 
         This ``inner`` argument does not support Frames. If a frame is selected,
         switch to default before running this.
 
+        The ``inner`` argument is new since SeleniumLibrary 4.0.
+
         Example:
         | `Set Window Size` | 800 | 600 |      |
         | `Set Window Size` | 800 | 600 | True |
