refactor!: rename SplitterDragendEvent and addSplitterDragendListener (#8112)

web-padawan · web-flow · commit bd2e9c49c720 · 2025-10-02T15:39:59.000+03:00
diff --git a/vaadin-split-layout-flow-parent/vaadin-split-layout-flow-integration-tests/src/main/java/com/vaadin/flow/component/splitlayout/test/SplitLayoutView.java b/vaadin-split-layout-flow-parent/vaadin-split-layout-flow-integration-tests/src/main/java/com/vaadin/flow/component/splitlayout/test/SplitLayoutView.java
@@ -69,7 +69,7 @@ private void addResizeNotificationLayout() {
 
         Span message = new Span("Drag and drop the splitter");
         AtomicInteger resizeCounter = new AtomicInteger();
-        layout.addSplitterDragendListener(
+        layout.addSplitterDragEndListener(
                 event -> message.setText("SplitLayout Resized "
                         + resizeCounter.incrementAndGet() + " times."));
 
diff --git a/vaadin-split-layout-flow-parent/vaadin-split-layout-flow-integration-tests/src/main/java/com/vaadin/flow/component/splitlayout/test/SplitterPositionView.java b/vaadin-split-layout-flow-parent/vaadin-split-layout-flow-integration-tests/src/main/java/com/vaadin/flow/component/splitlayout/test/SplitterPositionView.java
@@ -60,7 +60,7 @@ private void createLayoutJavaApi() {
 
         Span showSplitterPosition = new Span();
         showSplitterPosition.setId("showSplitterPositionJavaApi");
-        layout.addSplitterDragendListener(e -> showSplitterPosition
+        layout.addSplitterDragEndListener(e -> showSplitterPosition
                 .setText(String.valueOf(e.getSource().getSplitterPosition())));
         add(setSplitterPosition, layout, showSplitterPosition);
     }
@@ -83,7 +83,7 @@ private void createLayoutElementApi() {
 
         Span showSplitterPosition = new Span();
         showSplitterPosition.setId("showSplitterPositionElementApi");
-        layout.addSplitterDragendListener(e -> showSplitterPosition
+        layout.addSplitterDragEndListener(e -> showSplitterPosition
                 .setText(String.valueOf(e.getSource().getSplitterPosition())));
         add(setSplitterPosition, layout, showSplitterPosition);
     }
diff --git a/vaadin-split-layout-flow-parent/vaadin-split-layout-flow/src/main/java/com/vaadin/flow/component/splitlayout/SplitLayout.java b/vaadin-split-layout-flow-parent/vaadin-split-layout-flow/src/main/java/com/vaadin/flow/component/splitlayout/SplitLayout.java
@@ -78,7 +78,7 @@ public SplitLayout(Orientation orientation) {
         setOrientation(orientation);
         addAttachListener(
                 e -> this.requestStylesUpdatesForSplitterPosition(e.getUI()));
-        addSplitterDragendListener(e -> {
+        addSplitterDragEndListener(e -> {
             splitterPosition = calcNewSplitterPosition(
                     e.primaryComponentFlexBasis, e.secondaryComponentFlexBasis);
 
@@ -210,7 +210,7 @@ public Component getSecondaryComponent() {
      * 0 and 100. The value will be null unless the splitter position has been
      * explicitly set on the server-side, or the splitter has been moved on the
      * client side. The splitter position is automatically updated when as part
-     * of the {@link SplitterDragendEvent}.
+     * of the {@link SplitterDragEndEvent}.
      *
      * @return the splitter position, may be null
      */
@@ -339,7 +339,7 @@ public void removeAll() {
     }
 
     @DomEvent("splitter-dragend")
-    public static class SplitterDragendEvent
+    public static class SplitterDragEndEvent
             extends ComponentEvent<SplitLayout> {
 
         private static final String PRIMARY_FLEX_BASIS = "element.querySelector(':scope > [slot=\"primary\"]').style.flexBasis";
@@ -348,7 +348,7 @@ public static class SplitterDragendEvent
         String primaryComponentFlexBasis;
         String secondaryComponentFlexBasis;
 
-        public SplitterDragendEvent(SplitLayout source, boolean fromClient,
+        public SplitterDragEndEvent(SplitLayout source, boolean fromClient,
                 @EventData(PRIMARY_FLEX_BASIS) String primaryComponentFlexBasis,
                 @EventData(SECONDARY_FLEX_BASIS) String secondaryComponentFlexBasis) {
             super(source, fromClient);
@@ -366,9 +366,9 @@ public SplitterDragendEvent(SplitLayout source, boolean fromClient,
      *            the listener to add
      * @return a registration for removing the listener
      */
-    public Registration addSplitterDragendListener(
-            ComponentEventListener<SplitterDragendEvent> listener) {
-        return addListener(SplitterDragendEvent.class,
+    public Registration addSplitterDragEndListener(
+            ComponentEventListener<SplitterDragEndEvent> listener) {
+        return addListener(SplitterDragEndEvent.class,
                 (ComponentEventListener) listener);
     }
 
diff --git a/vaadin-split-layout-flow-parent/vaadin-split-layout-flow/src/test/java/com/vaadin/flow/component/splitlayout/tests/SplitLayoutUnitTest.java b/vaadin-split-layout-flow-parent/vaadin-split-layout-flow/src/test/java/com/vaadin/flow/component/splitlayout/tests/SplitLayoutUnitTest.java
@@ -26,7 +26,7 @@
 import com.vaadin.flow.component.html.Div;
 import com.vaadin.flow.component.html.Span;
 import com.vaadin.flow.component.splitlayout.SplitLayout;
-import com.vaadin.flow.component.splitlayout.SplitLayout.SplitterDragendEvent;
+import com.vaadin.flow.component.splitlayout.SplitLayout.SplitterDragEndEvent;
 
 public class SplitLayoutUnitTest {
 
@@ -156,7 +156,7 @@ public void testUpdateSplitterPosition_dragEndEvent_widthInPixels() {
         double splitterPosition = 45.66;
         splitLayout.setSplitterPosition(splitterPosition);
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "432.68px", "267.32px"));
 
         Assert.assertEquals(61.81, splitLayout.getSplitterPosition(), 0.01);
@@ -168,7 +168,7 @@ public void testUpdateSplitterPosition_dragEndEvent_widthInPercentage() {
         double splitterPosition = 45.66;
         splitLayout.setSplitterPosition(splitterPosition);
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "58.23%", "41.77%"));
 
         Assert.assertEquals(58.23, splitLayout.getSplitterPosition(), 0);
@@ -189,7 +189,7 @@ public void constructorWithOrientation() {
         double splitterPosition = 45.66;
         splitLayout.setSplitterPosition(splitterPosition);
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "58.23%", "41.77%"));
 
         Assert.assertEquals(58.23, splitLayout.getSplitterPosition(), 0);
@@ -206,7 +206,7 @@ public void constructorWithComponents() {
         double splitterPosition = 45.66;
         splitLayout.setSplitterPosition(splitterPosition);
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "58.23%", "41.77%"));
 
         Assert.assertEquals(58.23, splitLayout.getSplitterPosition(), 0);
@@ -227,7 +227,7 @@ public void constructorWithComponentsAndOrientation() {
         double splitterPosition = 45.66;
         splitLayout.setSplitterPosition(splitterPosition);
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "58.23%", "41.77%"));
 
         Assert.assertEquals(58.23, splitLayout.getSplitterPosition(), 0);
@@ -246,7 +246,7 @@ public void testUpdateSplitterPosition_dragEndEvent_primaryWidthNull() {
         splitLayout.setSplitterPosition(splitterPosition);
 
         ComponentUtil.fireEvent(splitLayout,
-                new SplitterDragendEvent(splitLayout, true, null, "41.77%"));
+                new SplitterDragEndEvent(splitLayout, true, null, "41.77%"));
 
         Assert.assertEquals(45.66, splitLayout.getSplitterPosition(), 0);
     }
@@ -258,7 +258,7 @@ public void testUpdateSplitterPosition_dragEndEvent_secondaryWidthNull() {
         splitLayout.setSplitterPosition(splitterPosition);
 
         ComponentUtil.fireEvent(splitLayout,
-                new SplitterDragendEvent(splitLayout, true, "41.77%", null));
+                new SplitterDragEndEvent(splitLayout, true, "41.77%", null));
 
         Assert.assertEquals(45.66, splitLayout.getSplitterPosition(), 0);
     }
@@ -267,7 +267,7 @@ public void testUpdateSplitterPosition_dragEndEvent_secondaryWidthNull() {
     public void testUpdateSplitterPosition_noInitialPosition() {
         SplitLayout splitLayout = new SplitLayout();
 
-        ComponentUtil.fireEvent(splitLayout, new SplitterDragendEvent(
+        ComponentUtil.fireEvent(splitLayout, new SplitterDragEndEvent(
                 splitLayout, true, "432.68px", "267.32px"));
 
         Assert.assertEquals(61.81, splitLayout.getSplitterPosition(), 0.01);
