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
Add Gutters
property to control padding on applicable inputs
#8857
Comments
@henon @ScarletKuro Good idea? May want to roll this into an overhaul of the margin and density system altogether #8844. I believe @Garderoben and @mckaragoz were interested in this. |
Thanks for wanting to do a PR, @danielchalmers ! We try to merge all non-breaking bugfixes and will deliberate the value of new features for the community. Please note there is no guarantee your pull request will be merged, so if you want to be sure before investing the work, feel free to contact the team first. |
It would be ok to add |
I believe to have one |
Closing in favor of unified approach in #8946 - can reopen in the future if it doesn't work out |
Feature request type
Enhance component
Component name
No response
Is your feature request related to a problem?
The padding on many components are too much for certain designs.
Describe the solution you'd like
A
Gutters
property that istrue
by default will remove all padding iffalse
.This property should exist in as many inputs as possible.
The current way that the gutters are done is that the component adds the padding and then removes it if the property is false. This should be the other way around as mentioned in #8586 and could become part of the implementation of resolving this issue.
Note: Need to think about how this impacts labels.
Have you seen this feature anywhere else?
A few other components have
Gutters
: https://github.com/search?q=repo%3AMudBlazor%2FMudBlazor%20gutters&type=codeDescribe alternatives you've considered
I need the components to scale nicely with font size and be more dense, so currently I will remove the padding entirely or replace it with relative units. This ends up looking something like this:
Pull Request
Code of Conduct
The text was updated successfully, but these errors were encountered: