clarify todos

pull/35/head
Miepee 4 years ago
parent 3d208fdaef
commit df0fb9e22e

@ -816,6 +816,8 @@ public partial class MainForm : Form
ProfileXML profile = profileList[modSettingsProfileDropDown.SelectedIndex]; ProfileXML profile = profileList[modSettingsProfileDropDown.SelectedIndex];
log.Info($"User wants to create a desktop shortcut for {profile.Name}."); log.Info($"User wants to create a desktop shortcut for {profile.Name}.");
//TODO: put most of this into Lib?
// We want to give a warning to users, so they don't complain with "why didn't I get 2.0???" or "why did save broke?" // We want to give a warning to users, so they don't complain with "why didn't I get 2.0???" or "why did save broke?"
string messageText = Text.ShortcutWarningSaves; string messageText = Text.ShortcutWarningSaves;
if (profile.Name == "Community Updates (Latest)") if (profile.Name == "Community Updates (Latest)")
@ -884,7 +886,12 @@ public partial class MainForm : Form
} }
else if (OS.IsMac) else if (OS.IsMac)
{ {
throw new NotImplementedException("Creating Desktop Shortcuts on Mac has currently not been implemented!"); // TODO: implement this for mac
Application.Instance.Invoke(() =>
{
MessageBox.Show(this, "Creating Desktop Shortcuts on Mac has currently not been implemented!", Text.ErrorWindowTitle, MessageBoxType.Error);
});
return;
} }
else else
{ {

@ -168,7 +168,7 @@ public static class HelperMethods
log.Info("Internet connection established!"); log.Info("Internet connection established!");
return true; return true;
// TODO: For some reason, using the below approach creates zombie process when checking for Xdelta // TODO: For some reason, using the below approach creates zombie process when checking for Xdelta on Linux
// I have no idea why, but I also can't be bothered to troubleshoot why that is the case right now. // I have no idea why, but I also can't be bothered to troubleshoot why that is the case right now.
// Until someone figures out why that is the case, and makes the below approach not create zombie processes // Until someone figures out why that is the case, and makes the below approach not create zombie processes
// it will stay commented out and the slower above approach will be used instead. // it will stay commented out and the slower above approach will be used instead.

Loading…
Cancel
Save