{ "id": "170778", "key": "TIMOB-25670", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "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": [], "resolution": { "id": "2", "description": "The problem described is an issue which will never be fixed.", "name": "Won't Fix" }, "resolutiondate": "2019-07-08T17:53:20.000+0000", "created": "2018-01-12T06:09:12.000+0000", "priority": { "name": "Critical", "id": "1" }, "labels": [ "android", "sdk-7", "videoplayer" ], "versions": [ { "id": "19988", "description": "", "name": "Release 7.0.1", "archived": false, "released": true, "releaseDate": "2017-12-21" } ], "issuelinks": [ { "id": "57714", "type": { "id": "10003", "name": "Relates", "inward": "relates to", "outward": "relates to" }, "inwardIssue": { "id": "173807", "key": "TIMOB-27184", "fields": { "summary": "Android: Add VideoPlayer \"scalingMode\" constants VIDEO_SCALING_RESIZE_*", "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 } } } }, { "id": "57717", "type": { "id": "10003", "name": "Relates", "inward": "relates to", "outward": "relates to" }, "inwardIssue": { "id": "173808", "key": "TIMOB-27185", "fields": { "summary": "Windows: Add VideoPlayer \"scalingMode\" constants VIDEO_SCALING_RESIZE_*", "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": null, "updated": "2019-07-08T19:26:25.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": [], "description": "Since upgrading to Ti SDK 7.x and finding that `Ti.Media.VIDEO_SCALING_ASPECT_FILL` has been removed I changed to `Ti.Media.VIDEO_SCALING_RESIZE_ASPECT_FILL`. This works on iOS but on Android, it acts like `Ti.Media.VIDEO_SCALING_RESIZE_ASPECT` which resizes to fit the available area. Here is my embed code:\r\n\r\n``\r\n\r\nAttached is a screenshot. You can see the bar down the left side where the video has not stretched to fill.", "attachment": [ { "id": "63956", "filename": "Screenshot_20180112-165135.png", "author": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "created": "2018-01-12T06:06:10.000+0000", "size": 109519, "mimeType": "image/png" }, { "id": "63960", "filename": "videoplayer demo.PNG", "author": { "name": "nsalahin", "key": "nsalahin", "displayName": "Nazmus Salahin", "active": true, "timeZone": "Asia/Dhaka" }, "created": "2018-01-13T10:17:52.000+0000", "size": 284264, "mimeType": "image/png" } ], "flagged": false, "summary": "Android: scalingMode - VIDEO_SCALING_RESIZE_ASPECT_FILL not working since upgrading to Ti SDK 7.0.x", "creator": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "subtasks": [], "reporter": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "environment": "Android 7\r\nTi SKD 7.0.0.GA and 7.0.2xxx", "comment": { "comments": [ { "id": "433063", "author": { "name": "mrahman", "key": "mrahman", "displayName": "Mostafizur Rahman", "active": true, "timeZone": "Asia/Dhaka" }, "body": "Hello [~raefa],\r\nThanks for sharing with us. Can you please test the sample code using the latest SDK 7.0.1.GA and let us know if the issue occurs.\r\nBest", "updateAuthor": { "name": "mrahman", "key": "mrahman", "displayName": "Mostafizur Rahman", "active": true, "timeZone": "Asia/Dhaka" }, "created": "2018-01-12T08:36:36.000+0000", "updated": "2018-01-12T08:36:36.000+0000" }, { "id": "433065", "author": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "body": "Hey there! Besides the fact that some demo-code would help, you can still use the old constant for iOS, as it has only been removed for iOS due to updating the underlaying API to the recommended one by Apple. Let us know if that helps! But still, we should improve the parity here.", "updateAuthor": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "created": "2018-01-12T09:25:20.000+0000", "updated": "2018-01-12T09:25:20.000+0000" }, { "id": "433074", "author": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "body": "Hi @Mostafizur Rahman and @Hans Knöchel.\r\n\r\nTested on SDK 7.0.1.GA. Same deal.\r\n\r\nI did not even think to check if the removed Ti.Media.VIDEO_SCALING_ASPECT_FILL would still work on Android! It does. For those that run into this issue and come across this post, here is what I have done for now. This workaround works:\r\n\r\n```\r\n\"#videoPlayer[platform=ios]\": {\r\n scalingMode: Ti.Media.VIDEO_SCALING_RESIZE_ASPECT_FILL\r\n}\r\n\"#videoPlayer[platform=android]\": {\r\n scalingMode: Ti.Media.VIDEO_SCALING_ASPECT_FILL\r\n}\r\n```\r\n\r\nAs for some demo code, here are the contents of my index.xml in a brand new project. \r\n\r\n```\r\n\r\n\t\r\n\t\t\r\n\t\r\n\r\n```\r\n\r\nThe only other thing I added was the intro.mp4 in the assets directory. I can create and share a repo if with this if you like ... or will this do?\r\n\r\nKeep up the good work!", "updateAuthor": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "created": "2018-01-12T10:46:59.000+0000", "updated": "2018-01-12T10:46:59.000+0000" }, { "id": "433117", "author": { "name": "nsalahin", "key": "nsalahin", "displayName": "Nazmus Salahin", "active": true, "timeZone": "Asia/Dhaka" }, "body": "Hello,\r\nTested the sample code and the issue was reproduced.\r\nEnvironment-Ti SDK-7.0.1 GA\r\nGoogle nexus 6p-7.0.0-android 24(emulator)\r\n*\r\nTest steps:*\r\n\r\n1.Copy the following code in index.xml-\r\n{{\r\n\r\n\r\n\r\n}}\r\n2.For index.js-\r\n{{$.container.open()}}\r\n3.Run\r\n\r\n*Test result-* See the attached file\r\n\r\n\r\n\r\n\r\n !videoplayer demo.PNG|thumbnail! \r\n\r\n", "updateAuthor": { "name": "nsalahin", "key": "nsalahin", "displayName": "Nazmus Salahin", "active": true, "timeZone": "Asia/Dhaka" }, "created": "2018-01-13T10:19:46.000+0000", "updated": "2018-01-13T10:20:37.000+0000" }, { "id": "433234", "author": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "body": "Hi All, I was using video in another place today and noticed that `Ti.Media.VIDEO_SCALING_RESIZE_ASPECT` also does not work as expected. It stretches the video to fit the available space (like you would expect for `Ti.Media.VIDEO_SCALING_RESIZE`). I would assume that this is related to `VIDEO_SCALING_RESIZE_ASPECT_FILL` not working too so I thought I would mention it here.", "updateAuthor": { "name": "raefa", "key": "raefa", "displayName": "Raef Akehurst", "active": true, "timeZone": "Australia/Sydney" }, "created": "2018-01-17T05:31:09.000+0000", "updated": "2018-01-17T05:31:09.000+0000" }, { "id": "449356", "author": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "body": "This is not a bug, but I can understand why this might be confusing.\r\n\r\nThe following constants were added to Titanium 7.0.0 and are only supported by iOS. Our website correctly documents them as iOS-only constants.\r\n* [VIDEO_SCALING_RESIZE|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_RESIZE]\r\n* [VIDEO_SCALING_RESIZE_ASPECT|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_RESIZE_ASPECT]\r\n* [VIDEO_SCALING_RESIZE_ASPECT_FILL|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_RESIZE_ASPECT_FILL] \r\n\r\nAndroid only supports the old video scaling constants below. And these constants have been removed from iOS. Unfortunately, our docs flag them as fully removed when they are only removed on iOS.\r\n* [VIDEO_SCALING_ASPECT_FILL|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_ASPECT_FILL]\r\n* [VIDEO_SCALING_ASPECT_FIT|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_ASPECT_FIT]\r\n* [VIDEO_SCALING_MODE_FILL|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_MODE_FILL]\r\n* [VIDEO_SCALING_NONE|https://docs.appcelerator.com/platform/latest/#!/api/Titanium.Media-property-VIDEO_SCALING_NONE]\r\n\r\nSo, the solution for now is to use the old scale modes on Android and the new scale modes on iOS.\r\n\r\nOn the Titanium side, we need to look into adding these new constants to Android for parity with iOS.\r\n\r\n*_Edit:_*\r\n_Windows only supports the old video scaling constants as well._", "updateAuthor": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2019-06-26T00:20:32.000+0000", "updated": "2019-06-26T00:28:48.000+0000" }, { "id": "449646", "author": { "name": "ahutton", "key": "ahutton", "displayName": "Alan Hutton", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Closing per developer comments.", "updateAuthor": { "name": "ahutton", "key": "ahutton", "displayName": "Alan Hutton", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2019-07-08T17:53:21.000+0000", "updated": "2019-07-08T17:53:21.000+0000" }, { "id": "449651", "author": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "body": "I've written up enhancement request tickets to add the iOS video scaling constants to Android and Windows. To be done in the future. Please see [TIMOB-27184] and [TIMOB-27185].\r\n", "updateAuthor": { "name": "jquick", "key": "jquick", "displayName": "Joshua Quick", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2019-07-08T19:26:25.000+0000", "updated": "2019-07-08T19:26:25.000+0000" } ], "maxResults": 9, "total": 9, "startAt": 0 } } }