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

M2-7745/fixing paragraph Text scroll on Android #857

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

felipeMetaLab
Copy link
Contributor

📝 Description

🔗 Jira Ticket M2-7745
Allowing user scroll downward in Android.
I am adding a extra padding always keyboard opens to allow user scroll down.

📸 Screenshots

IOS (didn't change):
https://github.com/user-attachments/assets/e7f7a74d-b8e4-4f50-b510-f86a49c33192

ANDROID:
https://github.com/user-attachments/assets/3733e49f-f7b3-4717-873f-f235bb3a4631

Testing Android with long description:
https://github.com/user-attachments/assets/e723d713-0eae-47aa-baf5-73b46f8bd73c

Result:
Screenshot 2024-09-10 at 5 00 25 PM

🪤 Peer Testing

1 - Build application on Android
2 - Create a ParagraphText item and add it to an applet
3 - Once you open Paragraph text, check if when keyboard is open you can scroll down or not.

✏️ Notes

on IOS there should be no changes

@felipeMetaLab
Copy link
Contributor Author

fixing the tests.

src/shared/ui/survey/ParagraphText.tsx Outdated Show resolved Hide resolved
src/shared/ui/survey/ParagraphText.tsx Outdated Show resolved Hide resolved
src/shared/ui/survey/ParagraphText.tsx Outdated Show resolved Hide resolved
src/shared/ui/survey/ParagraphText.tsx Outdated Show resolved Hide resolved
@ChaconC ChaconC changed the base branch from release/2.2.2 to release/2.2.3 September 12, 2024 17:26
…lataform.OS to IS_ANDROID and removing * 0.8 from setKeyboardHeight
Base automatically changed from release/2.2.3 to dev September 19, 2024 23:40
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.

2 participants