[Enhancement] Save asset view options settings

[Kryptos-FR]

Good first issue
Issues reserved for newcomers. Consider taking it only if it's your first time working on a feature or a bug in Xenko.

The view options (such as level of zoom, filters, grid or tile view, etc.) are not saved in the settings and thus use their default value when restarting the Game Studio.

Starting point:

• AssetViewUserControl.xaml in Xenko.Core.Assets.Editor
• EditorSettings in Xenko.Core.Assets.Editor
• AssetCollectionViewModel in Xenko.Core.Assets.Editor

[dadofboi]

Hi there, I'm new to Stride (actually, I've never actually participated in an open-source project before this), and I'm interested in helping out. My coding experience is limited to programing business software, so I think that this issue would be a good fit for me as a starting point, since it doesn't seem to require technical knowledge specific to game engine development.

Would it be OK if I tried to solve this issue?

[herocrab]

@dadofboi Yes, definitely! PRs are welcome!

[dadofboi]

Hi, I've been trying to look through the XAML files and whenever I try to view the Designer tab, I get this error. Everything works perfectly fine when building Stride though, or when I create another WPF project.

Is this a bug, or is it actually expected behavior? I'm not familiar at all with WPF so sorry if this is a trivial issue.

Edit: For reference, here are the error messages:

Editing the attributes causing the errors gave me this error instead:

Commenting out the Window.Resources node allow me the load the designer, but with other errors:

[manio143]

In complex WPF applications you can't really use the designer because there's a lot of dynamic parts to the views - depending on the data you're working on in the project. So you wouldn't be working with a designer but straight with XAML. While debugging you can use VS tools to select a part of the window (a control or panel) and show it in the visual tree view which can allow you to map it back to a line in the XAML file.

[dadofboi]

Thanks for the help! My company still uses WinForms (yeah, I know), so I had no idea this is how WPF works. In that case I'll spend some time learning about WPF and then try to tackle this issue again.

[soorMSWE]

Hello, is it okay if I take a stab at implementing this enhancement?

[Eideren]

Definitely, please do !

[soorMSWE]

Been looking at this and I think I have a good understanding on this system and what needs to be done, but just in case I was hoping to get some feedback real quick:

• So the fields we want to save (These view options) will essentially need to be added to the EditorSettings as new SettingsKey objects.
• In the EditorSettings the load and save behaviors are performed so the idea here would be mainly to get these fields and link them to the SettingsKey objects to be properly saved and loaded as needed.
• In order to access for example the thumbnail size, this TileThumbnailSizeProperty will need to be somehow linked to the SettingsKey object from EditorSettings.

Still have a few questions:

• In the AssetViewUserControl.xaml file, we can see the tile and grid size properties are set from the UI from user input, would we need to access those size vars from its corresponding cs file? (IE from the AssetViewUserControl.xaml.cs)
• At first I think getting one property to save and load would be easiest to implement and test but what other fields should be saved eventually? I know the issue mentions Filter (from AssetCollectionViewModel.cs),Tile and Grid Views and Current Zoom level.
• In terms of debugging this in vs, I was having some trouble finding the specific UI that these actions (like thumbnail and grid size changes) are performed in.

[Eideren]

1. Yes
2. You can keep it as just the options listed over here:
   
   So, as the first post mentions, the filter, display, sort and view type, although for that last one I'm not sure if there is any other views but the tiles, maybe @Kryptos-FR could clarify this issue further, but what I've listed should be good enough.
3. From what I understood, the screenshot above should clarify this, let me know if that's not enough and I could send a short clip over. Or maybe I just didn't understand the question properly, in which case let me know as well :)

[soorMSWE]

Thank you so much for the quick response! These help very much, and I look forward to implementing this!