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.

Sign up for GitHub

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

Jump to bottom

Modals add padding to page's body element #412

Closed
brycehewett opened this issue Nov 23, 2020 · 3 comments
Closed

Modals add padding to page's body element #412

brycehewett opened this issue Nov 23, 2020 · 3 comments
Labels
enhancement New feature or request Ready

Comments

@brycehewett
Copy link

When opening a modal BlazorStrap is adding what seems like unnecessary padding to the page's body element.

Screen Shot 2020-11-23 at 4 17 48 PM
@jbomhold3
Copy link
Collaborator

Oldest of old features in the library. It's assumed there is a scroll bar. Thats why your getting padding as thats the default bootstrap behavior. @chanan You want to tackle this one? We do need to add a check to see if the padding is even required.

@jbomhold3 jbomhold3 added enhancement New feature or request Ready labels Dec 1, 2020
@jbomhold3
Copy link
Collaborator

Added: Padding-right is no longer added when scrollbar is not present.

@jbomhold3
Copy link
Collaborator

Ready and will be in next release closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request Ready
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brycehewett @jbomhold3