Fixed #75: Pick the correct size of the icon from a .ico file. (#76)

README.md

@@ -50,3 +50,9 @@ The sample below shows some of the properties of the control. For a more compreh
 </Window>
 ```
 
+## Contributors and Thanks
+
+Hi, I'm Philipp! This little library was originally written by me, but is currently mostly maintained by [Jan Karger](https://github.com/punker76) and [Robin Krom](https://github.com/Lakritzator). Big, big kudos to the two of you, and everybody else who [contributed](https://github.com/hardcodet/wpf-notifyicon/graphs/contributors) to this library. You rock!
+
+Make sure to check out Robin's great [Greenshot](https://getgreenshot.org/) tool (that I use on a daily basis), and Jan's [MahApps](https://github.com/MahApps) UI framework.
+

src/NotifyIconWpf/Interop/Size.cs

@@ -0,0 +1,20 @@
+using System.Runtime.InteropServices;
+
+namespace Hardcodet.Wpf.TaskbarNotification.Interop
+{
+    /// <summary>
+    /// Win API struct representing a size with width and height.
+    /// </summary>
+    [StructLayout(LayoutKind.Sequential)]
+    public struct Size
+    {
+        /// <summary>
+        /// Width.
+        /// </summary>
+        public int Width;
+        /// <summary>
+        /// Height.
+        /// </summary>
+        public int Height;
+    }
+}

src/NotifyIconWpf/Interop/SystemInfo.cs

@@ -81,5 +81,50 @@ public static Point ScaleWithDpi(this Point point)
                 Y = (int)(point.Y / DpiFactorY)
             };
         }
+
+        /// <summary>
+        /// Scale the supplied size to the current DPI settings
+        /// </summary>
+        /// <param name="size"></param>
+        /// <returns>Size</returns>
+        [Pure]
+        public static Size ScaleWithDpi(this Size size)
+        {
+            return new Size
+            {
+                Height = (int)(size.Height / DpiFactorY),
+                Width = (int)(size.Width / DpiFactorX)
+            };
+        }
+
+        #region SmallIconSize
+
+        private static Size? _smallIconSize = null;
+
+        private const int CXSMICON = 49;
+        private const int CYSMICON = 50;
+
+        /// <summary>
+        /// Gets a value indicating the recommended size, in pixels, of a small icon
+        /// </summary>
+        public static Size SmallIconSize
+        {
+            get
+            {
+                if (!_smallIconSize.HasValue)
+                {
+                    Size smallIconSize = new Size
+                    {
+                        Height = WinApi.GetSystemMetrics(CYSMICON),
+                        Width = WinApi.GetSystemMetrics(CXSMICON)
+                    };
+                    _smallIconSize = smallIconSize.ScaleWithDpi();
+                }
+
+                return _smallIconSize.Value;
+            }
+        }
+
+        #endregion
     }
 }

src/NotifyIconWpf/Interop/WinApi.cs

@@ -110,5 +110,12 @@ public static Point GetCursorPosition(NotifyIconVersion version)
 
             return TrayInfo.GetDeviceCoordinates(cursorPosition);
         }
+
+
+        /// <summary>
+        /// Gets the specified system metric.
+        /// </summary>
+        [DllImport(User32)]
+        internal static extern int GetSystemMetrics(int nIndex);
     }
-}
+}

src/NotifyIconWpf/Util.cs

@@ -179,7 +179,8 @@ public static Icon ToIcon(this ImageSource imageSource)
                 throw new ArgumentException(msg);
             }
 
-            return new Icon(streamInfo.Stream);
+            Interop.Size iconSize = SystemInfo.SmallIconSize;
+            return new Icon(streamInfo.Stream, new System.Drawing.Size(iconSize.Width, iconSize.Height));
         }
 
         #endregion