[TIMOB-20569] Windows: View visible property doesn't seem to work on Windows phone

GitHub Issue	n/a
Type	Bug
Priority	High
Status	Closed
Resolution	Fixed
Resolution Date	2016-03-25T18:00:28.000+0000
Affected Version/s	Release 5.2.0
Fix Version/s	Release 5.3.0
Components	Windows
Labels	qe-5.3.0, view, visible, windows8.1
Reporter	bertelle
Assignee	Kota Iguchi
Created	2016-02-29T18:02:32.000+0000
Updated	2016-04-16T00:05:20.000+0000

Description

I can't hide a view with visible property set property to false, tried hide() method. Working fine on iOS and Android

Comments

Jebun Naher 2016-03-09

Hello, I tested this issue on my working environment. Visible property for view is working as expected for me. *Testing Environment:*

   Operating System
     Name                        = Microsoft Windows 8.1 Enterprise
     Version                     = 6.3.9600
     Architecture                = 32bit
     # CPUs                      = 4
     Memory                      = 3.9GB
   
   Node.js
     Node.js Version             = 0.12.7
     npm Version                 = 2.11.3
   
   Appcelerator CLI
     Installer                   = 4.2.3
     Core Package                = 5.2.0
   
   Titanium CLI
     CLI Version                 = 5.0.6
     node-appc Version           = 0.2.31
   Windows phone device: 8.1
   Ti.SDK = 5.1.2GA

*Test Case:*

   var win = Titanium.UI.createWindow();
   var view = Titanium.UI.createView({
      borderRadius:10,
      backgroundColor:'red',
      width:80,
      height:80,
      visible: false
   });
   win.add(view);
   win.open();

*Steps to test:* Create a classic project with the above code Run the app and observe that view is not visible. Thanks

Zakhar Zhuravlev 2016-03-14

I have the same error. I am completly sure, that visibility not working properly. Example: XML: {noformat} Text {noformat} TSS: {noformat} ".navbar": { height: Ti.UI.SIZE, backgroundColor: 'red', } //--------------------- "#settings": { visible: false } {noformat} And Settings label will be visible!
Christopher Williams 2016-03-21

[~kota] Looks like the visible property isn't taking the full child hierarchy into account?
Kota Iguchi 2016-03-23

https://github.com/appcelerator/titanium_mobile_windows/pull/588
Harry Bryant 2016-04-16

Verified as fixed, view visible property now works correctly on windows phone, including child elements. Tested on: Windows 10 Pro Windows Phone 10.0 (Microsoft Lumia 640 LTE) Appc Studio: 4.6.0.201604151027 Ti SDK: 5.3.0.v20160415081725 Appc NPM: 4.2.5-1 Appc Core: 5.3.0-12 Node: v4.4.2 *Closing Ticket.*

JSON Source