feat: generate banners for androidTV #581
Conversation
|
This would be a great addition for anyone developing for Android TV apps. Was hoping that it would already exist; and look at that: the code is already here and just needs to be merged. Would love to see it coming in (maybe as an active "cli arg" to activate since it may not be needed for most people). |
| @@ -67,6 +67,7 @@ export class Project extends MobileProject { | |||
| androidIconForeground: await this.loadInputAsset('android/icon-foreground', AssetKind.Icon, Platform.Android), | |||
| androidIconBackground: await this.loadInputAsset('android/icon-background', AssetKind.Icon, Platform.Android), | |||
|
|
|||
| androidBanner: await this.loadInputAsset('android/banner', AssetKind.Banner, Platform.Android), | |||
There was a problem hiding this comment.
The README update shows that the banner.png file should be placed in the assets/banner.png folder directly.
But it is not read that way. One has to place it in the assets/android/banner.png folder.
Is this intended? Then it should be noted in the README docs. Alternatively, and I think more logically, it should work in both folders regardless where it is placed.
There was a problem hiding this comment.
Also found that the banner addition seems to add the manifest line regardless android:banner="@drawable/banner" regardless if it's actually adding the banner (aka if the file is not in assets/android/banner.png it will still add the line in the AndroidManifest.xml)
There was a problem hiding this comment.
The PR places the files into the @drawable/banner but the banner.png file isn not placed into the res/drawable folder itself and only is placed in a very view select folders.
AndroidManifest.xml line also states that the images are to be found in the res/drawable format. But the docs (Source) state that the banner images should be placed next to the icon files in the mipmap folder structure.
This PR adds functionality to create banners for androidTV, they are close to horizontal splash screens, but sometimes marginally different
#580