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: Removes padding from fluent-field host element #32002

Merged
merged 6 commits into from
Jul 17, 2024

Conversation

eljefe223
Copy link
Contributor

Previous Behavior

The fluent-field had padding baked in which goes against the component libraries principles and created awkward styling challenges for the end user.

New Behavior

Remove any paddings or margins from the :host element

Related Issue(s)

  • Fixes #

@eljefe223 eljefe223 requested a review from a team as a code owner July 15, 2024 18:05
@fabricteam
Copy link
Collaborator

fabricteam commented Jul 15, 2024

📊 Bundle size report

✅ No changes found

@eljefe223 eljefe223 force-pushed the users/jes/remove-field-padding branch from 094c6d1 to 5dfdff9 Compare July 16, 2024 04:04
@eljefe223 eljefe223 force-pushed the users/jes/remove-field-padding branch from 584125a to 8bef175 Compare July 16, 2024 16:26
@eljefe223 eljefe223 force-pushed the users/jes/remove-field-padding branch from 24d4a15 to c320ae6 Compare July 17, 2024 20:56
@eljefe223 eljefe223 merged commit be10f2c into microsoft:master Jul 17, 2024
16 of 18 checks passed
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.

6 participants