First sentence of javadoc should end with a period (#10114)

Also includes minor other javadoc and formatting fixes.

28 files changed, 76 insertions, 73 deletions

diff --git a/shared/src/main/java/com/vaadin/shared/AbstractComponentState.java b/shared/src/main/java/com/vaadin/shared/AbstractComponentState.java
index 9b949111a7..3f942c81a4 100644
--- a/shared/src/main/java/com/vaadin/shared/AbstractComponentState.java
+++ b/shared/src/main/java/com/vaadin/shared/AbstractComponentState.java
@@ -44,11 +44,11 @@ public class AbstractComponentState extends SharedState {
     public String id = null;
     public String primaryStyleName = null;
 
-    /** HTML formatted error message for the component */
+    /** HTML formatted error message for the component. */
     public String errorMessage = null;
 
     /**
-     * Level of error
+     * Level of error.
      *
      * @since 8.2
      */
diff --git a/shared/src/main/java/com/vaadin/shared/ApplicationConstants.java b/shared/src/main/java/com/vaadin/shared/ApplicationConstants.java
index 4ce1ee0ac0..6a16508afa 100644
--- a/shared/src/main/java/com/vaadin/shared/ApplicationConstants.java
+++ b/shared/src/main/java/com/vaadin/shared/ApplicationConstants.java
@@ -55,7 +55,7 @@ public class ApplicationConstants implements Serializable {
     public static final String FRONTEND_PROTOCOL_PREFIX = "frontend://";
 
     /**
-     * Prefix used for theme resource URLs
+     * Prefix used for theme resource URLs.
      *
      * @see com.vaadin.server.ThemeResource
      * @since 7.5.0
@@ -156,14 +156,14 @@ public class ApplicationConstants implements Serializable {
     public static final String PUSH_ID_PARAMETER = "v-pushId";
 
     /**
-     * The name of the parameter used to transmit RPC invocations
+     * The name of the parameter used to transmit RPC invocations.
      *
      * @since 7.2
      */
     public static final String RPC_INVOCATIONS = "rpc";
 
     /**
-     * The name of the parameter used to transmit the CSRF token
+     * The name of the parameter used to transmit the CSRF token.
      *
      * @since 7.2
      */
@@ -199,7 +199,7 @@ public class ApplicationConstants implements Serializable {
 
     /**
      * The name of the parameter used for sending the widget set version to the
-     * server
+     * server.
      *
      * @since 7.6
      */
diff --git a/shared/src/main/java/com/vaadin/shared/ContextClickRpc.java b/shared/src/main/java/com/vaadin/shared/ContextClickRpc.java
index f51f2fdf95..6288f3ea99 100644
--- a/shared/src/main/java/com/vaadin/shared/ContextClickRpc.java
+++ b/shared/src/main/java/com/vaadin/shared/ContextClickRpc.java
@@ -18,7 +18,7 @@ package com.vaadin.shared;
 import com.vaadin.shared.communication.ServerRpc;
 
 /**
- * Client-to-server RPC interface for context click events
+ * Client-to-server RPC interface for context click events.
  *
  * @since 7.6
  * @author Vaadin Ltd
diff --git a/shared/src/main/java/com/vaadin/shared/MouseEventDetails.java b/shared/src/main/java/com/vaadin/shared/MouseEventDetails.java
index 3954206138..a438f604d5 100644
--- a/shared/src/main/java/com/vaadin/shared/MouseEventDetails.java
+++ b/shared/src/main/java/com/vaadin/shared/MouseEventDetails.java
@@ -54,7 +54,7 @@ public class MouseEventDetails implements Serializable {
         }
 
         /**
-         * Returns a human readable text representing the button
+         * Returns a human readable text representing the button.
          *
          * @return
          */
diff --git a/shared/src/main/java/com/vaadin/shared/VBrowserDetails.java b/shared/src/main/java/com/vaadin/shared/VBrowserDetails.java
index fcdbc5bf0e..61161ea3cc 100644
--- a/shared/src/main/java/com/vaadin/shared/VBrowserDetails.java
+++ b/shared/src/main/java/com/vaadin/shared/VBrowserDetails.java
@@ -358,7 +358,7 @@ public class VBrowserDetails implements Serializable {
     }
 
     /**
-     * Tests if the browser is using the Gecko engine
+     * Tests if the browser is using the Gecko engine.
      *
      * @return true if it is Gecko, false otherwise
      */
@@ -367,7 +367,7 @@ public class VBrowserDetails implements Serializable {
     }
 
     /**
-     * Tests if the browser is using the WebKit engine
+     * Tests if the browser is using the WebKit engine.
      *
      * @return true if it is WebKit, false otherwise
      */
@@ -376,7 +376,7 @@ public class VBrowserDetails implements Serializable {
     }
 
     /**
-     * Tests if the browser is using the Presto engine
+     * Tests if the browser is using the Presto engine.
      *
      * @return true if it is Presto, false otherwise
      */
@@ -385,7 +385,7 @@ public class VBrowserDetails implements Serializable {
     }
 
     /**
-     * Tests if the browser is using the Trident engine
+     * Tests if the browser is using the Trident engine.
      *
      * @since 7.1.7
      * @return true if it is Trident, false otherwise
diff --git a/shared/src/main/java/com/vaadin/shared/communication/PushMode.java b/shared/src/main/java/com/vaadin/shared/communication/PushMode.java
index cf19c0b39e..b31726b16f 100644
--- a/shared/src/main/java/com/vaadin/shared/communication/PushMode.java
+++ b/shared/src/main/java/com/vaadin/shared/communication/PushMode.java
@@ -54,7 +54,7 @@ public enum PushMode {
     AUTOMATIC;
 
     /**
-     * Checks whether the push mode is using push functionality
+     * Checks whether the push mode is using push functionality.
      *
      * @return <code>true</code> if this mode requires push functionality;
      *         <code>false</code> if no push functionality is used for this
diff --git a/shared/src/main/java/com/vaadin/shared/communication/URLReference.java b/shared/src/main/java/com/vaadin/shared/communication/URLReference.java
index 24bde87b3b..0b993a196d 100644
--- a/shared/src/main/java/com/vaadin/shared/communication/URLReference.java
+++ b/shared/src/main/java/com/vaadin/shared/communication/URLReference.java
@@ -33,7 +33,7 @@ public class URLReference implements Serializable {
     }
 
     /**
-     * Sets the URL that this object refers to
+     * Sets the URL that this object refers to.
      *
      * @param URL
      */
diff --git a/shared/src/main/java/com/vaadin/shared/ui/AlignmentInfo.java b/shared/src/main/java/com/vaadin/shared/ui/AlignmentInfo.java
index b595e5ca1f..e2e3f98f8a 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/AlignmentInfo.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/AlignmentInfo.java
@@ -19,7 +19,7 @@ package com.vaadin.shared.ui;
 import java.io.Serializable;
 
 public final class AlignmentInfo implements Serializable {
-    /** Bitmask values for client server communication */
+    /** Bitmask values for client server communication. */
     public static class Bits implements Serializable {
         public static final int ALIGNMENT_LEFT = 1;
         public static final int ALIGNMENT_RIGHT = 2;
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ErrorLevel.java b/shared/src/main/java/com/vaadin/shared/ui/ErrorLevel.java
index 95fdf0a0d9..e3196e91ad 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ErrorLevel.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ErrorLevel.java
@@ -17,6 +17,7 @@ package com.vaadin.shared.ui;
 
 /**
  * Represents the error levels displayed on components.
+ * 
  * @author Vaadin Ltd
  * @since 8.2
  */
@@ -48,7 +49,7 @@ public enum ErrorLevel {
     SYSTEM;
 
     /**
-     * Integer representation of error severity for comparison
+     * Integer representation of error severity for comparison.
      *
      * @return integer for error severity
      */
diff --git a/shared/src/main/java/com/vaadin/shared/ui/MultiSelectMode.java b/shared/src/main/java/com/vaadin/shared/ui/MultiSelectMode.java
index f4ea23377e..54f30684d4 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/MultiSelectMode.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/MultiSelectMode.java
@@ -20,12 +20,12 @@ package com.vaadin.shared.ui;
  */
 public enum MultiSelectMode {
     /**
-     * The default behavior of the multi select mode
+     * The default behavior of the multi select mode.
      */
     DEFAULT,
 
     /**
-     * The previous more simple behavior of the multi select
+     * The previous more simple behavior of the multi select.
      */
     SIMPLE
 }
diff --git a/shared/src/main/java/com/vaadin/shared/ui/datefield/DateResolution.java b/shared/src/main/java/com/vaadin/shared/ui/datefield/DateResolution.java
index 70f552673f..4103eb9fe2 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/datefield/DateResolution.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/datefield/DateResolution.java
@@ -19,7 +19,7 @@ import java.util.ArrayList;
 import java.util.List;
 
 /**
- * Resolutions for DateFields
+ * Resolutions for DateFields.
  *
  * @author Vaadin Ltd.
  * @since 8.0
diff --git a/shared/src/main/java/com/vaadin/shared/ui/dnd/DropTargetState.java b/shared/src/main/java/com/vaadin/shared/ui/dnd/DropTargetState.java
index edd8beddbd..39224e66d8 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/dnd/DropTargetState.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/dnd/DropTargetState.java
@@ -29,12 +29,12 @@ import com.vaadin.shared.ui.dnd.criteria.Criterion;
  */
 public class DropTargetState extends SharedState {
     /**
-     * {@code DataTransfer.dropEffect} parameter for the drag event
+     * {@code DataTransfer.dropEffect} parameter for the drag event.
      */
     public DropEffect dropEffect;
 
     /**
-     * Criteria script to allow drop event on the element
+     * Criteria script to allow drop event on the element.
      */
     public String criteriaScript;
 
@@ -45,7 +45,7 @@ public class DropTargetState extends SharedState {
 
     /**
      * Declares whether any or all of the given criteria should match the
-     * payload
+     * payload.
      */
     public Criterion.Match criteriaMatch = Criterion.Match.ANY;
 }
diff --git a/shared/src/main/java/com/vaadin/shared/ui/dnd/EffectAllowed.java b/shared/src/main/java/com/vaadin/shared/ui/dnd/EffectAllowed.java
index e5afa8f5eb..5181aae9b0 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/dnd/EffectAllowed.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/dnd/EffectAllowed.java
@@ -63,7 +63,7 @@ public enum EffectAllowed {
     ALL("all"),
 
     /**
-     * Default state, equivalent to ALL
+     * Default state, equivalent to ALL.
      */
     UNINITIALIZED("uninitialized");
 
diff --git a/shared/src/main/java/com/vaadin/shared/ui/dnd/criteria/Criterion.java b/shared/src/main/java/com/vaadin/shared/ui/dnd/criteria/Criterion.java
index c90637a6e0..f5dba346e5 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/dnd/criteria/Criterion.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/dnd/criteria/Criterion.java
@@ -62,13 +62,13 @@ public class Criterion implements Serializable {
     }
 
     /**
-     * Creates a criterion object with the default comparison operator {@link
-     * ComparisonOperator#EQUALS}.
+     * Creates a criterion object with the default comparison operator
+     * {@link ComparisonOperator#EQUALS}.
      *
      * @param key
-     *         key of the payload to be compared
+     *            key of the payload to be compared
      * @param value
-     *         value of the payload to be compared
+     *            value of the payload to be compared
      */
     public Criterion(String key, String value) {
         this(key, ComparisonOperator.EQUALS, value, Payload.ValueType.STRING);
@@ -78,11 +78,11 @@ public class Criterion implements Serializable {
      * Creates a criterion object.
      *
      * @param key
-     *         key of the payload to be compared
+     *            key of the payload to be compared
      * @param operator
-     *         comparison operator
+     *            comparison operator
      * @param value
-     *         value of the payload to be compared
+     *            value of the payload to be compared
      */
     public Criterion(String key, ComparisonOperator operator, int value) {
         this(key, operator, String.valueOf(value), Payload.ValueType.INTEGER);
@@ -92,11 +92,11 @@ public class Criterion implements Serializable {
      * Creates a criterion object.
      *
      * @param key
-     *         key of the payload to be compared
+     *            key of the payload to be compared
      * @param operator
-     *         comparison operator
+     *            comparison operator
      * @param value
-     *         value of the payload to be compared
+     *            value of the payload to be compared
      */
     public Criterion(String key, ComparisonOperator operator, double value) {
         this(key, operator, String.valueOf(value), Payload.ValueType.DOUBLE);
@@ -106,13 +106,13 @@ public class Criterion implements Serializable {
      * Creates a criterion object.
      *
      * @param key
-     *         key of the payload to be compared
+     *            key of the payload to be compared
      * @param operator
-     *         comparison operator
+     *            comparison operator
      * @param value
-     *         value of the payload to be compared
+     *            value of the payload to be compared
      * @param valueType
-     *         type of the payload to be compared
+     *            type of the payload to be compared
      */
     private Criterion(String key, ComparisonOperator operator, String value,
             Payload.ValueType valueType) {
@@ -123,7 +123,7 @@ public class Criterion implements Serializable {
     }
 
     /**
-     * Gets the key of the payload to be compared
+     * Gets the key of the payload to be compared.
      *
      * @return key of the payload to be compared
      */
@@ -132,7 +132,7 @@ public class Criterion implements Serializable {
     }
 
     /**
-     * Gets the value of the payload to be compared
+     * Gets the value of the payload to be compared.
      *
      * @return value of the payload to be compared
      */
@@ -141,7 +141,7 @@ public class Criterion implements Serializable {
     }
 
     /**
-     * Gets the type of the payload value to be compared
+     * Gets the type of the payload value to be compared.
      *
      * @return type of the payload value to be compared
      */
@@ -165,15 +165,17 @@ public class Criterion implements Serializable {
      * and value type.
      *
      * @param payloadCollection
-     *         collection of payloads to compare the criterion against
+     *            collection of payloads to compare the criterion against
      * @return {@code false} if there exists a payload in the collection with
-     * the same key and value type and it doesn't match the criterion, {@code
+     *         the same key and value type and it doesn't match the criterion,
+     *         {@code
      * true} otherwise
      */
     public boolean resolve(Collection<Payload> payloadCollection) {
         Optional<Payload> payload = payloadCollection.stream()
-                .filter(p -> p.getKey().equals(key) && p.getValueType()
-                        .equals(valueType)).findAny();
+                .filter(p -> p.getKey().equals(key)
+                        && p.getValueType().equals(valueType))
+                .findAny();
 
         return payload.map(this::compareCriterionValue).orElse(true);
     }
diff --git a/shared/src/main/java/com/vaadin/shared/ui/grid/GridConstants.java b/shared/src/main/java/com/vaadin/shared/ui/grid/GridConstants.java
index a38ce4cd78..9d3a8744c7 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/grid/GridConstants.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/grid/GridConstants.java
@@ -74,28 +74,28 @@ public final class GridConstants implements Serializable {
     public static final double DEFAULT_COLUMN_WIDTH_PX = -1;
 
     /**
-     * Event ID for item click events
+     * Event ID for item click events.
      */
     public static final String ITEM_CLICK_EVENT_ID = "itemClick";
 
-    /** The default save button caption in the editor */
+    /** The default save button caption in the editor. */
     public static final String DEFAULT_SAVE_CAPTION = "Save";
 
-    /** The default cancel button caption in the editor */
+    /** The default cancel button caption in the editor. */
     public static final String DEFAULT_CANCEL_CAPTION = "Cancel";
 
     /**
-     * Event ID constant for editor open event
+     * Event ID constant for editor open event.
      */
     public static final String EDITOR_OPEN_EVENT_ID = "editorOpen";
 
     /**
-     * Event ID constant for editor move event
+     * Event ID constant for editor move event.
      */
     public static final String EDITOR_MOVE_EVENT_ID = "editorMove";
 
     /**
-     * Event ID constant for editor close event
+     * Event ID constant for editor close event.
      */
     public static final String EDITOR_CLOSE_EVENT_ID = "editorClose";
 }
diff --git a/shared/src/main/java/com/vaadin/shared/ui/grid/GridStaticCellType.java b/shared/src/main/java/com/vaadin/shared/ui/grid/GridStaticCellType.java
index 5867603912..1c2ecfd1e8 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/grid/GridStaticCellType.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/grid/GridStaticCellType.java
@@ -23,17 +23,17 @@ package com.vaadin.shared.ui.grid;
  */
 public enum GridStaticCellType {
     /**
-     * Text content
+     * Text content.
      */
     TEXT,
 
     /**
-     * HTML content
+     * HTML content.
      */
     HTML,
 
     /**
-     * Widget content
+     * Widget content.
      */
     WIDGET;
 }
diff --git a/shared/src/main/java/com/vaadin/shared/ui/loginform/LoginFormConstants.java b/shared/src/main/java/com/vaadin/shared/ui/loginform/LoginFormConstants.java
index 0efa948f52..6679df30e5 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/loginform/LoginFormConstants.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/loginform/LoginFormConstants.java
@@ -18,7 +18,7 @@ package com.vaadin.shared.ui.loginform;
 import java.io.Serializable;
 
 /**
- * Constants for LoginForm
+ * Constants for LoginForm.
  *
  * @since 7.7
  * @author Vaadin Ltd
diff --git a/shared/src/main/java/com/vaadin/shared/ui/progressindicator/ProgressBarState.java b/shared/src/main/java/com/vaadin/shared/ui/progressindicator/ProgressBarState.java
index 13e78b9b2d..cdc6a2bcf0 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/progressindicator/ProgressBarState.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/progressindicator/ProgressBarState.java
@@ -21,7 +21,7 @@ import com.vaadin.shared.annotations.NoLayout;
 import com.vaadin.shared.communication.SharedState;
 
 /**
- * {@link SharedState} for {@link com.vaadin.ui.ProgressBar}
+ * {@link SharedState} for {@link com.vaadin.ui.ProgressBar}.
  *
  * @since 7.1
  * @author Vaadin Ltd
diff --git a/shared/src/main/java/com/vaadin/shared/ui/tabsheet/TabsheetState.java b/shared/src/main/java/com/vaadin/shared/ui/tabsheet/TabsheetState.java
index 2755f48b43..3af0da9b42 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/tabsheet/TabsheetState.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/tabsheet/TabsheetState.java
@@ -38,10 +38,10 @@ public class TabsheetState extends AbstractComponentContainerState {
 
     public List<TabState> tabs = new ArrayList<>();
 
-    /** true to show the tab bar, false to only show the contained component */
+    /** true to show the tab bar, false to only show the contained component. */
     public boolean tabsVisible = true;
 
-    /** the key of the currently selected tab */
+    /** the key of the currently selected tab. */
     public String selected;
 
     @DelegateToWidget
diff --git a/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusParentRpc.java b/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusParentRpc.java
index 333fc84d35..9f45ba7428 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusParentRpc.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusParentRpc.java
@@ -26,7 +26,7 @@ import com.vaadin.shared.communication.ServerRpc;
 public interface FocusParentRpc extends ServerRpc {
 
     /**
-     * Focuses cell in the row parent
+     * Focuses cell in the row parent.
      *
      * @param rowKey
      *            the row key
diff --git a/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusRpc.java b/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusRpc.java
index fa02a0d712..5fc89e8152 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusRpc.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/treegrid/FocusRpc.java
@@ -27,7 +27,7 @@ import com.vaadin.shared.communication.ClientRpc;
 public interface FocusRpc extends ClientRpc {
 
     /**
-     * Focuses a cell
+     * Focuses a cell.
      *
      * @param rowIndex
      *            the row index
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ui/DebugWindowServerRpc.java b/shared/src/main/java/com/vaadin/shared/ui/ui/DebugWindowServerRpc.java
index 3a3b04091c..760c7b1abc 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ui/DebugWindowServerRpc.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ui/DebugWindowServerRpc.java
@@ -38,7 +38,7 @@ public interface DebugWindowServerRpc extends ServerRpc {
     public void showServerDebugInfo(Connector connector);
 
     /**
-     * Invokes the layout analyzer on the server
+     * Invokes the layout analyzer on the server.
      *
      * @since 7.1
      */
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ui/Transport.java b/shared/src/main/java/com/vaadin/shared/ui/ui/Transport.java
index 3e0aea6f1a..c8bd19ef12 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ui/Transport.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ui/Transport.java
@@ -17,31 +17,31 @@
 package com.vaadin.shared.ui.ui;
 
 /**
- * Transport modes for Push
+ * Transport modes for Push.
  *
  * @since 7.1
  * @author Vaadin Ltd
  */
 public enum Transport {
     /**
-     * Websockets
+     * Websockets.
      */
     WEBSOCKET("websocket"),
     /**
-     * Websockets for server to client, XHR for client to server
+     * Websockets for server to client, XHR for client to server.
      *
      * @since 7.6
      */
     WEBSOCKET_XHR("websocket-xhr"),
     /**
-     * HTTP streaming
+     * HTTP streaming.
      *
      * @deprecated Use the more reliable {@link Transport#LONG_POLLING} instead.
      */
     @Deprecated
     STREAMING("streaming"),
     /**
-     * HTTP long polling
+     * HTTP long polling.
      */
     LONG_POLLING("long-polling");
 
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ui/UIClientRpc.java b/shared/src/main/java/com/vaadin/shared/ui/ui/UIClientRpc.java
index 1b9a31ac6e..30e51234fd 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ui/UIClientRpc.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ui/UIClientRpc.java
@@ -18,7 +18,7 @@ package com.vaadin.shared.ui.ui;
 import com.vaadin.shared.communication.ClientRpc;
 
 /**
- * Server to Client RPC methods for UI
+ * Server to Client RPC methods for UI.
  *
  * @since 7.1
  * @author Vaadin Ltd
@@ -26,7 +26,7 @@ import com.vaadin.shared.communication.ClientRpc;
 public interface UIClientRpc extends ClientRpc {
 
     /**
-     * Informs the client that the UI has been closed
+     * Informs the client that the UI has been closed.
      *
      * @param sessionExpired
      *            true if the ui was closed because the session expired, false
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ui/UIConstants.java b/shared/src/main/java/com/vaadin/shared/ui/ui/UIConstants.java
index 54da337189..ba76f765b3 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ui/UIConstants.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ui/UIConstants.java
@@ -31,7 +31,7 @@ public class UIConstants implements Serializable {
     public static final String ATTRIBUTE_REPLACE_STATE = "rs";
 
     /**
-     * Name of the parameter used to transmit UI ids back and forth
+     * Name of the parameter used to transmit UI ids back and forth.
      */
     public static final String UI_ID_PARAMETER = "v-uiId";
 
diff --git a/shared/src/main/java/com/vaadin/shared/ui/ui/UIState.java b/shared/src/main/java/com/vaadin/shared/ui/ui/UIState.java
index a86d79da05..182363f849 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/ui/UIState.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/ui/UIState.java
@@ -70,7 +70,7 @@ public class UIState extends AbstractSingleComponentContainerState {
      */
     public LocaleServiceState localeServiceState = new LocaleServiceState();
     /**
-     * Configuration for the push channel
+     * Configuration for the push channel.
      */
     public PushConfigurationState pushConfiguration = new PushConfigurationState();
     /**
diff --git a/shared/src/main/java/com/vaadin/shared/ui/upload/UploadState.java b/shared/src/main/java/com/vaadin/shared/ui/upload/UploadState.java
index 5245905441..3fb4e32828 100644
--- a/shared/src/main/java/com/vaadin/shared/ui/upload/UploadState.java
+++ b/shared/src/main/java/com/vaadin/shared/ui/upload/UploadState.java
@@ -24,7 +24,7 @@ import com.vaadin.shared.AbstractComponentState;
  */
 public class UploadState extends AbstractComponentState {
 
-    /** Is the upload component in immediate mode or not */
+    /** Is the upload component in immediate mode or not. */
     public boolean immediateMode = true;
 
     {
diff --git a/shared/src/main/java/com/vaadin/shared/util/SharedUtil.java b/shared/src/main/java/com/vaadin/shared/util/SharedUtil.java
index 4f1b71687b..2cab2fd5b5 100644
--- a/shared/src/main/java/com/vaadin/shared/util/SharedUtil.java
+++ b/shared/src/main/java/com/vaadin/shared/util/SharedUtil.java
@@ -173,7 +173,7 @@ public class SharedUtil implements Serializable {
 
     /**
      * Capitalizes the first character in the given string in a way suitable for
-     * use in code (methods, properties etc)
+     * use in code (methods, properties etc).
      *
      * @since 7.4
      * @param string
@@ -230,8 +230,8 @@ public class SharedUtil implements Serializable {
      * @param uri
      *            The uri to which the parameters should be added.
      * @param extraParams
-     *            One or more parameters in the format "a=b" or "c=d&amp;e=f". An
-     *            empty string is allowed but will not modify the url.
+     *            One or more parameters in the format "a=b" or "c=d&amp;e=f".
+     *            An empty string is allowed but will not modify the url.
      * @return The modified URI with the get parameters in extraParams added.
      */
     public static String addGetParameters(String uri, String extraParams) {