> Something just doesn't sit right with me about passing data to > ImageView, and then telling it to only show part of it.
I think it makes sense. And it does seem consistent with the CSS usage. > It just seems cleaner to prepare the data first, and then let > ImageView simply show whatever it has been given. Much the same as > when populating a ListView, TableView or TreeView for example. The difference is that the renderer would be modifying the data, rather than the application. That isn't good. > I suppose it could be argued that the ImageView is just 'rendering' > its data in a different way I think this is a good analogy. > What if a ListItem (or similar source of content) should be displayed > with different icons depending on whether or not it was selected or > disabled? I'd suggest adding a disabledIcon property to ListItem and adding support for it to ListViewItemRenderer.