Spec: use MVVM - Githubissues

IviriusCommunity / Rebound

Windows 11 done right.

https://ivirius.vercel.app

MIT License

230 stars 3 forks source link

Spec: use MVVM #51

Open Lamparter opened 11 hours ago

Lamparter commented 11 hours ago

📄 Description

Currently, Rebound does not implement MVVM namespace structure. Code should be moved from code-behind into reusable ViewModels/Models, like so:

[ ] Rebound.Models
[ ] Rebound.ViewModels
[x] Rebound.Views

⚡ API Changes

API	Description
`VIEWS`	Contains views for pages with little code-behind except button handlers
`MODELS`	Contains reusable models for logic
`VIEWMODELS`	Contains ViewModels for logic attached to pages

👥 Usage scenarios

For example, the code-behind for the installer (https://github.com/IviriusCommunity/ReboundHub/blob/c42825d4e3014a74b34788f424907d2e566727bf/Rebound/InstallationWindow.xaml.cs) would now be a separate model.

🔙 Backward Compatibility

Shouldn't affect anything.

🤔 Difficulty

Medium