From 87d6c7c87234458e3b0fbe87df1c017e82e72938 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Helmut=20H=C3=A4nsel?= <helmut.haensel@gmx.de>
Date: Tue, 7 Nov 2023 23:45:22 +0100
Subject: [PATCH 1/2] alias qmenu, more general syntax, fix docstring

---
 src/Menus.jl | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/src/Menus.jl b/src/Menus.jl
index 4faae735..73a404c9 100644
--- a/src/Menus.jl
+++ b/src/Menus.jl
@@ -3,12 +3,12 @@ module Menus
 using Genie, Stipple, StippleUI, StippleUI.API
 import Genie.Renderer.Html: HTMLString, normal_element, register_normal_element
 
-export menu
+export qmenu
 
 register_normal_element("q__menu", context = @__MODULE__)
 
 """
-      menu(fieldname::Union{Symbol,Nothing} = nothing, args...; content::Union{String,Vector} = "", kwargs...)
+      menu(fieldname::Union{Symbol,Nothing}, args...; content::Union{String,Vector} = "", kwargs...)
 
 The `menu` component is a convenient way to show menus. Goes very well with `list` as dropdown content, but it’s by no means limited to it.   
 
@@ -19,7 +19,7 @@ The `menu` component is a convenient way to show menus. Goes very well with `lis
 ### View
 
 ```julia-repl
-julia> btn("Basic Menu", color="primary", [StippleUI.menu( [p("Hello")])
+julia> btn("Basic Menu", color="primary", [StippleUI.menu(nothing, [p("Hello")])])
 ```
 
 -----------
@@ -46,7 +46,7 @@ julia> btn("Basic Menu", color="primary", [StippleUI.menu( [p("Hello")])
       * `maxwidth::String` - The maximum width of the menu; Size in CSS units, including unit name ex. `16px` `2rem`
 """
 function menu(
-              fieldname::Union{Symbol,Nothing} = nothing,
+              fieldname::Symbol,
               args...;
               content::Union{String,Vector} = "",
               kwargs...)
@@ -56,11 +56,22 @@ function menu(
   end
 end
 
+function menu(
+      args...;
+      content::Union{String,Vector} = "",
+      kwargs...)
+
+  q__menu(args...; kw(kwargs)...) do
+    join(content)
+  end
+end
+
 function menu(content::Function,
-              fieldname::Union{Symbol,Nothing} = nothing,
               args...;
               kwargs...)
-  menu(label, fieldname, args...; content = content(), kwargs...)
+  menu(args...; content = content(), kwargs...)
 end
 
+const qmenu = menu
+
 end

From 650cf47e1fa06b874f0ad2f0fa7aaa1d452a86c8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Helmut=20H=C3=A4nsel?= <helmut.haensel@gmx.de>
Date: Wed, 8 Nov 2023 00:15:50 +0100
Subject: [PATCH 2/2] handle Bool values for val keyword correctly

---
 src/Radios.jl | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/Radios.jl b/src/Radios.jl
index 9785015b..620ec004 100644
--- a/src/Radios.jl
+++ b/src/Radios.jl
@@ -41,7 +41,10 @@ julia> radio("Polygon", :shape, val="polygon")
       * `keep-color::Bool` - Should the color (if specified any) be kept when checkbox is unticked?
 2. General
       * `tabindex::Union{Int, String}` - Tabindex HTML attribute value
+3. Model
+      * `fieldname::Symbol` - Name of the variable to bind to.
 4. Label
+      * `label::AbstractString` - Label
       * `leftlabel::Bool` - Label (if any specified) should be displayed on the left side of the checkbox
 5. State
       * `disable::Bool` - Put component in disabled mode
@@ -54,8 +57,10 @@ julia> radio("Polygon", :shape, val="polygon")
 function radio( label::AbstractString = "",
                 fieldname::Union{Symbol,Nothing} = nothing,
                 args...;
+                val = nothing,
                 kwargs...)
-  q__radio(args...; kw([:label => label, :fieldname => fieldname, kwargs...])...)
+  kw_val = val === nothing ? Dict() : Dict(:val => (val isa Bool ? js_attr(val) : val))
+  q__radio(args...; kw([:label => label, :fieldname => fieldname, kw_val..., kwargs...])...)
 end
 
 end