Merge pull request #182 from juliandescottes/enhancement-gif-encoding

Enhancement gif encoding

src/js/controller/PalettesListController.js

@@ -10,6 +10,8 @@
   // I apologize to my future self for this one.
   var NO_SCROLL_MAX_COLORS = 20;
 
+  var MAX_COLORS = 100;
+
   ns.PalettesListController = function (paletteController, usedColorService) {
     this.usedColorService = usedColorService;
     this.paletteController = paletteController;
@@ -77,6 +79,11 @@
         colors = palette.colors;
       }
     }
+
+    if (colors.length > MAX_COLORS) {
+      colors = colors.slice(0, MAX_COLORS);
+    }
+
     return colors;
   };
 

src/js/controller/settings/GifExportController.js

@@ -2,6 +2,7 @@
   var ns = $.namespace("pskl.controller.settings");
 
   var URL_MAX_LENGTH = 60;
+  var MAX_GIF_COLORS = 256;
 
   ns.GifExportController = function (piskelController) {
     this.piskelController = piskelController;
@@ -114,11 +115,14 @@
   };
 
   ns.GifExportController.prototype.renderAsImageDataAnimatedGIF = function(zoom, fps, cb) {
+    var colorCount = pskl.app.currentColorsService.getCurrentColors().length;
+    var preserveColors = colorCount < MAX_GIF_COLORS;
     var gif = new window.GIF({
       workers: 2,
-      quality: 10,
+      quality: 1,
       width: this.piskelController.getWidth()*zoom,
-      height: this.piskelController.getHeight()*zoom
+      height: this.piskelController.getHeight()*zoom,
+      preserveColors : preserveColors
     });
 
     for (var i = 0; i < this.piskelController.getFrameCount(); i++) {