Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revamp Explorer section for Kino #879

Merged
merged 3 commits into from
Jan 17, 2022
Merged

Revamp Explorer section for Kino #879

merged 3 commits into from
Jan 17, 2022

Conversation

josevalim
Copy link
Contributor

The following changes are included:

  1. Talk about Mermaid in the introductory chapter
  2. Remove Kino from the introductory portal guide
  3. Add missing examples to introduction to Kino
  4. Move fun with VM introspection under the Kino guides
  5. Rewrite fun with VM introspection to connect to itself, no longer needing a remote node
  6. Add a guide on chat apps

I haven't reviewed the other two new Kino guides (pong + custom widgets). I plan to do that next, potentially in a separate PR as this one gets merged.

cookie_input = Kino.Input.text("Cookie")
```
node_input = Kino.Input.text("Node", default: node())
cookie_input = Kino.Input.text("Cookie", default: Node.get_cookie())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fantastic idea with the default!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Courtesy of kino powered inputs! 💯

in our inbox:

```elixir
Process.info(self(), :messages)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, the pong notebook uses IEx.Helpers.flush(), but this is much better.

Copy link
Member

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful!! :shipit:

@josevalim josevalim merged commit 4d79706 into main Jan 17, 2022
@josevalim
Copy link
Contributor Author

💚 💙 💜 💛 ❤️

@josevalim josevalim deleted the jv-05-notebooks branch January 17, 2022 16:14
Comment on lines -198 to -208
inputs to pass different values to our code. To render inputs, we need to
install the [Kino](https://github.com/livebook-dev/kino) library:
the different tuples:

```elixir
Mix.install(
[
{:kino, github: "livebook-dev/kino"}
],
consolidate_protocols: false
)
```
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This portal livebook now doesn't mention how to install kino anymore, which cause errors when it is used during the "Notebook connections" section

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@esambo it's been addressed in #953 and should be good on Livebook main :)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Never mind, this has already been fixed in main with #953

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants