Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-25415] Android: remove method for scrollview does not remove the view

GitHub Issuen/a
TypeBug
PriorityHigh
StatusClosed
ResolutionFixed
Resolution Date2017-10-23T22:31:13.000+0000
Affected Version/sRelease 6.3.0
Fix Version/sRelease 6.3.0
ComponentsAndroid
Labelsandroid, qe-6.3.0, scrollview
ReporterLokesh Choudhary
AssigneeJoshua Quick
Created2017-10-19T20:55:50.000+0000
Updated2017-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

FileDateSize
ScrollViewAddRemoveTest.js2017-10-19T22:46:29.000+00001848

Comments

  1. Joshua Quick 2017-10-19

    Attached script [^ScrollViewAddRemoveTest.js] to better test add(), insertTo(), and remove() functionality.
  2. Joshua Quick 2017-10-23

    PR (6.3.x): https://github.com/appcelerator/titanium_mobile/pull/9543 PR (master): https://github.com/appcelerator/titanium_mobile/pull/9545
  3. Lokesh Choudhary 2017-10-23

    FR Passed for both PR's. Master merged, waiting for merge to get enabled for 6.3.0.
  4. Lokesh Choudhary 2017-10-26

    PR for backport merged.
  5. Lokesh Choudhary 2017-10-26

    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

JSON Source