You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
The exported projects for each sample contain too much of the gallery shell and should be simplified even further to optimize for simplicity and getting started. The current papercuts of the exported projects are:
The page to the sample is in a different file than the MainWindow where I would expect it. The file is also named differently for each sample, making it difficult to find exactly which one it is
The actual configuration of the sample is all contained in the SampleNavigationParameters.cs and that is not intuitive
Namespaces for projects are different making it harder to mix samples together
Describe the solution you'd like
We should change how the samples are exported by:
We should remove the SampleNavigationParameters and inject the values in the MainWIndow class as properties, or directly in the OnNavigatedTo method
We should replace SampleNavigationParameter.RequestWaitForCompletion and SampleNavigationParameter.NotifyCompletion with code to show the injected loading view. We can pattern match the line if (e.Parameter is SampleNavigationParameters sampleParam) to identify the name of the parameter and replace usage accordingly.
Helper classes should be in a separate folder to make it easy to identify the entry point to the sample (MainWindow)
Make the root namespace the same for all projects. AIDevGallery.Sample and AIDevGallery.Sample.Utils
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
The exported projects for each sample contain too much of the gallery shell and should be simplified even further to optimize for simplicity and getting started. The current papercuts of the exported projects are:
Describe the solution you'd like
We should change how the samples are exported by:
The text was updated successfully, but these errors were encountered: