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(theme): avoid layout shift caused by scrollbar #1844

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zqianem
Copy link
Contributor

@zqianem zqianem commented Jan 26, 2023

fixes #1054

Same as #1568 which was mistakenly closed by me.

@github-actions github-actions bot added the stale label Nov 6, 2023
@github-actions github-actions bot removed the stale label Nov 25, 2023
@github-actions github-actions bot added the stale label Feb 2, 2024
@github-actions github-actions bot removed the stale label Apr 28, 2024
zqianem added a commit to zqianem/vitepress-python-editor that referenced this pull request May 11, 2024
@zqianem
Copy link
Contributor Author

zqianem commented May 27, 2024

I recently applied this patch (along with others) to the latest version of VitePress (1.2.2) for a project of mine and can confirm the fix still works well.

@brc-dd I am still interested in getting this PR merged — is there anything else I can do to help that process along? The only potentially problematic thing I can see for this patch is the use of overflow-x: hidden on body; I can try to find an alternative to that if so desired.

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.

Could it be better to adjust width of some html element ?
2 participants