{ "id": "168318", "key": "TIMOB-24801", "fields": { "issuetype": { "id": "7", "description": "gh.issue.story.desc", "name": "Story", "subtask": false }, "project": { "id": "10153", "key": "TIMOB", "name": "Titanium SDK/CLI", "projectCategory": { "id": "10100", "description": "Titanium and related SDKs used in application development", "name": "Client" } }, "fixVersions": [ { "id": "19572", "description": "Important fixes for 6.1.0", "name": "Release 6.1.1", "archived": false, "released": true, "releaseDate": "2017-06-16" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2017-06-23T01:30:49.000+0000", "created": "2017-06-09T14:36:31.000+0000", "priority": { "name": "High", "id": "2" }, "labels": [ "android", "regression" ], "versions": [ { "id": "17608", "name": "Release 6.1.0", "archived": false, "released": true, "releaseDate": "2017-05-26" } ], "issuelinks": [ { "id": "55171", "type": { "id": "10122", "name": "Gantt: start-finish", "inward": "is triggered by", "outward": "is triggering" }, "inwardIssue": { "id": "62250", "key": "TIMOB-1618", "fields": { "summary": "Android: Expose Ti.UI.Label.minimumFontSize for Parity", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "priority": { "name": "High", "id": "2" }, "issuetype": { "id": "4", "description": "An improvement or enhancement to an existing feature or task.", "name": "Improvement", "subtask": false } } } } ], "assignee": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "updated": "2017-06-23T22:11:41.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "10202", "name": "Android", "description": "Android Platform" } ], "description": "We introduced the {{minimumFontSize}} property on Android in TIMOB-1618 for iOS parity. There are reports on Slack (thanks [~perdona]) that the size changes randomly. [~cornflakesuperstar] (thanks to you as well) narrowed down the issue and found out it has to do with the density-unit:\r\n{quote}\r\nI have narrowed this down some further for my app and determined it is a problem with android os 6.0.1 (marshmallow) for my app regardless of which version of the ti sdk it is built with (I have tried ti 5.5.1, 6.0.4 and 6.1.0).\r\n\r\nI removed \r\n\r\n\r\nfrom tiapp.xml and I also removed ‘dp’ and ‘sp’ denominations from fontSize specifications in all the tss for my app in order to fix the font size issue.\r\n\r\nIt also seems related to a bizarre shrinking bug whereby the actionbar for sub-windows in the app appear to be smaller than normal and the contents of those actionbars are also shrunk.\r\n{quote}", "attachment": [], "flagged": false, "summary": "Android: Ti.UI.Label.minimumFontSize property changes size randomly", "creator": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "subtasks": [], "reporter": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "environment": null, "comment": { "comments": [ { "id": "421696", "author": { "name": "cornflakesuperstar", "key": "cornflakesuperstar", "displayName": "Jason Kotchoff", "active": true, "timeZone": "America/Los_Angeles" }, "body": "I was able to narrow the issue down further and identify that in my case, it was related to usage of the {{ti.admob}} module in my app (I wasn't using {{minimumFontSize}}) and that it's consistently reproduceable. Specifically, after I add any {{Admob.createView}} to my android app (in a sub-window in my case), all subsequent labels on that window and subsequent windows render as incorrectly sized for Android 6.0.1. Android 4.2.2 doesn't have any problems. \r\n\r\nI was initially able to work around this by removing the -dp- from the tiapp.xml and also removing -dp- and -sp- from all my label size definitions in all tss however I was unable to fix the label icons in my ActionBars which is causing my ActionBars to shrink and become practically unusable and so for now my workaround is just to remove {{ti.admob}} completely from my app.\r\n\r\nNote, the ti.admob module doesn't have a published 6.0.0 compatible .zip so I created one in this pull request: https://github.com/appcelerator-modules/ti.admob/pull/70 and that is what is causing this bug. I didn't make any code changes to it though so it seems like a problem with the actual admob module itself.", "updateAuthor": { "name": "cornflakesuperstar", "key": "cornflakesuperstar", "displayName": "Jason Kotchoff", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-09T23:33:19.000+0000", "updated": "2017-06-09T23:35:13.000+0000" }, { "id": "421697", "author": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "body": "In my case, I was unable to identify the cause. \r\nA listView was changing its row's labels fontsizes at runtime, exploding views and making text unreadable. \r\nSo I set the property to null on Android and moved on.\r\nIs it relevant to say that my project is classic (no alloy) ?\r\nAlso I have a video demonstrating what's going on visually on the device, but dont know the best way to attach here...\r\n", "updateAuthor": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-10T05:11:40.000+0000", "updated": "2017-06-10T05:11:40.000+0000" }, { "id": "421701", "author": { "name": "gmathews", "key": "gmathews", "displayName": "Gary Mathews", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~perdona] Could you provide a test case that reproduces this issue?", "updateAuthor": { "name": "gmathews", "key": "gmathews", "displayName": "Gary Mathews", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-10T17:47:35.000+0000", "updated": "2017-06-10T17:47:35.000+0000" }, { "id": "421877", "author": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "body": "@Gary Mathews Unfortunately I can't. Already lost too much time fixing the issues 6.1.0.GA have brought to my classic Ti app.\r\nWhat I can tell you is that I'm using {code:javascript} minimumFontSize: 14 {code} and my app started to behave like in the video below.\r\n\r\nhttps://www.dropbox.com/s/l4kgfceunw25b8v/2017_06_13_14_18_59.mp4?dl=0", "updateAuthor": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-13T17:43:03.000+0000", "updated": "2017-06-13T17:43:03.000+0000" }, { "id": "421881", "author": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~perdona], thanks for the video footage. The issue here (and we're able to reproduce it) is that the label should only scale down. It should not scale up. That's why you're seeing large clipped text on Android. We're looking into fixing this issue now.", "updateAuthor": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-13T18:07:47.000+0000", "updated": "2017-06-13T18:07:47.000+0000" }, { "id": "421889", "author": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "body": "@Joshua Quick sorry I wasn't able to help in a more effective way!\r\nBest of luck with this issue and thanks in advance!", "updateAuthor": { "name": "perdona", "key": "perdona", "displayName": "Caio Perdona", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-13T18:15:53.000+0000", "updated": "2017-06-13T18:15:53.000+0000" }, { "id": "421910", "author": { "name": "cornflakesuperstar", "key": "cornflakesuperstar", "displayName": "Jason Kotchoff", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Joshua, thanks for taking this up. If you can, it would be great if you could test that your solution works with the appcelerator ti.admob module as well.", "updateAuthor": { "name": "cornflakesuperstar", "key": "cornflakesuperstar", "displayName": "Jason Kotchoff", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-13T22:47:33.000+0000", "updated": "2017-06-13T22:47:33.000+0000" }, { "id": "422332", "author": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "body": "PR (6.1.x): https://github.com/appcelerator/titanium_mobile/pull/9161", "updateAuthor": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-19T22:51:27.000+0000", "updated": "2017-06-19T22:51:27.000+0000" }, { "id": "422600", "author": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "body": "FR Passed for 6.1.1", "updateAuthor": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-23T01:30:49.000+0000", "updated": "2017-06-23T01:30:49.000+0000" }, { "id": "422606", "author": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "body": "PR (master): https://github.com/appcelerator/titanium_mobile/pull/9175", "updateAuthor": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-23T02:47:12.000+0000", "updated": "2017-06-23T02:47:12.000+0000" }, { "id": "422677", "author": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "body": "FR passed for master.", "updateAuthor": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-23T20:59:32.000+0000", "updated": "2017-06-23T20:59:32.000+0000" }, { "id": "422692", "author": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Verified the fix.\r\nTest info in the PR.\r\n\r\nClosing.\r\n\r\nStudio Ver: 4.9.0.201705302345\r\nSDK Ver: 6.2.0.v20170623140932, 6.1.1.v20170623141152\r\nOS Ver: 10.12.3\r\nXcode Ver: Xcode 8.3.3\r\nAppc NPM: 4.2.9\r\nAppc CLI: 6.2.2\r\nTi CLI Ver: 5.0.14\r\nAlloy Ver: 1.9.11\r\nNode Ver: 6.10.1\r\nJava Ver: 1.8.0_101\r\nDevices: ⇨ google Pixel — Android 7.1.1\r\n⇨ google Nexus 5 — Android 6.0.1\r\n", "updateAuthor": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-06-23T22:11:35.000+0000", "updated": "2017-06-23T22:11:35.000+0000" } ], "maxResults": 14, "total": 14, "startAt": 0 } } }