Fixed colour picker css, must fix the rest

css/pixel-editor.scss

@@ -1231,6 +1231,7 @@ svg {
 #colour-picker {
 	background-color:$basecolor;
 	width:250px;
+	height:350px;
 	position:absolute;
 	display:inline-block;
 
@@ -1297,6 +1298,7 @@ svg {
 	display:flex;
 	font-family: 'Roboto', sans-serif;
 	background-color:$basetextweak;
+	width:100%;
 	
 	button {
 		font-size:14px;
@@ -1459,7 +1461,6 @@ div#palette-block {
 	z-index:1000;
 	position:relative;
 	resize: horizontal;
-	padding: 0 0 0 0;
 	margin: 0 0 0 0;
 	width:600px;
 	height:400px;
@@ -1468,10 +1469,14 @@ div#palette-block {
 div#palette-container {
 	display:inline-block;
 	background-color:black;
-	position:relative;
+	position:absolute;
 	scrollbar-color: #332f35 #232125;
 	scroll-behavior: smooth;
-/*
+	left:300px;
+	width:300px;
+	height:280px;
+	overflow-y: scroll;
+
 	&::-webkit-scrollbar {
 		background: #232125;
 		width: 0.5em;
@@ -1488,7 +1493,6 @@ div#palette-container {
 	&::-webkit-scrollbar-corner {
 		background: #232125;
 	}
-*/
 }
 
 ul#palette-list {
@@ -1510,3 +1514,25 @@ ul#palette-list {
 		max-height:75px;
 	}
 }
+
+
+div#pb-options {
+	position:relative;
+	left:300px;
+	height:100px;
+	width:300px;
+	top:280px;
+
+	button {
+		position:relative;
+		width:50%;
+		height:100%;
+		text-align:center;
+		cursor: pointer;
+	}
+
+	button:hover {
+		color: $basehovertext;
+		background-color: $basehover;
+	}
+}

js/_addColor.js

@@ -45,8 +45,6 @@ function addColor (newColor) {
             showDialogue("palette-block", false);
     });
 
-    //console.log(currentPalette);
-
     return listItem;
 }
 

js/_colorPicker.js

@@ -371,8 +371,6 @@ function movePickerIcon(event) {
         let left = (cursorPos[0] - startPickerIconPos[0][0] - 8);
         let top = (cursorPos[1] - startPickerIconPos[0][1] - 8);
 
-        console.log(left + "," + top);
-
         if (left > -8 && top > -8 && left < canvasRect.width-8 && top < canvasRect.height-8){
             activePickerIcon.style["left"] = "" + left + "px";
             activePickerIcon.style["top"]= "" + top + "px";
@@ -506,7 +504,7 @@ function updateMiniPickerColour() {
 function getMiniPickerColour() {
     let hex;
     let pickedColour;
-    console.log(currPickerIconPos[0]);
+
      pickedColour = miniPickerCanvas.getContext('2d').getImageData(currPickerIconPos[0][0] + 8, 
         currPickerIconPos[0][1] + 8, 1, 1).data;
 
@@ -583,7 +581,6 @@ function changePickingMode(event, newMode) {
         canvasContainer.removeChild(canvasContainer.children[2]);
 
         // Deleting extra hex containers
-        console.log("Deleting :O");
         hexContainers[0].parentElement.removeChild(hexContainers[0].parentElement.children[1]);
         hexContainers[i] = null;
     }
@@ -760,7 +757,6 @@ function updateOtherIcons() {
     }
     // If I'm using analogous mode, I place the current colour in the middle
     else {
-        console.log("si si si sonooo sono qui");
         hexContainers[1].style.backgroundColor = colourValue.value;
         hexContainers[1].innerHTML = colourValue.value;
 

js/_dialogue.js

@@ -14,6 +14,10 @@ function showDialogue (dialogueName, trackEvent) {
     // Showing the window
     document.getElementById(dialogueName).style.display = 'block';
 
+    // If I'm opening the palette window, I initialize the colour picker
+    if (dialogueName == 'palette-block')
+        cpInit();
+
     //track google event
     if (trackEvent)
         ga('send', 'event', 'Palette Editor Dialogue', dialogueName); /*global ga*/

js/_paletteBlock.js

@@ -0,0 +1,281 @@
+let coloursList = document.getElementById("palette-list");
+
+let rampMenu = document.getElementById("pb-ramp-options");
+let pbRampDialogue = document.getElementById("pb-ramp-dialogue");
+
+let currentSquareSize = coloursList.children[0].clientWidth;
+let blockData = {blockWidth: 500, blockHeight: 200, squareSize: 50};
+let isRampSelecting = false;
+let ramps = [];
+let currentSelection = {startIndex:0, endIndex:0, startCoords:[], endCoords: [], name: "", colour: "", label: null};
+
+// Making the palette list sortable
+new Sortable(document.getElementById("palette-list"), {
+    animation: 100,
+    onEnd: updateRampSelection
+});
+
+// Listening for the palette block resize
+new ResizeObserver(updateSizeData).observe(coloursList.parentElement);
+
+/** Listens for the mouse wheel, used to change the size of the squares in the palette list
+ * 
+ */
+coloursList.parentElement.addEventListener("wheel", function (mouseEvent) {
+    // Only resize when pressing alt, used to distinguish between scrolling through the palette and
+    // resizing it
+    if (mouseEvent.altKey) {
+        resizeSquares(mouseEvent);
+    }
+});
+
+/** Tells whether a colour is in the palette or not
+ * 
+ * @param {*} colour The colour to add
+ */ 
+function hasColour(colour) {
+    for (let i=0; i<coloursList.childElementCount; i++) {
+        let currentCol = coloursList.children[i].style.backgroundColor;
+        let currentHex = cssToHex(currentCol);
+
+        if (currentHex == colour) {
+            return true;
+        }
+    }
+    return false;
+}
+
+/** Adds a single colour to the palette
+ * 
+ * @param {*} colour The colour to add
+ */
+function addSingleColour(colour) {
+    if (!hasColour(colour)) {
+        let li = document.createElement("li");
+
+        li.style.width = currentSquareSize + "px";
+        li.style.height = currentSquareSize + "px";
+        li.style.backgroundColor = colour;
+        li.addEventListener("mousedown", startRampSelection);
+        li.addEventListener("mouseup", endRampSelection);
+        li.addEventListener("mousemove", updateRampSelection);
+
+        coloursList.appendChild(li);
+    }
+}
+
+/** Adds all the colours currently selected in the colour picker
+ * 
+ */
+function pbAddColours() {
+    let colours = getSelectedColours();
+    
+    for (let i=0; i<colours.length; i++) {
+        addSingleColour(colours[i]);
+    }
+}
+
+/** Removes all the currently selected colours from the palette
+ * 
+ */
+function pbRemoveColours() {
+    let startIndex = currentSelection.startIndex;
+    let endIndex = currentSelection.endIndex;
+
+    if (startIndex > endIndex) {
+        let tmp = startIndex;
+        startIndex = endIndex;
+        endIndex = tmp;
+    }
+
+    for (let i=startIndex; i<=endIndex; i++) {
+        coloursList.removeChild(coloursList.children[startIndex]);
+    }
+    clearBorders();
+
+    // TODO: make it so that ramps update correctly (change start and end indexes if necessary)
+}
+
+/** Starts selecting a ramp. Saves the data needed to draw the outline.
+ * 
+ * @param {*} mouseEvent 
+ */
+function startRampSelection(mouseEvent) {
+    if (mouseEvent.which == 3) {
+        let index = getElementIndex(mouseEvent.target);
+
+        isRampSelecting = true;
+
+        currentSelection.startIndex = index;
+        currentSelection.endIndex = index;
+
+        currentSelection.startCoords = getColourCoordinates(index);
+        currentSelection.endCoords = getColourCoordinates(index);
+    }
+}
+
+/** Updates the outline for the current selection.
+ * 
+ * @param {*} mouseEvent 
+ */
+function updateRampSelection(mouseEvent) {
+    if (mouseEvent != null && mouseEvent.which == 3) {
+        currentSelection.endIndex = getElementIndex(mouseEvent.target);
+    }
+    
+    if (mouseEvent == null || mouseEvent.which == 3) {
+        let startCoords = getColourCoordinates(currentSelection.startIndex);
+        let endCoords = getColourCoordinates(currentSelection.endIndex);
+
+        let startIndex = currentSelection.startIndex;
+        let endIndex = currentSelection.endIndex;
+
+        if (currentSelection.startIndex > endIndex) {
+            let tmp = startIndex;
+            startIndex = endIndex;
+            endIndex = tmp;
+
+            tmp = startCoords;
+            startCoords = endCoords;
+            endCoords = tmp;
+        }
+
+        clearBorders();
+
+        for (let i=startIndex; i<=endIndex; i++) {
+            let currentSquare = coloursList.children[i];
+            let currentCoords = getColourCoordinates(i);
+            let borderStyle = "4px solid white";
+            let bordersToSet = [];        
+
+            // Deciding which borders to use to make the outline
+            if (i == 0 || i == startIndex) {
+                bordersToSet.push("border-left");
+            }
+            if (currentCoords[1] == startCoords[1] || ((currentCoords[1] == startCoords[1] + 1)) && currentCoords[0] < startCoords[0]) {
+                bordersToSet.push("border-top");
+            }
+            if (currentCoords[1] == endCoords[1] || ((currentCoords[1] == endCoords[1] - 1)) && currentCoords[0] > endCoords[0]) {
+                bordersToSet.push("border-bottom");
+            }
+            if ((i == coloursList.childElementCount - 1) || (currentCoords[0] == Math.floor(blockData.blockWidth / blockData.squareSize) - 1) 
+                || i == endIndex) {
+                bordersToSet.push("border-right");
+            }
+            if (bordersToSet != []) {
+                console.log("qui");
+                currentSquare.style["box-sizing"] = "border-box";
+
+                for (let i=0; i<bordersToSet.length; i++) {
+                    currentSquare.style[bordersToSet[i]] = borderStyle;
+                }
+            }
+        }
+    }
+}
+/** Removes all the borders from all the squares. The borders are cleared only for the
+ *  current selection, so every border that is not white is kept.
+ * 
+ */
+function clearBorders() {
+    for (let i=0; i<coloursList.childElementCount; i++) {
+        // Resetting borders
+        if (coloursList.children[i].style["border-top"].includes("white")) {
+            coloursList.children[i].style["border-top"] = "none";
+            coloursList.children[i].style["border-left"] = "none";  
+            coloursList.children[i].style["border-right"] = "none";
+            coloursList.children[i].style["border-bottom"] = "none";
+        }
+    }
+}
+
+/** Ends the current selection, opens the ramp menu 
+ * 
+ * @param {*} mouseEvent 
+ */
+function endRampSelection(mouseEvent) {
+    if (mouseEvent.which == 3) {
+        // I'm not selecting a ramp anymore
+        isRampSelecting = false;
+        // Setting the end coordinates
+        currentSelection.endCoords = getColourCoordinates(getElementIndex(mouseEvent.target));
+    }
+}
+
+function closeAllSubmenus() {
+    let menus = document.getElementsByClassName("pb-submenu");
+
+    for (let i=0; i<menus.length; i++) {
+        menus[i].style.display = "none";
+    }
+}
+
+ /** Updates the current data about the size of the palette list (height, width and square size).
+  *  It also updates the outline after doing so.
+  * 
+  */
+ function updateSizeData() {
+    blockData.blockHeight = coloursList.parentElement.clientHeight;
+    blockData.blockWidth = coloursList.parentElement.clientWidth;
+    blockData.squareSize = coloursList.children[0].clientWidth;
+
+    updateRampSelection();
+ }
+
+ /** Gets the colour coordinates relative to the colour list seen as a matrix. Coordinates
+  *  start from the top left angle.
+  * 
+  * @param {*} index The index of the colour in the list seen as a linear array
+  */
+ function getColourCoordinates(index) {
+    let yIndex = Math.floor(index / Math.floor(blockData.blockWidth / blockData.squareSize));
+    let xIndex = Math.floor(index % Math.floor(blockData.blockWidth / blockData.squareSize));
+
+    return [xIndex, yIndex];
+ }
+
+ /** Returns the index of the element in the colour list
+  * 
+  * @param {*} element The element of which we need to get the index
+  */
+ function getElementIndex(element) {
+    for (let i=0; i<coloursList.childElementCount; i++) {
+        if (element == coloursList.children[i]) {
+            return i;
+        }
+    }
+
+    alert("Couldn't find the selected colour");
+}
+
+/** Resizes the squares depending on the scroll amount (only resizes if the user is 
+ *  also holding alt)
+ * 
+ * @param {*} mouseEvent 
+ */
+function resizeSquares(mouseEvent) {
+    let amount = mouseEvent.deltaY > 0 ? -5 : 5;
+    currentSquareSize += amount;
+
+    for (let i=0; i<coloursList.childElementCount; i++) {
+        let currLi = coloursList.children[i];
+
+        currLi.style["box-sizing"] = "content-box";
+        currLi.style.width = currLi.clientWidth + amount + "px";
+        currLi.style.height = currLi.clientHeight + amount + "px";
+    }
+
+    updateSizeData();
+}
+
+/** Converts a CSS colour eg rgb(x,y,z) to a hex string
+ * 
+ * @param {*} rgb 
+ */
+function cssToHex(rgb) {
+    rgb = rgb.match(/^rgb\((\d+),\s*(\d+),\s*(\d+)\)$/);
+    function hex(x) {
+        return ("0" + parseInt(x).toString(16)).slice(-2);
+    }
+    return "#" + hex(rgb[1]) + hex(rgb[2]) + hex(rgb[3]);
+}

js/pixel-editor.js

@@ -50,6 +50,7 @@
 //=include _resizeCanvas.js
 //=include _resizeSprite.js
 //=include _colorPicker.js
+//=include _paletteBlock.js
 
 /**load file**/
 //=include _loadImage.js

views/pixel-editor.hbs

@@ -368,21 +368,7 @@
         <!-- PALETTE -->
         <div id = "palette-block">
             <h1>Edit palette</h1>
-            <!--<div id = "palette-container" onresize="updateSizeData()">
-                <ul id = "palette-list">
-                    <li style = "background-color:rgb(255,0,0);width:50px;height:50px;"></li>
-                    <li style = "background-color:rgb(0,255,0);width:50px;height:50px;"></li>
-                </ul>
-            </div>-->
             
-            <!--
-            <div id="pb-options">
-                <ul id = "palette-options">
-                    <li><button title="Add colours to palette" onclick="addColours()">{{svg "plus.svg" width="15px" height="15px"}}</button></li>
-                    <li><button title="Remove colours from palette" onclick="removeColours()">{{svg "minus.svg" width="15px" height="15px"}}</button></li>
-                </ul>
-            </div>
-            -->
             <div id = "colour-picker">
                 <div id = "cp-modes">
                     <button id="cp-rgb" class="cp-selected-mode" onclick="changePickerMode(this, 'rgb')">RGB</button>
@@ -436,6 +422,20 @@
                     </div>
                 </div>
             </div>
+
+            <div id = "palette-container" onresize="updateSizeData()">
+                <ul id = "palette-list">
+                    <li style = "background-color:rgb(255,0,0);width:50px;height:50px;" onmousedown="startRampSelection(event)"
+                    onmousemove="updateRampSelection(event)" onmouseup="endRampSelection(event)"></li>
+                    <li style = "background-color:rgb(0,255,0);width:50px;height:50px;"onmousedown="startRampSelection(event)"
+                    onmousemove="updateRampSelection(event)" onmouseup="endRampSelection(event)"></li>
+                </ul>
+            </div>
+
+            <div id="pb-options" class="update">
+                <button title="Add colours to palette" onclick="pbAddColours()">Add colours</button>
+                <button title="Remove colours from palette" onclick="pbRemoveColours()">Remove colours</button>
+            </div>
         </div>
 
         <div id="help">