[ALOY-1366] Alloy: images from widget theme not merged with Resources

GitHub Issue	n/a
Type	Bug
Priority	High
Status	Closed
Resolution	Invalid
Resolution Date	2016-04-15T23:18:55.000+0000
Affected Version/s	n/a
Fix Version/s	Release 5.4.0
Components	Widgets
Labels	alloy, image, merge, themes
Reporter	Andrey Tkachenko
Assignee	Feon Sua Xin Miao
Created	2016-03-06T08:56:58.000+0000
Updated	2016-04-15T23:18:55.000+0000

Description

From doc: In your theme folder you can include many of the files and folders found under the app folder to override or merge with them. ||Folder or Filename||Supported since||Overwrites or Merges|| |widgets/*/assets|Alloy 1.4.0|merges folders, overwrites files| |widgets/*/styles| Alloy 1.4.0|merges folders, overwrites files| But it not work for images inside assets. Demo app attached. Environment:

Operating System
  Name                        = Mac OS X
  Version                     = 10.10.5
  Architecture                = 64bit
  # CPUs                      = 4
  Memory                      = 16.0GB

Node.js
  Node.js Version             = 4.2.2
  npm Version                 = 2.14.7

Appcelerator CLI
  Installer                   = 4.2.3
  Core Package                = 5.2.0

Titanium CLI
  CLI Version                 = 5.0.6
  node-appc Version           = 0.2.31

Attachments

File	Date	Size
app.zip	2016-03-06T08:56:40.000+0000	13659
screenshot-1.png	2016-03-07T05:36:23.000+0000	29118
Screen Shot 2016-03-10 at 3.01.09 AM.png	2016-03-09T21:02:21.000+0000	34362

Comments

Feon Sua Xin Miao 2016-03-22

It should be resolved by ALOY-1365.
Chee Kiat Ng 2016-04-06

[~falko] Can you verify if this is still an issue or it has been resolved?
Chris Barber 2016-04-08

I'm not convinced this is related to ALOY-1365.
Feon Sua Xin Miao 2016-04-08

Looks like you are right! Remove qe-investigate label.

Feon Sua Xin Miao 2016-04-12

The file structure of a theme widget is illustrated under the _Themes_ section of this doc http://docs.appcelerator.com/platform/latest/#!/guide/Alloy_Widgets Specifically *theme widget assets* can contain platform-specific folder, i.e ios, android, and image files:

   ├── app
   │   ├── README
   │   ├── alloy.js
   │   ├── config.json
   │   ├── controllers
   │   │   └── index.js
   │   ├── styles
   │   │   ├── app.tss
   │   │   └── index.tss
   │   ├── themes
   │   │   └── test
   │   │       └── widgets
   │   │           └── testWidget
   │   │               ├── assets
   │   │               │   └── images // <-- remove this and move circle.png up, or rename to i.e ios, android
   │   │               │       └── circle.png
   │   │               └── styles
   │   │                   └── widget.tss

*app/themes/test/widgets/testWidget/styles/widget.tss*

   "#circle": {
   	borderColor: "#00ff00",
   	width: 200,
   	height: 200,
   	backgroundImage: WPATH('circle.png') // <-- change to this
   }

Feon Sua Xin Miao 2016-04-15

Close this as invalid. If desire please open a feature request ticket.

JSON Source