Fixed #12

src/renderer-process/application.js

@@ -7,7 +7,6 @@ const electron = require('electron')
 const controller = require('./controller.js')
 const network = require('./network.js')
 const path = require('path')
-const fs = require('fs-extra')
 
 var project
 
@@ -118,23 +117,16 @@ exports.setBabble = function(babbling) {
 }
 
 // Update the hotbar button for a character
-exports.updateCharacter = function(character, thumbnail) {
+exports.updateCharacter = function(character, updateThumbnail) {
 	var index = project.project.hotbar.indexOf(character.id)
 	if (index > -1) {
 		controller.updateCharacter(index, character)
 		if (('' + document.getElementById('char ' + index).className).indexOf('selected') > -1) {
 			controller.setPuppetLocal(index)
 		}
 		document.getElementById('char ' + index).getElementsByClassName('desc')[0].innerHTML = character.name
-	}
-	if (thumbnail) {
-		fs.ensureDirSync(path.join(project.assetsPath, '..', 'thumbnails'))
-		fs.writeFile(path.join(project.assetsPath, '..', 'thumbnails', character.id + '.png'), new Buffer(thumbnail, 'base64'), (err) => {
-	        if (err) console.log(err)
-	        if (index > -1) {
-	        	document.getElementById('char ' + index).style.backgroundImage = 'url(' + path.join(project.assetsPath, '..', 'thumbnails', character.id + '.png?random=' + new Date().getTime()) + ')'
-	        }
-	    })
+		if (updateThumbnail)
+			document.getElementById('char ' + index).style.backgroundImage = 'url(' + path.join(project.assetsPath, '..', 'thumbnails', 'new-' + character.id + '.png?random=' + new Date().getTime()) + ')'
 	}
 }
 

src/renderer-process/controller.js

@@ -6,6 +6,7 @@ const editor = require('./editor.js')
 const network = require('./network.js')
 const status = require('./status.js')
 const Stage = require('./stage.js').Stage
+const fs = require('fs-extra')
 const path = require('path')
 const url = require('url')
 
@@ -339,7 +340,15 @@ exports.updateCharacter = function(index, character) {
 
 exports.saveCharacter = function(character, thumbnail) {
     project.saveCharacter(character)
-    application.updateCharacter(character, thumbnail)
+	if (thumbnail) {
+		fs.ensureDirSync(path.join(project.assetsPath, '..', 'thumbnails'))
+		fs.writeFile(path.join(project.assetsPath, '..', 'thumbnails', 'new-' + character.id + '.png'), new Buffer(thumbnail, 'base64'), (err) => {
+	        if (err) console.log(err)
+	        application.updateCharacter(character, true)
+	    })
+	} else {
+		application.updateCharacter(character)
+	}    
 }
 
 exports.connect = function() {

src/renderer-process/editor.js

@@ -702,7 +702,10 @@ function openPuppetPanel() {
             var selector = document.createElement('div')
             selector.id = project.characters[characters[j]].name.toLowerCase()
             selector.className = "char"
-            selector.style.backgroundImage = 'url(' + path.join(project.assetsPath, '..', 'thumbnails', characters[j] + '.png?random=' + new Date().getTime()) + ')'
+            if (fs.existsSync(path.join(project.assetsPath, '..', 'thumbnails', 'new-' + characters[j] + '.png')))
+                selector.style.backgroundImage = 'url(' + path.join(project.assetsPath, '..', 'thumbnails', 'new-' + characters[j] + '.png?random=' + new Date().getTime()) + ')'
+            else
+                selector.style.backgroundImage = 'url(' + path.join(project.assetsPath, '..', 'thumbnails', characters[j] + '.png?random=' + new Date().getTime()) + ')'
             charList.appendChild(selector)
             selector.innerHTML = '<div class="desc">' + project.characters[characters[j]].name + '</div>'
             selector.charid = characters[j]

src/renderer-process/project.js

@@ -58,8 +58,12 @@ module.exports = exports = remote.getGlobal('project').project = {
 		fs.writeJson(settings.settings.openProject, this.project)
 		for (var i = 0; i < this.project.assets.length; i++)
 			fs.writeJson(path.join(settings.settings.openProject, '..', 'assets', this.project.assets[i].location), this.assets[this.project.assets[i].name])
-		for (var i = 0; i < this.project.characters.length; i++)
+		for (var i = 0; i < this.project.characters.length; i++) {
 			fs.writeJson(path.join(settings.settings.openProject, '..', 'characters', this.project.characters[i].location), this.characters[this.project.characters[i].id])
+			if (fs.existsSync(path.join(this.assetsPath, '..', 'thumbnails', 'new-' + this.project.characters[i].id + '.png')))
+                fs.renameSync(path.join(this.assetsPath, '..', 'thumbnails', 'new-' + this.project.characters[i].id + '.png'), 
+                	path.join(this.assetsPath, '..', 'thumbnails', this.project.characters[i].id + '.png'))
+		}
 		this.oldProject = JSON.stringify(this.project)
 		this.oldAssets = JSON.stringify(this.assets)
 		this.oldCharacters = JSON.stringify(this.characters)