-
Notifications
You must be signed in to change notification settings - Fork 25
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
feat: add "Log view" #17
base: main
Are you sure you want to change the base?
Conversation
This PR isn't just to add a feature about "Log View", however it's about allowing to execute code, which is misleading. |
Thank you for your comment!
Should I stop executing code in the playground (in terms of security risks, etc.)? |
I noticed your comment. |
You're right. Using There's a safer way to achieve this, but it's out of scope, because it's still under ECMAScript proposal (Stage 3). I'm not sure whether we really need this feature or not, however it's nice to have. We don't need to close this PR. Let's leave it and see how others says. If it's highly requested, we can consider it. |
Thank you very much for your detailed explanation.
Ok! I agree with this approach. |
Yes, it's ShadowRealm. |
Description
Add "Log view" at side bar.(like typescript playground)