feat: Implement HasStyle in LitTemplate so you can add class names to…

… the root element (#10903) (#11017) Makes it possible to use addClassName("hello") in the constructor. Class for the custom element cannot be added in the template. Co-authored-by: Mikhail Shabarov <[email protected]>
vaadin · May 18, 2021 · a83bb49 · a83bb49
1 parent ed88c32
commit a83bb49
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/flow-lit-template/src/main/java/com/vaadin/flow/component/littemplate/LitTemplate.java b/flow-lit-template/src/main/java/com/vaadin/flow/component/littemplate/LitTemplate.java
@@ -19,6 +19,7 @@
 
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.HasComponents;
+import com.vaadin.flow.component.HasStyle;
 import com.vaadin.flow.component.Tag;
 import com.vaadin.flow.component.dependency.JsModule;
 import com.vaadin.flow.component.littemplate.LitTemplateParser.LitTemplateParserFactory;
@@ -55,7 +56,7 @@
  * @author Vaadin Ltd
  * @since
  */
-public abstract class LitTemplate extends Component {
+public abstract class LitTemplate extends Component implements HasStyle {
 
     static {
         UsageStatistics.markAsUsed("flow/LitTemplate", null);

diff --git a/flow-lit-template/src/test/java/com/vaadin/flow/component/littemplate/LitTemplateTest.java b/flow-lit-template/src/test/java/com/vaadin/flow/component/littemplate/LitTemplateTest.java
@@ -102,6 +102,15 @@ public ElementWithTextLitTemplate(VaadinService service) {
 
     }
 
+    // This is for checking LitTemplate implements HasStyle
+    @Tag("custom-element-with-class")
+    private static class ElementWithStyleClass extends LitTemplate {
+        ElementWithStyleClass() {
+            // Should be possible to modify the class name
+            addClassName("custom-element-class-name");
+        }
+    }
+
     @Before
     public void setUp() {
         DeploymentConfiguration configuration = Mockito