Added option to disable screen docking.

Fixed issue causing net graph form location to reset to load position upon saving options.

Author: Bilge

Source/Forms/NetGraphForm.cs

@@ -82,7 +82,7 @@ private void SyncOptionsWithSampler(Options options) {
         }
 
         private void SyncOptionsWithUI(Options options) {
-            if (!options.Bounds.Location.IsEmpty) {
+            if (!options.Bounds.Location.IsEmpty && StartPosition != FormStartPosition.Manual) {
                 StartPosition = FormStartPosition.Manual;
                 Location = options.Bounds.Location;
             }
@@ -139,6 +139,10 @@ private void ToggleWindowVisibility() {
         }
 
         private bool CanSnap(int clientEdge, int containerEdge, int tension) {
+            if (!options.Docking) {
+                return false;
+            }
+
             const int DOCK_DISTANCE = 10;
 
             return Math.Abs(clientEdge - containerEdge) <= DOCK_DISTANCE && Math.Abs(tension) <= DOCK_DISTANCE;

Source/Forms/OptionsForm.Designer.cs

@@ -21,20 +21,30 @@ internal OptionsForm(Options options) {
 
             InitializeComponent();
             LoadNetworkInterfaces();
+            LoadSettings();
+
+            // Load default page.
+            networking.SimulateClick();
         }
 
         internal delegate void ApplyOptionsDelegate(OptionsForm sender, Options options);
 
         internal event ApplyOptionsDelegate ApplyOptions;
 
         private void SaveSettings() {
-            if (SelectedNic != null) {
+            if ((Options.NetworkInterface = SelectedNic) != null) {
                 Options.NicSpeeds[SelectedNic.Id] = ulong.Parse(customSpeed.Text);
             }
 
+            Options.Docking = dock.Checked;
+
             Options.Save();
         }
 
+        private void LoadSettings() {
+            dock.Checked = Options.Docking;
+        }
+
         private void LoadNetworkInterfaces() {
             var interfaces = showDisabledAdapters.Checked ? NetworkInterfaces.FetchAll() : NetworkInterfaces.FetchOperational();
 
@@ -99,8 +109,6 @@ private void apply_Click(object sender, EventArgs e) {
         }
 
         private void nics_SelectedIndexChanged(object sender, EventArgs e) {
-            Options.NetworkInterface = SelectedNic;
-
             detectedSpeed.Text = (SelectedNic?.Speed / 8).ToString();
             customSpeed.Text = SelectedNic != null && Options.NicSpeeds.ContainsKey(SelectedNic.Id)
                 ? Options.NicSpeeds[SelectedNic.Id].ToString()

Source/Forms/OptionsForm.cs

@@ -17,6 +17,7 @@ public Options(Settings settings) {
             Bounds = settings.Bounds;
             Topmost = settings.Topmost;
             Transparent = settings.Transparent;
+            Docking = settings.Docking;
         }
 
         public NetworkInterface NetworkInterface { get; set; }
@@ -29,6 +30,8 @@ public Options(Settings settings) {
 
         public bool Transparent { get; set; }
 
+        public bool Docking { get; set; }
+
         public Options Clone() {
             var clone = (Options)MemberwiseClone();
             clone.NicSpeeds = new Dictionary<string, ulong>(NicSpeeds);
@@ -42,6 +45,7 @@ public void Save() {
             settings.Bounds = Bounds;
             settings.Topmost = Topmost;
             settings.Transparent = Transparent;
+            settings.Docking = Docking;
 
             settings.Save();
         }