Skip to content
New issue

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

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

Already on GitHub? # to your account

Hangman #64

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Hangman #64

wants to merge 9 commits into from

Conversation

mitiemann
Copy link

A couple of simple games should attract (young) gamers. Let's see whether I can pull this off.

@fonsp
Copy link
Member

fonsp commented Jul 23, 2024

Hey @mitiemann !

Yayyy happy that you are joining us! Some quick feedback on your draft:

  • we should definitely publish the turtle code in a package! I will do that in 2 weeks (when i'm back from the bike trip), you should copy the code until then
  • You can use PlutoUI.confirm for the button
  • when this gets published on featured.plutojl.org, we will use https://github.com/JuliaPluto/PlutoSliderServer.jl to make it interactive on the web.
    Usually, this works seamlessly, but your notebook contains one pattern that PSS does not support: Warn when one bond defines another PlutoSliderServer.jl#138
    The solution is that random_word_box should just be a string, without UI element. That means replacing the code with this:
random_word_box="Julia
Notebook
Pluto
Hangman
Programmer
Feature
Bug
Gamer
Developer"

Then people can only edit the list when editing the notebook, but PSS will still work

@fonsp
Copy link
Member

fonsp commented Jul 23, 2024

You could also use HypertextLiteral.jl to style text if you want to make things bigger or add color.

@mitiemann
Copy link
Author

Thanks for your feedback :-)

we should definitely publish the turtle code in a package! I will do that in 2 weeks (when i'm back from the bike trip), you should copy the code until then

No hurries. Maybe I can even race you to the PR. Gamification for the win! ;-)

You can use PlutoUI.confirm for the button

Thanks. But doesn't this introduce the forbidden "bind chain"-pattern again?

Btw.: having stateful objects in a reactive notebook feels ... weird. E.g., how would you even define cell order with stateful cells? Is this something that y'all have already had discussions about?

You could also use HypertextLiteral.jl to style text if you want to make things bigger or add color.

Thanks, will try. So far: bugs.

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

Successfully merging this pull request may close these issues.

2 participants