Using Deserialize instaed of JsonObject

[ghost1372]

I can see in controlinfodatasource you are using jsonobject for deserializing json file, is there any specific reason for that? Why you are not using System.Text.JsonSerializer.Deserialize method?
Using Deserialize method is clean and we can use Async method and mybe faster than jsonobject!
I can make a pr for this.
@chingucoding can you answer this question?

[marcelwgn]

The reason why it wasn't used initially has to do with the history of the WinUI Gallery, formerly XAML Controls Gallery. This project started out as UWP app, well before the System.Text.JSON namespace was even a thing. Throughout the years the area did not really needed modification and only with the move to WinUI 3 and dropping UWP support, this became a valid option.

I don't think there is an argument against switching to this API now though, @karkarl what do you think?

[karkarl]

@chigucoding nailed the explanation. @ghost1372 if you can make a pr to optimize this, that'll be awesome!

[ghost1372]

Tnx, yes i can make a pr

[ghost1372]

Hi @chingucoding
I added a pr
#1276

[marcelwgn]

Awesome, I'll take a look later!

[karkarl]

Wooo! Fixed with #1276, thanks @ghost1372!