[TIMOB-25086] Android: Textarea scroll in tableview

GitHub Issue	n/a
Type	Bug
Priority	High
Status	Closed
Resolution	Fixed
Resolution Date	2017-09-21T21:10:25.000+0000
Affected Version/s	Release 6.1.2
Fix Version/s	Release 6.2.2
Components	Android
Labels	n/a
Reporter	Ricardo Ramirez
Assignee	Joshua Quick
Created	2017-08-11T14:58:48.000+0000
Updated	2017-11-02T19:10:51.000+0000

Description

Issue Description

In a tableviewrow, if a textarea is defined with a fixed height in Android, user cannot scroll the textarea to view the entire text entered. In iOS the scroll works fine.

Steps to reproduce

Download the sample code

Replace the app folder in a new project

Run the project

Scroll on the table view rows

Attachments

File	Date	Size
app.zip	2017-08-11T14:58:45.000+0000	10462625
ScrollViewNestingTest.js	2017-09-05T20:35:50.000+0000	4211
ScrollViewTextAreasTest.js	2017-09-05T20:35:50.000+0000	1503
TableOfTextAreasTest.js	2017-09-05T20:35:50.000+0000	1124

Comments

Lokesh Choudhary 2017-08-17

Verified the issue is reproducible with android with latest 6.2.0 SDK. Studio Ver: 4.9.1.201707200100 SDK Ver: 6.2.0.v20170816173122 OS Ver: 10.12.3 Xcode Ver: Xcode 8.3.3 Appc NPM: 4.2.9 Appc CLI: 6.2.3 Ti CLI Ver: 5.0.14 Alloy Ver: 1.9.13 Node Ver: 6.10.1 Java Ver: 1.8.0_101 Devices: ⇨ google Nexus 5 --- Android 6.0.1 ⇨ google Pixel --- Android 7.1.1 IOS Device: Iphone 7 IOS 11 beta4
Eric Merriman 2017-08-23

We have been looking at this since Monday. It's a bit of a technical challenge so will respond back soon with scheduling.
Gary Mathews 2017-08-28

master: https://github.com/appcelerator/titanium_mobile/pull/9361
Joshua Quick 2017-09-05

The following implements nested scrolling support for TextAreas and ScrollViews on *+all+* Android OS versions Titanium supports. PR (master): https://github.com/appcelerator/titanium_mobile/pull/9402
Gary Mathews 2017-09-14

6_2_X: https://github.com/appcelerator/titanium_mobile/pull/9433
Abir Mukherjee 2017-09-14

FR Passed. PR merged.
Lokesh Choudhary 2017-09-18

Verified the fix with SDK 6.2.1.v20170915135328. Closing. Studio Ver: 4.9.1.201707200100 SDK Ver: 6.2.1.v20170915135328 OS Ver: 10.12.3 Xcode Ver: Xcode 8.3.3 Appc NPM: 4.2.9 Appc CLI: 6.2.4 Ti CLI Ver: 5.0.14 Alloy Ver: 1.9.14 Node Ver: 7.10.1 Java Ver: 1.8.0_101 Devices: ⇨ google Pixel --- Android 7.1.1 ⇨ google Nexus 5 --- Android 6.0.1 Emulator: Android 4.4.2
Eric Merriman 2017-09-19

Reopening to change versions

Michael Gangolf 2017-11-02

sorry to add to this old ticket but I think it broke selecting a textfield with right alignment.

   var win = Ti.UI.createWindow({
   	backgroundColor: '#fff',
   	layout: "vertical"
   });
   
   var tf1 = Ti.UI.createTextField({
   	width: Ti.UI.FILL,
   	top: 10,
   	height: 40,
   	borderColor:"#000",
   	borderWidth:1,
   	bottom: 10
   });
   var tf2 = Ti.UI.createTextField({
   	width: Ti.UI.FILL,
   	top: 10,
   	height: 40,
   	borderColor:"#000",
   	borderWidth:1,
   	bottom: 10,
   	textAlign:Ti.UI.TEXT_ALIGNMENT_RIGHT
   });
   var tf3 = Ti.UI.createTextField({
   	width: Ti.UI.FILL,
   	top: 10,
   	height: 40,
   	borderColor:"#000",
   	borderWidth:1,
   	bottom: 10,
   	textAlign:Ti.UI.TEXT_ALIGNMENT_RIGHT
   });
   
   win.add(tf1);
   win.add(tf2);
   win.add(tf3);
   win.open();

I've tracked it down to 6.2.1.GA working and 6.2.2.GA+ not working. Since this was 6.2.2.GA had only 3 changes this should be the one that caused the problem. Device HTC A9. First textfield can be selected with a click inside the textfield (no matter where) but the next two with right alignment are very difficult to select at all with every SDK >=6.2.2.GA

Joshua Quick 2017-11-02

[~michael], I just tested it out. I'm able to reproduce this issue on an Android 7.1 Pixel XL with right-aligned TextFields, but not left-aligned or center-aligned TextFields. Also, I'm NOT able to reproduce this issue with right-aligned TextFields on an Android 5.0 Moto G or Android 4.1 Galaxy Nexus. Hmm... this requires further investigation. [~michael], would you mind writing this up as a separate bug report please? Thanks.
Michael Gangolf 2017-11-02

JSON Source