Skip to content

[RN78][public] TextField - no default preset shown in the screen#3946

Merged
M-i-k-e-l merged 2 commits into
v9from
fix/textfield-default-preset
Mar 9, 2026
Merged

[RN78][public] TextField - no default preset shown in the screen#3946
M-i-k-e-l merged 2 commits into
v9from
fix/textfield-default-preset

Conversation

@MIchaelZablianov

@MIchaelZablianov MIchaelZablianov commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Description

After investigating the issue - 4965, it came out that the issue is only in the demo screen, due to how Switch (SegmentedControl) is implemented.

  • Set the default TextField preset in the demo screen from undefined to TextField.presets.UNDERLINE so the demo launches with the underline style by default.

Changelog

  • TextField - Set default preset to UNDERLINE in demo screen

Additional info

ticket 4965

@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

✅ PR Description Validation Passed

All required sections are properly filled out:

  • Description
  • Changelog
  • Additional info

Your PR is good for review! 🚀


This validation ensures all sections from the PR template are properly filled.

@MIchaelZablianov MIchaelZablianov requested review from M-i-k-e-l and removed request for lidord-wix March 9, 2026 11:47
@MIchaelZablianov MIchaelZablianov changed the title [TextField] - Set default preset to UNDERLINE in demo screen [RN78][public] TextField - no default preset shown in the screen Mar 9, 2026

@M-i-k-e-l M-i-k-e-l left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Looking at this again I found this line:

TextField.defaultProps = {preset: TextField.presets.UNDERLINE, ...TextField.defaultProps};

Which when removed gives the same outcome in master.
So this solution seems valid, please remove the relevant line(s).

Also, please remove the [, ] in the changelog

@M-i-k-e-l M-i-k-e-l merged commit 52e3971 into v9 Mar 9, 2026
3 checks passed
@M-i-k-e-l M-i-k-e-l deleted the fix/textfield-default-preset branch March 9, 2026 14:52
M-i-k-e-l added a commit that referenced this pull request May 17, 2026
* RN 0.78 - basic upgrade

* Infra/v9 codemod (#3899)

* npx codemod@latest react/19/migration-recipe

* npx types-react-codemod@latest preset-19 .

* Infra/v9 prop types (#3900)

* Remove DemoScreen (unused)

* Remove prop-types

* Remove @types/prop-types

* Infra/v9 default props (#3901)

* Remove unneeded defaultProps

* defaultProps on asBaseComponent

* Remove defaultProps (unused af far as we can tell)

* Remove DocsGenerator (unused)

* Fix (most) typescript issue (#3902)

* Fix (most) typescript issue

* Infra/v9 tests (#3903)

* Fix button and update @testing-library/react-native version

* Remove unsupported modifiers methods, add docs and fix tests

The old TextField was the only usage of the removed methods and it was only used by MaskedInput which is deprecated

* V9 - fixes (#3904)

* npx codemod@latest react/19/migration-recipe

* npx types-react-codemod@latest preset-19 .

* Remove DemoScreen (unused)

* Remove prop-types

* Remove @types/prop-types

* Remove unneeded defaultProps

* defaultProps on asBaseComponent

* Remove defaultProps (unused af far as we can tell)

* Remove DocsGenerator (unused)

* Fix (most) typescript issue

* Fix button and update @testing-library/react-native version

* Remove unsupported modifiers methods, add docs and fix tests

The old TextField was the only usage of the removed methods and it was only used by MaskedInput which is deprecated

* KeyboardTrackingView - support defaultProps

* Bump uilib-native version

* uilib-native - fix npm package of uilib-native

* Update project.pbxproj

* Add check for do not merge

* Remove (will be added in master)

* Fix types

* Checkbox - fix label's type

* Update uilib-native to snapshot

* "Force" defaultProps on forwardRef

* Add uilib-native to the monorepo (#3923)

* Monorepo - lib to uilib-native

* Fix releaseNative script

* More build fixes

* Try adding --workspace

* Revert (Try adding --workspace)

* Infra/monorepo release script (#3931)

* Monorepo - lib to uilib-native

* Fix releaseNative script

* More build fixes

* Try adding --workspace

* Revert (Try adding --workspace)

* Unified release script for monorepo

* Fix dryRun always being true

* buildKiteVersion is only for release

* Delete packages/uilib-native/scripts/.npmrc

* Add Picker to the doc (dialog migration)

* [RN78][public] TextField - no default preset shown in the screen (#3946)

* fix: set default TextField preset to UNDERLINE in demo screen

Made-with: Cursor

* applied comments

* Demo's configurations - remove unused TextField import

* Marquee - move to Incubator

* Marquee - move to Incubator - docs

* Update Jest configuration to use setupFilesAfterEnv instead of setupFiles

* Update version to 9

---------

Co-authored-by: MIchaelZablianov <michaelza@wix.com>
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.

3 participants