refactor: move action buttons styles to css

Author: paodb

src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java

@@ -66,6 +66,7 @@
 
 @SuppressWarnings("serial")
 @CssImport(value = "./styles/multiselect-cb-hide.css", themeFor = "vaadin-grid")
+@CssImport(value = "./styles/twin-col-grid-button.css")
 public class TwinColGrid<T> extends VerticalLayout
     implements HasValue<ValueChangeEvent<Set<T>>, Set<T>>, HasComponents, HasSize {
 
@@ -115,13 +116,13 @@ public enum Orientation {
   /** @deprecated Use getRightGrid().getDataProvider() */
   @Deprecated protected ListDataProvider<T> rightGridDataProvider;
 
-  private final Button addAllButton = new Button();
+  private final Button addAllButton = createActionButton();
 
-  private final Button addButton = new Button();
+  private final Button addButton = createActionButton();
 
-  private final Button removeButton = new Button();
+  private final Button removeButton = createActionButton();
 
-  private final Button removeAllButton = new Button();
+  private final Button removeAllButton = createActionButton();
 
   private Component buttonContainer;
 
@@ -203,7 +204,7 @@ public TwinColGrid(final ListDataProvider<T> dataProvider, String caption) {
 
   /**
    * Sets orientation for TwinColGridComponent
-   * 
+   *
    * @param orientation
    * @return
    */
@@ -254,13 +255,9 @@ private VerticalLayout createVerticalContainer() {
 
   private VerticalLayout getVerticalButtonContainer() {
     addButton.setIcon(VaadinIcon.ANGLE_RIGHT.create());
-    addButton.setWidth("3em");
     addAllButton.setIcon(VaadinIcon.ANGLE_DOUBLE_RIGHT.create());
-    addAllButton.setWidth("3em");
     removeButton.setIcon(VaadinIcon.ANGLE_LEFT.create());
-    removeButton.setWidth("3em");
     removeAllButton.setIcon(VaadinIcon.ANGLE_DOUBLE_LEFT.create());
-    removeAllButton.setWidth("3em");
 
     VerticalLayout vButtonContainer =
         new VerticalLayout(
@@ -273,13 +270,9 @@ private VerticalLayout getVerticalButtonContainer() {
 
   private HorizontalLayout getHorizontalButtonContainer() {
     addButton.setIcon(VaadinIcon.ANGLE_DOWN.create());
-    addButton.setWidth("3em");
     addAllButton.setIcon(VaadinIcon.ANGLE_DOUBLE_DOWN.create());
-    addAllButton.setWidth("3em");
     removeButton.setIcon(VaadinIcon.ANGLE_UP.create());
-    removeButton.setWidth("3em");
     removeAllButton.setIcon(VaadinIcon.ANGLE_DOUBLE_UP.create());
-    removeAllButton.setWidth("3em");
 
     HorizontalLayout hButtonContainer =
         new HorizontalLayout(
@@ -766,4 +759,11 @@ public HasValue<? extends ValueChangeEvent<List<T>>, List<T>> asList() {
   public Set<T> getEmptyValue() {
     return Collections.emptySet();
   }
+
+  private Button createActionButton() {
+    Button button = new Button();
+    button.addThemeName("twin-col-grid-button");
+    return button;
+  }
+  
 }

src/main/resources/META-INF/resources/frontend/styles/twin-col-grid-button.css

@@ -0,0 +1,23 @@
+/*-
+ * #%L
+ * TwinColGrid add-on
+ * %%
+ * Copyright (C) 2017 - 2021 Flowing Code
+ * %%
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * #L%
+ */
+
+vaadin-button[theme~="twin-col-grid-button"] {
+	width: 3em;
+}