Merge pull request #90 from IliesAllali/fix/aplomb-sans-decrochage

Fil : plus de marche pour un enfant presque sous l'union

frontend/src/components/galaxy/renderers.js

@@ -101,9 +101,10 @@ function computeCouplePoints(p1Pos, p2Pos, unionPos, p1R, p2R, nodeRandomData, s
 export const FIL = { color: '#E3A590', width: 1.1, dot: '#D2694A', dotR: 2.6 }
 // Ligne de la fratrie : sous les prénoms et années des parents, au-dessus des enfants
 const BUS_DY = 100
-// Accolade : coins arrondis de rayon fixe (forme A, choisie par Patch), écart minimal en dessous duquel l'enfant descend droit
+// Accolade : coins arrondis de rayon fixe (forme A, choisie par Patch). En dessous de deux rayons d'écart,
+// deux coins ne tiennent pas sans faire une marche : l'enfant est relié par un trait droit depuis l'union.
 const SIB_R = 24
-const SIB_STRAIGHT_DX = 12
+const SIB_STRAIGHT_DX = SIB_R * 2
 
 function busY(sourceId, sy) {
   let h = 0
@@ -123,7 +124,14 @@ export function traceSibship(ctx, sourceId, sourcePos, items) {
   for (const { cx: ex, cy: ey } of items) {
     const dx = ex - sx
     ctx.moveTo(sx, sy)
-    if (Math.abs(dx) < SIB_STRAIGHT_DX) { ctx.lineTo(sx, by); ctx.lineTo(ex, by); ctx.lineTo(ex, ey); continue }
+    if (Math.abs(dx) < SIB_STRAIGHT_DX) {
+      // Le tronc descend jusqu'à la barre, puis un S doux, vertical au départ et à l'arrivée, rejoint l'enfant :
+      // il se sépare là où se séparent les autres enfants, pas sous l'union
+      const h = ey - by
+      ctx.lineTo(sx, by)
+      ctx.bezierCurveTo(sx, by + h * 0.55, ex, ey - h * 0.45, ex, ey)
+      continue
+    }
     const dir = Math.sign(dx)
     const k = Math.min(rr, Math.abs(dx) / 2)
     ctx.lineTo(sx, by - k)

frontend/src/components/galaxy/renderers.test.js

@@ -35,4 +35,17 @@ describe('traceSibship (accolade franche)', () => {
     traceSibship(ctx, 'u-1', src, [{ cx: 505, cy: 480 }])
     expect(count('arcTo')).toBe(0)
   })
+
+  it('un enfant à moins de deux rayons de l\'aplomb est relié par un seul S, sans marche', () => {
+    const { ctx, calls } = recorder()
+    traceSibship(ctx, 'u-1', src, [{ cx: 515, cy: 480 }])
+    expect(calls.filter(c => c[0] === 'arcTo').length).toBe(0)
+    // Tronc vertical jusqu'à la barre, puis le S
+    expect(calls.filter(c => c[0] === 'lineTo').map(c => c[1])).toEqual([500])
+    const [curve] = calls.filter(c => c[0] === 'bezierCurveTo')
+    // Départ vertical sous l'union, arrivée verticale sur l'enfant
+    expect(curve[1]).toBe(500)
+    expect(curve[3]).toBe(515)
+    expect(curve.slice(5)).toEqual([515, 480])
+  })
 })