Skip to content

[beta] fix console type #4677

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

Merged
merged 1 commit into from
May 24, 2022
Merged

[beta] fix console type #4677

merged 1 commit into from
May 24, 2022

Conversation

danilowoz
Copy link
Contributor

@danilowoz danilowoz commented May 23, 2022

To be merged on #4672
It fixes the console message type from the Sandpack client.

@github-actions
Copy link

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

🎉 Global Bundle Size Decreased

Page Size (compressed)
global 95.33 KB (-1 B)
Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

@harish-sethuraman harish-sethuraman merged commit 66eeacb into reactjs:try-sandpack-console May 24, 2022
@harish-sethuraman
Copy link
Collaborator

Thanks :)

@danilowoz danilowoz deleted the try-sandpack-console__fix-types branch May 24, 2022 08:26
gaearon added a commit that referenced this pull request Jun 10, 2022
* added code for sandpack console

* add log

* added console for older bundle

* Revert "[beta] Sandpack - new bundler (#4458)"

This reverts commit 3ab1245.

* adds proper console and removes new bundle

* modify styles

* remove unwanted code

* nit

* fix types (#4677)

* update console

* little nits

* remove unwanted code changes

* update bundler URL

* use `message.firstLoad` for clearing console

* use `refresh` event to clear logs as well (used when going away and coming back to sandpack)

* remove padding for code blocks inside console

* small UI revamps

* change p to div since the sandpack comes inside the p, add try catch and a try catch for the catch again

* tweaks

* Fixes

* Reset unrelated changes

* tweaks

* fix

* fixes

* oops

* Fix

* fix

Co-authored-by: Danilo Woznica <[email protected]>
Co-authored-by: Dan Abramov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants