Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-24308] Android: Child view's percent width incorrect of horizontal layout parent view (6.1.0 regression)

GitHub Issuen/a
TypeBug
PriorityCritical
StatusClosed
ResolutionFixed
Resolution Date2017-02-14T21:52:23.000+0000
Affected Version/sRelease 6.1.0
Fix Version/sRelease 6.0.2
ComponentsAndroid
Labelsandroid, layout, regression
ReporterDongwoo Gim
AssigneeGary Mathews
Created2017-01-13T12:21:46.000+0000
Updated2017-05-22T03:45:13.000+0000

Description

Attachments

FileDateSize
스크린샷 2017-01-13 오후 9.14.31.png2017-01-13T12:15:59.000+0000275840

Comments

  1. Dongwoo Gim 2017-01-13

    Fix it. https://github.com/appcelerator/titanium_mobile/pull/8750
  2. Dongwoo Gim 2017-01-20

    And, other bugs fix + commit cleanup https://github.com/appcelerator/titanium_mobile/pull/8766
  3. Gary Mathews 2017-01-26

    6_0_X: https://github.com/appcelerator/titanium_mobile/pull/8794
  4. Andy Waldman 2017-01-27

    Friday 27th January 2017 ENV: MacOS:10.12.1 XCODE: 8.2.1 GM (golden master) APPC CLI Core: 6.1.0 APPC CLI NPM: 4.2.8 SDK: 6.0.2.v20170126173908 Studio build: 4.8.1.201612050850 NPM: 2.15.9 Node: 4.5.0 Device: Google Pixel Device Verison: 7.1 Step 1) load the test app.js code :
       var win = Ti.UI.createWindow({
           backgroundColor: '#fff'
       });
        
       var view = Ti.UI.createView({
         height: Ti.UI.FILL,
         width: Ti.UI.FILL,
         layout: "horizontal",
         backgroundColor: "gray"
       });
        
       var view1 = Ti.UI.createView({
         width: "25%",
         height: Ti.UI.FILL,
         backgroundColor: "red"
       });
        
       var view2 = Ti.UI.createView({
         width: "25%",
         height: Ti.UI.FILL,
         backgroundColor: "green"
       });
        
       var view3 = Ti.UI.createView({
         width: "25%",
         height: Ti.UI.FILL,
         backgroundColor: "blue"
       });
        
       var view4 = Ti.UI.createView({
         width: "25%",
         height: Ti.UI.FILL,
         backgroundColor: "yellow"
       });
        
       view.add(view1);
       view.add(view2);
       view.add(view3);
       view.add(view4);
        
       win.add(view);
        
       win.open();
       
    Step 2) Run the application Step 3) verified that the portrait version for horizontal view is working Step 4) test landscape in which it failed this is because the yellow disappeared of screen and after testing the bug, it still had the yellow column, and so i believe that there is still a bug and so will reopen this ticket
  5. Andy Waldman 2017-01-27

    works portrait not landscape
  6. Andy Waldman 2017-01-30

    This only fails when using the GOOGLE PIXLE device
  7. Abir Mukherjee 2017-02-13

    Environment: MacOS:10.12.1 XCODE: 8.2.1 GM (golden master) APPC CLI Core: 6.1.0 APPC CLI NPM: 4.2.8 SDK: 6.0.2.v20170213083448 Studio build: 4.8.1.201612050850 NPM: 2.15.9 Node: 4.5.0 Device: Google Pixel and Google Pixel XL Device Verison: 7.1 Tested with both Google Pixel XL and Pixel. The issue is seen only on the Pixel; it is not seen with the Pixel XL.
  8. Gary Mathews 2017-02-13

    master: https://github.com/appcelerator/titanium_mobile/pull/8835 6_0_X: https://github.com/appcelerator/titanium_mobile/pull/8836
  9. Abir Mukherjee 2017-02-14

    NPM Version: 2.15.9 Node Version: 4.5.0 Mac OS: 10.12.1 Appc CLI: 6.1.0 Appc CLI NPM: 4.2.8 Passed FR with this environment: NPM Version: 2.15.9 Node Version: 4.5.0 Mac OS: 10.12.1 Appc CLI: 6.1.0 Appc CLI NPM: 4.2.8 Appcelerator Studio, build: 4.8.1.201612050850 Titanium SDK Version: 6.0.2 and 6.1.0 Android Device: 7.1, 7.1.1, 6.0.1 Tested with Google Pixel, and Pixel XL. Four color bars were seen in either Portrait or Landscape mode, as expected. Gray bar was not seen, as expected. For future regression testing, this needs to the tested with Pixel at the minimum.
  10. Abir Mukherjee 2017-02-14

    Fix is verified in SDK 6.0.2.v20170214112552, and Fix is verified in SDK 6.1.0.v20170215102838.
  11. Dongwoo Gim 2017-05-22

  12. Dongwoo Gim 2017-05-22

    I send PR after solving this problem. https://github.com/appcelerator/titanium_mobile/pull/9073

JSON Source