Skip to content

feat: add scrollEnabled prop - #226

Merged
exploIF merged 9 commits into
software-mansion:mainfrom
IvanIhnatsiuk:feat/add-scrollEnabled-prop
Nov 18, 2025
Merged

exploIF merged 9 commits into
software-mansion:mainfrom
IvanIhnatsiuk:feat/add-scrollEnabled-prop

Conversation

@IvanIhnatsiuk

Copy link
Copy Markdown
Contributor

Description

The scrollEnabled option has been added to prohibit scrolling of content inside a text field. In some cases, this can be useful (as in our application) when we want to show multiple EnrichedTextInput and a single ScrollView.

Screenshots

Android:

Screen.Recording.2025-11-05.at.23.12.19.mov

iOS:

Screen.Recording.2025-11-05.at.23.11.56.mov

Comment thread ios/EnrichedTextInputView.mm Outdated

@exploIF exploIF left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@IvanIhnatsiuk, thanks for the contribution. It looks good to me. If you can fix those two tiny comments, I will merge it and we can include it in the upcoming release

Comment thread android/src/main/java/com/swmansion/enriched/EnrichedTextInputView.kt Outdated
@IvanIhnatsiuk
IvanIhnatsiuk force-pushed the feat/add-scrollEnabled-prop branch from 2b24f00 to 1710177 Compare November 7, 2025 12:19
Comment thread ios/inputParser/InputParser.mm Outdated
Comment thread ios/EnrichedTextInputView.mm Outdated
Comment thread ios/inputParser/InputParser.mm Outdated
Co-authored-by: Tomasz Zawadzki <tomekzawadzki98@gmail.com>
@exploIF

exploIF commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

Hi @IvanIhnatsiuk, if you could remove unnecessary parser changes, we can ship this feature this week

@exploIF exploIF left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you 🔥

@exploIF
exploIF merged commit 8fc6472 into software-mansion:main Nov 18, 2025
5 checks passed
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.

4 participants