[TIMOB-24158] Android: Ti.UI.TableViewRow not shown in 6.1.0 (Regression)

GitHub Issue	n/a
Type	Bug
Priority	Critical
Status	Closed
Resolution	Fixed
Resolution Date	2016-11-22T02:50:12.000+0000
Affected Version/s	Release 6.1.0
Fix Version/s	Release 6.1.0
Components	Android
Labels	android, regression, tableview
Reporter	Dongwoo Gim
Assignee	Gary Mathews
Created	2016-11-18T12:26:05.000+0000
Updated	2016-11-22T13:51:52.000+0000

Description

TableViewRow don't drawing on Ti SDK 6.1.0.v20161118021723 Ti SDK 6.0.0.GA is ok.

...
      <TableView id="tableView" ...>
        <TableViewSection>
          <TableViewRow class="nonSelectionStyle">
            <Label id="sn_notifyPush" text="L('sn_notifyPush')" class="leftMargin" />
            <Switch class="rightMargin" id="permission_push_switch"/>
            <View class="lineBottom" />
          </TableViewRow>
          <TableViewRow class="nonSelectionStyle">
            <Label id="sn_notifyEventAndNotice" text="L('sn_notifyEventAndNotice')" class="leftMargin" />
            <Switch class="rightMargin" id="Installation_channels_eventSwitch"/>
            <View class="lineBottom" />
          </TableViewRow>
        </TableViewSection>

        ...
      </TableView>
...

Attachments

File	Date	Size
스크린샷 2016-11-18 오후 9.16.42.png	2016-11-18T12:23:39.000+0000	267867
스크린샷 2016-11-18 오후 9.17.51.png	2016-11-18T12:23:42.000+0000	294694

Comments

Hans Knöchel 2016-11-18

I am able to reproduce this issue. Test-case:

   var win = Ti.UI.createWindow({
       backgroundColor: "#fff"
   });
   
   var row = Ti.UI.createTableViewRow({height: 50});
   row.add(Ti.UI.createSwitch({right: 15}));
   row.add(Ti.UI.createLabel({left: 15, text: "My Row"}));
   
   var tableView = Ti.UI.createTableView({
       data: [row]
   });
   
   win.add(tableView);
   win.open();

[~fmerzadyan] Can you take a log in the git history between 6.0.0 and 6.1.0? Thx!

Gary Mathews 2016-11-21

master: https://github.com/appcelerator/titanium_mobile/pull/8632

Samir Mohammed 2016-11-22

Verified fixed, was able to see the switch on the test case provided above by Hans on SDK version 6.1.0.v20161121162514. *Environment*

   Appcelerator Command-Line Interface, version 6.0.0
   Android 7.0 (Google Nexus 6P)
   Operating System Name: Mac OS X El Capitan
   Operating System Version: 10.11.6
   Node.js Version: 4.6.0
   npm: 4.2.8
   Titanium CLI Version: 5.0.10
   Titanium SDK Version: 6.1.0.v20161121162514 
   Xcode: 8.0
   Appcelerator Studio: 4.8.0.201611121409

Dongwoo Gim 2016-11-22

Thank you!! I love Titanium & Appcelerator.

JSON Source