From e8e684bb8db5bc97ae9ddf946265da1ace9b2625 Mon Sep 17 00:00:00 2001
From: seiyria <kyle@seiyria.com>
Date: Thu, 29 Oct 2015 09:27:26 -0500
Subject: [PATCH] fix(tournaments): made between-match lines not overlap in
 some testable cases

closes #76
---
 src/js/directives/draw-to.js | 54 +++++++++++++++++++++++++++++++-----
 1 file changed, 47 insertions(+), 7 deletions(-)

diff --git a/src/js/directives/draw-to.js b/src/js/directives/draw-to.js
index 67e54bf..0d81729 100644
--- a/src/js/directives/draw-to.js
+++ b/src/js/directives/draw-to.js
@@ -12,14 +12,56 @@ site.directive('drawTo', ($timeout) => {
     const drawTo = JSON.parse(attrs.drawTo);
     if(!drawTo || $(`#line-${getId(JSON.stringify(drawTo))}`).length > 0) return;
 
+    const checkDuplicates = (item) => {
+
+      console.log('checking',item, item.attr('data-x'));
+
+      const xOffset = ~~item.attr('data-x');
+
+      const otherItems = $(`[data-x='${xOffset}']`).not(item[0]).add(`[data-x='${xOffset+4}']`);
+      if(otherItems.length === 0) return 0;
+
+      // what happens if there's duplicates
+      let adjustment = 0;
+      const push = 4;
+
+      otherItems.each((i, el) => {
+
+        // recalculate these each time in case they changed
+        const myOffset = item.offset();
+        const myTotalHeight = myOffset.top + item.width();
+
+        const extra = $(el);
+        const exOffset = extra.offset();
+
+        if(exOffset.top < myTotalHeight && myTotalHeight < exOffset.top + extra.width()
+          || exOffset.top < myOffset.top && myOffset.top < exOffset.top + extra.width()) {
+
+          // push it 4px to the right
+          item.css({ left: `+=${push}` });
+          item.attr('data-x', ~~item.attr('data-x') + push);
+          adjustment += push + checkDuplicates(item);
+        }
+      });
+
+      return adjustment;
+    };
+
     const drawLine = (x1, y1, x2, y2, thickness = 1, color = '#888') => {
       const length = Math.sqrt(((x2-x1) * (x2-x1)) + ((y2-y1) * (y2-y1)));
       const cx = ((x1 + x2) / 2) - (length / 2);
       const cy = ((y1 + y2) / 2) - (thickness / 2);
       const angle = Math.atan2((y1-y2),(x1-x2))*(180/Math.PI);
-      const htmlLine = `<div id='line-${getId(JSON.stringify(drawTo))}' style='padding:0; margin:0; height: ${thickness}px; background-color:${color}; line-height:1px; position:absolute; left: ${cx}px; top: ${cy}px; width: ${length}px; transform:rotate(${angle}deg);' />`;
+      const htmlLine = `<div class='line-${getId(JSON.stringify(drawTo))}' style='padding:0; margin:0; height: ${thickness}px; background-color:${color}; line-height:1px; position:absolute; left: ${cx}px; top: ${cy}px; width: ${length}px; transform:rotate(${angle}deg);' />`;
 
-      $('.duel-area').append($(htmlLine));
+      const item = $(htmlLine);
+
+      $('.duel-area').append(item);
+
+      if(x1 === x2) {
+        item.attr('data-x', ~~item.offset().left);
+        return checkDuplicates(item);
+      }
     };
 
     $timeout(() => {
@@ -29,8 +71,6 @@ site.directive('drawTo', ($timeout) => {
       const $targetRound = $baseTarget.find('.round-type');
       if(!$target.length || !$me.length) return;
 
-      // console.log($me, $baseTarget, $target);
-
       const meBounds = $me[0].getBoundingClientRect();
       const targetBounds = $target[0].getBoundingClientRect();
 
@@ -53,9 +93,9 @@ site.directive('drawTo', ($timeout) => {
 
       const mid = x1+((x2-x1)/xModifier) - subModifier;
 
-      drawLine(x1, y1, mid, y1);
-      drawLine(mid, y1, mid, y2);
-      drawLine(mid, y2, x2, y2);
+      const adjustment = ~~drawLine(mid, y1, mid, y2);
+      drawLine(x1, y1, mid+adjustment, y1);
+      drawLine(mid+adjustment, y2, x2, y2);
 
     }, 0);
   };