sure, i'll take a look if you prepare a branch for me
Hello! I've been out today
@grokys Thank you! I've prepared the branch. Just clone this: https://github.com/SuperJMN/Perspex-OmniXAMLed.git
And check out the branch named "TreeView" (the one I prepared with the failing scenario)
If you run the classic TestApplication, you will see that it contains only a TreeView with an item
it loads perfectly
but if you run XamlTestApplication (I have separated both test apps), you will see that it throws an exception
inside the XAML tester you will see a Views folder with 3 XAML files. The one that is loaded during the test is "PerspexWindow-Full.xaml"
You will soon notice that is doesn't contain much! in fact, it contains the same composition as before (Window with TreeView with one Item)
ok, thanks! it will probably be at least tomorrow before i can take a look
but i'm sure it'll be something simple
for sure!
something that depends on order
by the way, I modified the Items property in ItemsControl
I added a default value
since it's IEnumerable, the object assembler cannot instantiate it (it's an interface) so the default value is a Collection<object>
WPF also does this
ah really? ok
if the object in the XAML has a collection, it requires to have it ready
fore example, this doesn't inflate!
ah really?
this doesn't inflate:
class Sample
{
public IEnumerable<Collection> Children {get; set; }
}
this DOES work:
i was actually thinking of changing the type of
Items to be an ICollection
class Sample
{
Sample()
{
Children = new List<object>();
}
public IEnumerable<Collection> Children {get; set; }
}
yeah, that makes sense i guess
since it's just an interface, it cannot choose which implementation
it just knows that it will use it, and add items
sure, though probably it could just choose one (probably
List<T>)
(via casting and tricky stuff)
but i understand you want to follow what System.Xaml does
haha, that is an option !
yes, I didn't want to put the loader the responsibility to choose which implementation to use
it's a bit silly in that aspect
:)
so, when you try the code, keep in mind that I did that change in my fork
the rest is unchanged
ah! also DataTemplate has a default constructor
but that doesn't affect, I suppose :)
yeah that should all be ok!
thanks!