GoomiiV2 / Meldii

Melder V2
MIT License
11 stars 7 forks source link

Launch Firefall and spelling fixes #3

Closed LoneBoco closed 9 years ago

LoneBoco commented 9 years ago

This will add the ability to launch Firefall via a button on the titlebar. I also experimented with a small button above the addon listing, but I think this looks better.

GoomiiV2 commented 9 years ago

Looks pretty good and thanks for fixing up some of my typos >,> The only thing I'd say to change would be to replace

"MessageBox.Show("Error launching application.  Could not find FirefallClient.exe.", "Launch error", MessageBoxButton.OK, MessageBoxImage.Error);"

with

ShowAlert("Launch error", "Error launching application.\nCould not find FirefallClient.exe.");

So that it fits the style of the other popups.

LoneBoco commented 9 years ago

Nice. I switched to ShowAlert.

GoomiiV2 commented 9 years ago

I'll push out a build with this in a bit :>