From 77b701d375cdd3629304bed9895186c824235e0d Mon Sep 17 00:00:00 2001
From: "daniel.watson" <daniel.watson@bioclinica.com>
Date: Mon, 12 Jun 2023 20:14:48 -0400
Subject: [PATCH 1/2] implemented STACKED orientation

---
 .../taefi/component/ToggleButtonGroup.java    | 24 ++++++++++++++++---
 .../addons-styles/toggle-button-group.css     |  8 +++++++
 .../component/ToggleButtonGroupView.java      | 12 +++++++++-
 3 files changed, 40 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/vaadin/addons/taefi/component/ToggleButtonGroup.java b/src/main/java/org/vaadin/addons/taefi/component/ToggleButtonGroup.java
index 557d0ac..0c0bc4e 100644
--- a/src/main/java/org/vaadin/addons/taefi/component/ToggleButtonGroup.java
+++ b/src/main/java/org/vaadin/addons/taefi/component/ToggleButtonGroup.java
@@ -24,7 +24,7 @@
 public class ToggleButtonGroup<T> extends CustomField<T> {
 
     public enum Orientation {
-        HORIZONTAL, VERTICAL
+        HORIZONTAL, VERTICAL, STACKED
     }
 
     private List<T> items;
@@ -95,6 +95,10 @@ private void init() {
         }
 
         addButtonsToLayout(buttons);
+        
+        if(orientation == Orientation.STACKED) {
+            setValue(items.get(0), false);
+        }
     }
 
     protected void addButtonsToLayout(Button[] buttons) {
@@ -159,7 +163,15 @@ protected Optional<String> getLastButtonClass() {
     }
 
     private String getOrientationStylePostfix() {
-        return orientation == Orientation.HORIZONTAL ? "h" : "v";
+        switch(orientation) {
+        case HORIZONTAL:
+            return "h";
+        case STACKED:
+            return "s";
+        case VERTICAL:
+            return "v";
+        default: throw new IllegalArgumentException();
+        }
     }
 
     private Comparator<T> getDefaultComparator() {
@@ -186,7 +198,13 @@ protected void buttonsActionListener(ClickEvent<Button> event) {
             Serializable selectedValueId = itemIdGenerator.apply(selectedValue);
             Serializable previousValueId = itemIdGenerator.apply(getValue());
             if (Objects.equals(selectedValueId, previousValueId)) {
-                setValue(null, event.isFromClient());
+                if(orientation == Orientation.STACKED) {
+                    int itemIndex = items.indexOf(selectedValue)+1;
+                    T nextItem = items.get(itemIndex == items.size() ? 0 : itemIndex);
+                    setValue(nextItem, event.isFromClient());
+                }
+                else setValue(null, event.isFromClient());
+                
                 return;
             }
         }
diff --git a/src/main/resources/META-INF/resources/frontend/addons-styles/toggle-button-group.css b/src/main/resources/META-INF/resources/frontend/addons-styles/toggle-button-group.css
index 580ff02..09e7e76 100644
--- a/src/main/resources/META-INF/resources/frontend/addons-styles/toggle-button-group.css
+++ b/src/main/resources/META-INF/resources/frontend/addons-styles/toggle-button-group.css
@@ -3,6 +3,14 @@
 
 }
 
+.toggle-button-group-button-s:not([theme~=primary]) {
+	display:none;
+}
+
+.toggle-button-group-button-s[theme~=primary] {
+	display: unset;
+}
+
 .toggle-button-group-button-h {
     border-style: solid;
     border-color: var(--_lumo-button-primary-background-color, var(--lumo-primary-color));
diff --git a/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java b/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
index 225dc01..08621b7 100644
--- a/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
+++ b/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
@@ -226,9 +226,19 @@ public ToggleButtonGroupView() {
             case RIGHT -> VaadinIcon.ALIGN_RIGHT.create();
         });
         group110.setItemLabelGenerator(textAlignment -> "");
+        
+        ToggleButtonGroup<TextAlignment> group1101 = new ToggleButtonGroup<>("Alignment:");
+        group1101.setItems(TextAlignment.values());
+        group1101.setItemIconGenerator(align -> switch (align) {
+            case LEFT -> VaadinIcon.ALIGN_LEFT.create();
+            case CENTER -> VaadinIcon.ALIGN_CENTER.create();
+            case RIGHT -> VaadinIcon.ALIGN_RIGHT.create();
+        });
+        group1101.setItemLabelGenerator(textAlignment -> "");
+        group100.setOrientation(ToggleButtonGroup.Orientation.STACKED);
 
 
-        VerticalLayout halfLayout = new VerticalLayout(line10, line15, group20, group30, line40, line50, group60, group70, group80, line90, group100, group110);
+        VerticalLayout halfLayout = new VerticalLayout(line10, line15, group20, group30, line40, line50, group60, group70, group80, line90, group100, group110,group1101);
         halfLayout.setId("parent-layout");
         halfLayout.getStyle().set("width", "50%");
         halfLayout.getStyle().set("border", "solid red 1px");

From 5bcb0eaa0b458b8f6467e93fe7bfa8f21fbf8047 Mon Sep 17 00:00:00 2001
From: "daniel.watson" <daniel.watson@bioclinica.com>
Date: Mon, 12 Jun 2023 20:45:37 -0400
Subject: [PATCH 2/2] corrected demo example

---
 .../component/ToggleButtonGroupView.java      | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java b/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
index 08621b7..d91a1df 100644
--- a/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
+++ b/src/test/java/org/vaadin/addons/taefi/component/ToggleButtonGroupView.java
@@ -217,6 +217,8 @@ public ToggleButtonGroupView() {
         );
         group100.setId("group100");
         group100.setOrientation(ToggleButtonGroup.Orientation.VERTICAL);
+        
+        
 
         ToggleButtonGroup<TextAlignment> group110 = new ToggleButtonGroup<>("Alignment:");
         group110.setItems(TextAlignment.values());
@@ -227,18 +229,16 @@ public ToggleButtonGroupView() {
         });
         group110.setItemLabelGenerator(textAlignment -> "");
         
-        ToggleButtonGroup<TextAlignment> group1101 = new ToggleButtonGroup<>("Alignment:");
-        group1101.setItems(TextAlignment.values());
-        group1101.setItemIconGenerator(align -> switch (align) {
-            case LEFT -> VaadinIcon.ALIGN_LEFT.create();
-            case CENTER -> VaadinIcon.ALIGN_CENTER.create();
-            case RIGHT -> VaadinIcon.ALIGN_RIGHT.create();
-        });
-        group1101.setItemLabelGenerator(textAlignment -> "");
-        group100.setOrientation(ToggleButtonGroup.Orientation.STACKED);
-
+        ToggleButtonGroup<Status> group120 = new ToggleButtonGroup<>(
+                "Status: [orientation = Stacked]",
+                Status.values()
+        );
+        group120.setId("group120");
+        group120.setOrientation(ToggleButtonGroup.Orientation.STACKED);
+        group120.setValue(Status.APPROVED);
+        
 
-        VerticalLayout halfLayout = new VerticalLayout(line10, line15, group20, group30, line40, line50, group60, group70, group80, line90, group100, group110,group1101);
+        VerticalLayout halfLayout = new VerticalLayout(line10, line15, group20, group30, line40, line50, group60, group70, group80, line90, group100, group110,group120);
         halfLayout.setId("parent-layout");
         halfLayout.getStyle().set("width", "50%");
         halfLayout.getStyle().set("border", "solid red 1px");