unify view functions for cards

[confetti] / static / therapy_draw.js

diff --git a/static/therapy_draw.js b/static/therapy_draw.js
index 5a06728683532f1888753042794ab06bf8c79b3e..8a8e936c6873b443a0d7ba0799b491e7f5d50f7e 100644 (file)
--- a/static/therapy_draw.js
+++ b/static/therapy_draw.js
@@ -29,16 +29,17 @@ image_serialize=""
 stx=0, sty=0
 cux=0, cuy=0
 
+function setstroke(w) {
+  image.lineWidth = w
+  data.value += " stroke-width " + image.lineWidth
+}
 function setcol(c) {
   this.c = c
   image.strokeStyle = c
   image.fillStyle =  c
-  if ( c == "#FFF" ) image.lineWidth = 32
-  else image.lineWidth = 4
-  data.value += " stroke " + c + "F" + " stroke-width " + image.lineWidth
+  data.value += " stroke " + c + "F"
 }
 
-
 function relX(x){
   if ( body.clientWidth >= 800 ){
     return Math.floor(cscaleW * (x - canvas.offsetLeft))
@@ -73,9 +74,8 @@ function drawstart(x, y) {
 
   stx=relX(x), sty=relY(y)
 
-  document.getElementsByName("color").forEach(
-    function(x, y, z){ if (x.checked) setcol(x.value); }
-  );
+  setstroke(document.querySelector('input[name="penwidth"]:checked').value);
+  setcol(document.querySelector('input[name="color"]:checked').value);
 
   image.beginPath()
   draw(x, y)  // why must this not use relative Coords ???
@@ -86,7 +86,7 @@ function drawstart(x, y) {
 function drawstop() {
  
   // if path ends close to beginning ( < 50 px); then close path and fill
-  if ( mouse == 1 && Math.sqrt( Math.pow(stx - cux, 2) + Math.pow(sty - cuy, 2)) <= 50 && c !== "#FFF" ){
+  if ( false && mouse == 1 && Math.sqrt( Math.pow(stx - cux, 2) + Math.pow(sty - cuy, 2)) <= 50 && c !== "#FFF" ){
     image.lineTo( stx, sty )
     image.stroke()