• Using flow instead of StateFlow
• Exposing a suspendable getter that retrieves the last emitted element
• Instead of having a flow as default value, using a single element

Adding entries to the dsl. Usage example:

    private fun demoTweakGraph() = tweaksGraph {
        tweakCategory("Screen 1") {
            tweakGroup("Group 1") {
                label(
                    key = "timestamp",
                    name = "Current timestamp",
                ) {
                    flow {
                        while (true) {
                            emit("${System.currentTimeMillis() / 1000}")
                            delay(1000)
                        }
                    }
                }
                editableString(
                    key = "value1",
                    name = "Value 1",
                )
                editableBoolean(
                    key = "value2",
                    name = "Value 2",
                )
                editableInt(
                    key = "value3",
                    name = "Value 3",
                    defaultValue = flow {
                        while (true) {
                            counter += 1
                            emit(counter)
                            delay(1000)
                        }
                    }
                )
                editableLong(
                    key = "value4",
                    name = "Value 4",
                    defaultValue = 0L,
                )

                button(
                    key = "button1",
                    name = "Demo button"
                ) {
                    Toast.makeText(this@TweakDemoApplication, "Demo button", Toast.LENGTH_LONG)
                        .show()
                }
            }
        }
    }

This is a proposal of a DSL to create Tweak graphs. It is intended to be used as follows:

tweakGraph {
    tweakCategory("Some category Name") {
        tweakGroup("Some gtoup name") {
            addEntry(SomeEntryTweak())
            ....
        }
    }
}

I've kept the public constructor so the graph can still be built using previous syntax.

I will update the README.md if this approach is accepted 🙂

Description

There was an undesired effect in the demo app that was making the counter example to have a strange behavior. I'm using the remember function on TweakEntry composables so state is resued between recompositions to avoid having to subscribe to a flow again and again:

I'm taking EditableIntTweakEntryBody as example:

• value is of type State<T> and its accessor is used in this same Composable function scope
• Compose compiler makes every Composable function to be recomposed when an object of type State that is observed within its scope is updated
• Each time this Composable is recomposed it is being subscribed to a cold flow that will start emitting its events from the beggining
• To avoid creating that flow subscription again and again, we should use remember to keep the reference to that State so it is reused across recompositions

Result:

https://user-images.githubusercontent.com/18336786/137685416-9f14e1d6-f672-4090-a62f-62a3650a78e1.mov

Duplicate class com.gmerino.tweaks.BuildConfig found in modules jetified-tweaks-0.0.2-runtime (io.github.gmerinojimenez:tweaks:0.0.2) and jetified-tweaks-no-op-0.0.2-runtime (io.github.gmerinojimenez:tweaks-no-op:0.0.2)