[ALOY-716] When using view-model binding, whitespace inside curly braces disables binding
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Medium |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2018-03-05T17:48:07.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Alloy 1.2.0, 2013 Sprint 13 |
Components | Models, XML |
Labels | qe-testadded, regression |
Reporter | Tony Lukasavage |
Assignee | Tony Lukasavage |
Created | 2013-06-20T22:48:30.000+0000 |
Updated | 2018-03-05T17:48:13.000+0000 |
Description
problem
http://developer.appcelerator.com/question/153856/alloy-update-view-with-fetched-collectionexpected
Both labels in the example below should be data bound, regardless of the superfluous whitespace.
<Alloy>
<TableView dataCollection="someCollection">
<TableViewRow layout="vertical">
<Label text="{text1}"/>
<Label text="{ text2 }"/>
</TableViewRow>
</TableView>
</Alloy>
Attachments
File | Date | Size |
---|---|---|
Screen Shot 2013-06-20 at 8.20.48 PM.png | 2013-06-21T00:21:04.000+0000 | 65849 |
PR: https://github.com/appcelerator/alloy/pull/162 test app: https://github.com/appcelerator/alloy/tree/master/test/apps/testing/ALOY-716 Functional testing can be verified by testing on all supported platforms and running the app successfully. When the app loads, it should consist of a table view that has a number of rows that contain a populated title, subtitle, and image on the right hand side. These values are populated by the collection models and make use of some really extreme examples of superfluous whitespace in the view binding. The attached screenshot gives an indication of what the successfully running app should look like.
Also to note, the original Q&A question has been answered with a reference to this ticket and its resolution.
Verified as fixed. Environment: Titanium SDK 3.1.2.v20130710144553 Appcelerator Studio 3.1.2.201307101037 Alloy 1.2.0 Closing.
I submitted pull requests to resolve ALOY-1124. After testing, please close this ticket again too.
Reopening to fix. Somehow appeared in our latest sprint.