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

fix(scrollbar): remove hijack custom style #315

Merged
merged 4 commits into from
Jan 20, 2022
Merged

fix(scrollbar): remove hijack custom style #315

merged 4 commits into from
Jan 20, 2022

Conversation

danilowoz
Copy link
Member

Removes the custom style for scrollbars and relies on the system configuration to whether shows the scrollbar or not.

@vercel vercel bot temporarily deployed to Preview – sandpack-docs January 20, 2022 16:12 Inactive
@github-actions
Copy link

github-actions bot commented Jan 20, 2022

Size changes

sandpack-react

Total base Total current +/-
828 kb 827 kb 🎉 -252 bytes (-0.03%)
Details
Dependency name / file Base Current +/-
index.js 22 kb 22 kb ⚠️ +5 bytes (+0.02%)
types.js 54 bytes 54 bytes ✅ 0 byte
./sandpack-react/dist/index.css 9 kb 8 kb 🎉 -257 bytes (-2.94%)
@code-hike/classer 550 bytes 550 bytes ✅ 0 byte
@codemirror/closebrackets 54 kb 54 kb ✅ 0 byte
@codemirror/commands 62 kb 62 kb ✅ 0 byte
@codemirror/comment 12 kb 12 kb ✅ 0 byte
@codemirror/gutter 50 kb 50 kb ✅ 0 byte
@codemirror/highlight 59 kb 59 kb ✅ 0 byte
@codemirror/history 50 kb 50 kb ✅ 0 byte
@codemirror/lang-css 71 kb 71 kb ✅ 0 byte
@codemirror/lang-html 110 kb 110 kb ✅ 0 byte
@codemirror/lang-javascript 92 kb 92 kb ✅ 0 byte
@codemirror/language 58 kb 58 kb ✅ 0 byte
@codemirror/matchbrackets 54 kb 54 kb ✅ 0 byte
@codemirror/state 12 kb 12 kb ✅ 0 byte
@codemirror/view 54 kb 54 kb ✅ 0 byte
@codesandbox/sandpack-client 7 kb 7 kb ✅ 0 byte
@react-hook/intersection-observer 4 kb 4 kb ✅ 0 byte
codesandbox-import-utils 128 bytes 128 bytes ✅ 0 byte
react-devtools-inline 49 kb 49 kb ✅ 0 byte
react-is 824 bytes 824 bytes ✅ 0 byte

sandpack-client

Total base Total current +/-
10 kb 10 kb ✅ 0 byte
Details
Dependency name / file Base Current +/-
client.js 4 kb 4 kb ✅ 0 byte
file-resolver-protocol.js 777 bytes 777 bytes ✅ 0 byte
iframe-protocol.js 623 bytes 623 bytes ✅ 0 byte
index.js 4 kb 4 kb ✅ 0 byte
types.js 54 bytes 54 bytes ✅ 0 byte
utils.js 1 kb 1 kb ✅ 0 byte

@vercel vercel bot temporarily deployed to Preview – sandpack-theme January 20, 2022 16:15 Inactive
@vercel vercel bot temporarily deployed to Preview – sandpack-landing January 20, 2022 16:15 Inactive
@vercel vercel bot temporarily deployed to Preview – sandpack-theme January 20, 2022 16:17 Inactive
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 91b45bd:

Sandbox Source
Sandpack Blog Example 1 Configuration

@vercel vercel bot temporarily deployed to Preview – sandpack-landing January 20, 2022 16:19 Inactive
@vercel vercel bot temporarily deployed to Preview – sandpack-docs January 20, 2022 16:19 Inactive
@danilowoz danilowoz merged commit 5781e97 into main Jan 20, 2022
@danilowoz danilowoz deleted the fix/scrollbar branch January 20, 2022 16:27
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