UI hot reload

[PGimenez]

One of the reasons why the page is not reloading when changing the UI code is that it is in an anonymous function. In this example, after editing the code in ui and uianon while the app is running, only the pages /page /file and /route are refreshed with the new content.

using GenieFramework
@genietools

@app begin
    @out var1 = 0
end

function ui()
    # StippleUI.layout(["aaa"])
    "<h1>test</h1>"
end

uianon = () -> "<h1>test</h1>"

@page("/page", ui)
@page("/file", "ui.jl")
route("/route") do
    model = @init
    page(model, ui) |> html
end
@page("/pageanon", uianon)
route("/routeanon") do
    model = @init
    page(model, uianon) |> html
end

However, changing the uidefinition in ui.jlto a standard function does not fix the reload issue. What does fix it is placing the entire uifunction inside app.jl, all in one file. Therefore, I guess the issue lies in how the code is imported via include.

Ideally we'd use @app("/", "ui.jl"), but we can't yet as there's an error related to q-layout(issue here). We'll look into it.

[ms10596]

Thanks a lot for looking into those issues

1. For the first one, Yes I think you're right it's related to the way "includes" works. I tried to make a new macro that could initialize a model before registering the view based on the current @page macro but the model was not configured correctly however.

macro modelpage(url, view)
  global model = Model |> init |> handlers
  :(@page($url, $view, Stipple.ReactiveTools.DEFAULT_LAYOUT(), $model)) |> esc
end

2. For the second one hhaensel workaround is working with me, I wish I had known it earlier 😄

[PGimenez]

@essenciary fixed the Layout issue in this commit GenieFramework/StippleUI.jl#107 , and confirmed that the hotreload issue is due to using include. Included code is not reloadable by Revise

However, we still can't use @page because of mixins. The use of mixins requires a named model, which in turn requires using route so that we can initialize the model in it. I'm trying to get mixins working with both anonymous and named models so that we can use @pageand solve the hot reload thing. Also, this would make the code a little simpler

As a temporary workaround, you can do using Revise; includet("ui.jl") to enable hot reloading.

[PGimenez]

Fixed in #3