Skip to content

Nested control flow#164

Draft
moodymudskipper wants to merge 4 commits intomasterfrom
f-162-nested-control-flow
Draft

Nested control flow#164
moodymudskipper wants to merge 4 commits intomasterfrom
f-162-nested-control-flow

Conversation

@moodymudskipper
Copy link
Copy Markdown
Owner

For #162

This works ok on given example and bquote but :

  • tweak background color for a better look
  • clean up flow_data, the logic should live in another function
  • check how this affects flow_run etc
  • the left right arrow doesn't always look nice/is not visible enough in png output, so we should explore emojis or alternatives
  • I think we do need special casing for functions for the display to be nicer, use function() for anonymous functions.
  • implement recursive boxes
  • the way we implement data$nodes is not satisfying and will likely break the recursive boxes, should find a better way

Rplot01

Rplot

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.

1 participant