fix palette picker color index and offset, test appveyor config

Author: unitycoder

PixelArtTool/MainWindow.xaml.cs

@@ -301,11 +301,12 @@ void PickPalette(MouseEventArgs e)
         {
             byte[] ColorData = { 0, 0, 0, 0 }; // B G R !
             int x = (int)(e.GetPosition(paletteImage).X / paletteScaleX);
-            int y = (int)((e.GetPosition(paletteImage).Y - 1) / paletteScaleY); // -1 to fix offset issue
+            int y = (int)(e.GetPosition(paletteImage).Y / paletteScaleY);
             if (x < 0 || x > paletteResolutionX - 1) return;
             if (y < 0 || y > paletteResolutionY - 1) return;
-            currentColorIndex = y * paletteResolutionX + x + 1; // +1 for fix index magic number..
-            if (currentColorIndex >= palette.Length) currentColorIndex--;
+
+            currentColorIndex = y * paletteResolutionX + x;
+            if (currentColorIndex > palette.Length) currentColorIndex--;
             currentColor = palette[currentColorIndex];
             ResetCurrentBrightnessPreview(currentColor);
         }
@@ -585,6 +586,7 @@ void DrawingAreaMouseMoved(object sender, MouseEventArgs e)
             // snap preview rectangle to grid
             var left = x * canvasScaleX;
             var top = y * canvasScaleX;
+            // NOTE: this causes palette pixels to distort/move?
             rectPixelPos.Margin = new Thickness(89 + left, 50 + top, 0, 0);
 
         } // drawingareamousemoved

PixelArtTool/Tools.cs

@@ -40,6 +40,23 @@ public static Color Win32GetScreenPixel(int x, int y)
             return Color.FromArgb(255, (byte)((a >> 0) & 0xff), (byte)((a >> 8) & 0xff), (byte)((a >> 16) & 0xff));
         }
 
+        public static PixelColor Win32GetScreenPixelColor()
+        {
+            CustomPoint cursor;
+            GetCursorPos(out cursor);
+            IntPtr desk = GetDesktopWindow();
+            IntPtr dc = GetWindowDC(desk);
+            int a = (int)GetPixel(dc, cursor.X, cursor.Y);
+            ReleaseDC(desk, dc);
+            var c = new PixelColor();
+            c.Alpha = 255;
+            c.Red = (byte)((a >> 0) & 0xff);
+            c.Green = (byte)((a >> 8) & 0xff);
+            c.Blue = (byte)((a >> 16) & 0xff);
+            return c;
+        }
+
+
         // fix savefiledialog extension https://stackoverflow.com/a/6104319/5452781
         public static void UseDefaultExtensionAsFilterIndex(FileDialog dialog)
         {
@@ -95,9 +112,9 @@ public static PixelColor[] LoadPalette(string path, WriteableBitmap targetBitmap
             x = y = 0;
             for (int i = 0, len = palette.Length; i < len; i++)
             {
-                SetPixel(targetBitmap, x, y, (int)palette[i].ColorBGRA);
                 x = i % paletteResolutionX;
                 y = (i % len) / paletteResolutionX;
+                SetPixel(targetBitmap, x, y, (int)palette[i].ColorBGRA);
             }
 
             return palette;

appveyor.yml

@@ -0,0 +1,19 @@
+version: 1.0.{build}
+branches:
+  only:
+  - master
+skip_tags: true
+skip_branch_with_pr: true
+configuration: Release
+build:
+  verbosity: minimal
+after_build:
+- cmd: 7z a PixelArtTool.zip %APPVEYOR_BUILD_FOLDER%\PixelArtTool\bin\Release\*.exe %APPVEYOR_BUILD_FOLDER%\PixelArtTool\bin\Release\*.exe.config
+artifacts:
+- path: PixelArtTool.zip
+  name: deploy
+deploy:
+- provider: GitHub
+  auth_token:
+    secure: c/GncTkUgNJ4TT+O8brAgr1NUhL7/vcKzsB5GXMpUj1QiI0MJf/9vXJ1yVIRf8Td1OK1kJpWlRmlry7cYeWPu8JxzGihuoJyj11Us6POKDc=
+  artifact: deploy