[TIMOB-25415] Android: remove method for scrollview does not remove the view
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | High |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2017-10-23T22:31:13.000+0000 |
Affected Version/s | Release 6.3.0 |
Fix Version/s | Release 6.3.0 |
Components | Android |
Labels | android, qe-6.3.0, scrollview |
Reporter | Lokesh Choudhary |
Assignee | Joshua Quick |
Created | 2017-10-19T20:55:50.000+0000 |
Updated | 2017-10-26T22:10:43.000+0000 |
Description
*This is a regression. Not seen in SDK 6.2.2.GA.*
Steps to reproduce:
1. Use the code below in your app.js:
var win = Ti.UI.createWindow();
var scrollView = Ti.UI.createScrollView({
width: 400,
height: 400,
contentWidth: "auto",
contentHeight: "auto",
backgroundColor: 'red'
});
var view = Ti.UI.createView({
backgroundColor: 'green',
borderColor: 'gray',
borderWidth: 4,
height: 100,
width: 100
});
view.addEventListener('click', function(_event) {
scrollView.remove(_event.source);
});
scrollView.add(view);
win.add(scrollView);
win.open();
2. Build for android device/emulator.
3. After app launch tap on the green view.
Actual results:
1. Clicking on the view removes just the background color & not the border.Expected results:
1. Clicking on the view should remove the view completely.Attachments
File | Date | Size |
---|---|---|
ScrollViewAddRemoveTest.js | 2017-10-19T22:46:29.000+0000 | 1848 |
Attached script [^ScrollViewAddRemoveTest.js] to better test add(), insertTo(), and remove() functionality.
PR (6.3.x): https://github.com/appcelerator/titanium_mobile/pull/9543 PR (master): https://github.com/appcelerator/titanium_mobile/pull/9545
FR Passed for both PR's. Master merged, waiting for merge to get enabled for 6.3.0.
PR for backport merged.
Verified the fix in SDK 6.3.0.v20171026120401 & 7.0.0.v20171025141436. Closing. Studio Ver: 4.10.0.201709271713 OS Ver: 10.12.3 Xcode Ver: Xcode 8.3.3 Appc NPM: 4.2.10-2 Appc CLI: 6.3.0-master.15 Ti CLI Ver: 5.0.14 Alloy Ver: 1.10.6 Node Ver: 7.10.1 Java Ver: 1.8.0_101 Devices: ⇨ google Nexus 5 — Android 6.0.1 ⇨ google Pixel — Android 7.1.1