Simplify creation of new screens #1525
Conversation
This also changes from an enum key to a string key so that we can switch to rememberSaveable soon.
| if (content == null) { | ||
| Text("No screen found with key '$screenKey'!") |
There was a problem hiding this comment.
I thought about automatically falling back to home, but if we get rememberSaveable, you rename a key, and the app reloads, I want what happened to be slightly more clear.
|
We get nice screen names now because they're strings.
|
| class TestContext( | ||
| val httpClient: HttpClient, | ||
| ) |
There was a problem hiding this comment.
This will become a context receiver eliminating the need to explicitly pass any values in the lambdas above.
|
@JakeWharton I appreciate your effort in doing this task. I tried to implement #1518 yesterday, but it did not work as expected. I realized that the reason was the enum screen now. |
|
Yeah a string should work much better with |
This also changes from an enum key to a string key so that we can switch to rememberSaveable soon (#1518).