Skip to content

Add textFieldContentPadding modifier for compact text fields#470

Open
IT-Guy007 wants to merge 2 commits into
skiptools:mainfrom
MollisWare:feature/textfield-content-padding
Open

Add textFieldContentPadding modifier for compact text fields#470
IT-Guy007 wants to merge 2 commits into
skiptools:mainfrom
MollisWare:feature/textfield-content-padding

Conversation

@IT-Guy007

Copy link
Copy Markdown

Adds a textFieldContentPadding(_:) View modifier (plus a bridged variant) that overrides the OutlinedTextField content padding. When set, TextField renders via BasicTextField + OutlinedTextFieldDefaults.DecorationBox so the padding — and therefore the field height — is honored; the default OutlinedTextField rendering path is unchanged.

Thank you for contributing to the Skip project! Please review the contribution guide at https://skip.dev/docs/contributing/ for advice and guidance on making high-quality PRs.

Use this space to describe your change and add any labels (bug, enhancement, documentation, etc.) to help categorize your contribution.

Skip Pull Request Checklist:

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device
  • REQUIRED: I have checked whether this change requires a corresponding update in the Skip Fuse UI repository (link related PR if applicable)
  • OPTIONAL: I have added an example of any UI changes to the Showcase sample app

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes.

skip-fuse-ai will also get a pr

Adds a textFieldContentPadding(_:) View modifier (plus a bridged variant)
that overrides the OutlinedTextField content padding. When set, TextField
renders via BasicTextField + OutlinedTextFieldDefaults.DecorationBox so the
padding — and therefore the field height — is honored; the default
OutlinedTextField rendering path is unchanged.
@cla-bot cla-bot Bot added the cla-signed label Jun 22, 2026
@IT-Guy007

Copy link
Copy Markdown
Author

@marcprux ready for review

@marcprux

Copy link
Copy Markdown
Member

Do you happen to have a screenshot of how this looks?

@IT-Guy007

Copy link
Copy Markdown
Author

I do not, I tested it locally with skip-fuse-ui the default padding was gone

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.

2 participants