search

Codeusa / SteamCleaner

:us: A PC utility for restoring disk space from various game clients like Origin, Steam, Uplay, Battle.net, GoG and Nexon :us:
https://andrew.im/
GNU General Public License v3.0
1.97k stars 96 forks source link

Interested in using Material Design for this? #12

Closed ButchersBoy closed 8 years ago

ButchersBoy commented 8 years ago

Threw together a quick hack to illustrate:

image

andrewmd5 commented 8 years ago

Material design does look much nicer than the standard form design, if you want to implement it I'll gladly accept the pull.

ButchersBoy commented 8 years ago

:cool: I'll try and tidy it up over the weekend.

OranShuster commented 8 years ago

this means moving to WFP doesn't it? its a newer standard and thus feel more "natural". however, if Codeusa have no experience in WFP it might be hard for him to continue development without first acquiring the skills

ButchersBoy commented 8 years ago

...sorry yeah...it's WPF...

Although the code is pretty clean and straight forward and the existing form isn't to complex, so I reckon could be done without too much fuss.

Just let me know if you want to carry on, or I can fix up the pull request and you can take a look at the code.

ButchersBoy commented 8 years ago

@Codeusa OK, see what you think. Visually, I can still tidy up a bit, but at this stage it switches it from WinForms to WPF and all the existing functionality is plugged in.

Basically, MainWindow and MainWindowViewModel replace Form1, and everything else is the same. There's a few extra files to support WPF and Material Design Theme.

If you accept this PR I'd have to make a second PR to go in and remove the WinForms completely, and I could add a progress bar etc.

steamcleanerwpf

ButchersBoy commented 8 years ago

P.S make sure to restore the nuget package

andrewmd5 commented 8 years ago

@ButchersBoy i've read over everything and it all looks straight forward, nothing outrageous to learn.

Submit a new pull with all the final changes (making two is pointless) and I'll accept that after review.

Since the app isn't going to have any complex UI, this is a welcome change to make it standout a bit.

ButchersBoy commented 8 years ago

Moving to new PR as requested.