Merge pull request #203 from BeastyBlacksmith/bbs/index-by-id

add index by id

Author: etpinard

.github/workflows/label.yml

@@ -7,6 +7,6 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/labeler@v2
+    - uses: actions/labeler@v4
       with:
         repo-token: "${{ secrets.GITHUB_TOKEN }}"

src/app/dashapp.jl

@@ -40,6 +40,28 @@ mutable struct DashApp
 
 end
 
+const VecChildTypes = Union{NTuple{N, DashBase.Component} where {N}, Vector{<:DashBase.Component}}
+
+function Base.getindex(component::DashBase.Component, id::AbstractString)
+  component.id == id && return component
+  hasproperty(component, :children) || return nothing
+  cc = component.children
+  return if cc isa Union{VecChildTypes, DashBase.Component}
+        cc[id]
+    elseif cc isa AbstractVector
+        identity.(filter(x->hasproperty(x, :id), cc))[id]
+    else
+        nothing
+    end
+end
+function Base.getindex(children::VecChildTypes, id::AbstractString)
+  for element in children
+    element.id == id && return element
+    el = element[id]
+    el !== nothing && return el
+  end
+end
+
 #only name, index_string and layout are available to set
 function Base.setproperty!(app::DashApp, property::Symbol, value)
     property == :index_string && return set_index_string!(app, value)

test/core.jl

@@ -200,3 +200,18 @@ end
     end
     @test_throws ErrorException make_handler(app)
 end
+
+@testset "Index by id" begin
+    app = dash()
+    app.layout = html_div() do
+            dcc_input(id = "my-id", value="initial value", type = "text"),
+            html_div(id = "my-div", children = [
+                html_div(),
+                "string",
+                html_div(id = "target")
+            ]),
+            html_div(id = "my-div2")
+        end
+    @test app.layout["target"].id == "target"
+    @test app.layout["ups"] === nothing
+end