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

Allow interacting with stdin #995

Merged
merged 19 commits into from
Nov 8, 2023
Merged

Allow interacting with stdin #995

merged 19 commits into from
Nov 8, 2023

Conversation

shepmaster
Copy link
Member

No description provided.

We will need the sequence number in order to connect an active
execution with subsequent stdin data.
We will return this to allow sending stdin data without worrying about
maintaining the multiplexed ID ourselves.
Previously, we always added the menu immediately after the triggering
button. This can mess with CSS based on first/last child.
This will allow us to more easily reuse them for stdin
interactions. As a bonus, the HTML structure feels much cleaner.
This has some *minor* styling changes that make hover and click states
a wee bit more noticeable.
The previous border would create an issue when the arrow was attached
to the bottom of the pop up. This replaces the border with a dark
shadow.
This allows executing programs that read until output is complete.
@shepmaster shepmaster added the enhancement Something new the playground could do label Nov 6, 2023
@shepmaster shepmaster merged commit 103d561 into main Nov 8, 2023
@shepmaster shepmaster deleted the stdin branch November 8, 2023 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Something new the playground could do
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant