From b9700f3ee9b561a90d2d11dcf43730676e9a1990 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Tue, 17 Oct 2017 23:26:12 +0100
Subject: [PATCH] poster-tube-lid: sort out StrapHoop

---
 poster-tube-lid.scad | 45 +++++++++++++++++++++++++-------------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/poster-tube-lid.scad b/poster-tube-lid.scad
index 659a023..ee04e76 100644
--- a/poster-tube-lid.scad
+++ b/poster-tube-lid.scad
@@ -55,6 +55,7 @@ jig_mark = 5;
 strap_hoop_thick = 6;
 strap_hoop_inside = 10;
 strap_hoop_strlen = 10;
+strap_hoop_elevation = 45;
 
 // calculated
 
@@ -173,23 +174,29 @@ module StrapHoopProfile(){
 
 module StrapHoop(){ ////toplevel
   bigrad = strap_hoop_inside + strap_hoop_thick/2;
-  extralen = strap_hoop_thick;
-
-  for (x= [ -1, +1 ] * bigrad) {
-    translate([x, -extralen, 0])
-      rotate([-90,0,0])
-      linear_extrude(height= extralen + strap_hoop_strlen + 0.1,
-		     convexity=10)
-      StrapHoopProfile();
-  }
-  translate([0, strap_hoop_strlen, 0]){
-    intersection(){
-      rotate_extrude(convexity=10)
-	translate([bigrad, 0,0])
-	StrapHoopProfile();
-      translate([0,50,0])
-	cube([100,100,100], center=true);
+  extralen = strap_hoop_thick * 5;
+
+  intersection(){
+    rotate([strap_hoop_elevation, 0,0]){
+      for (x= [ -1, +1 ] * bigrad) {
+	translate([x, -extralen, 0])
+	  rotate([-90,0,0])
+	  linear_extrude(height= extralen + strap_hoop_strlen + 0.1,
+			 convexity=10)
+	  StrapHoopProfile();
+      }
+      translate([0, strap_hoop_strlen, 0]){
+	intersection(){
+	  rotate_extrude(convexity=10)
+	    translate([bigrad, 0,0])
+	    StrapHoopProfile();
+	  translate([0,50,0])
+	    cube([100,100,100], center=true);
+	}
+      }
     }
+    translate([0, 50, 0])
+      cube(100, center=true);
   }
 }
 
@@ -298,9 +305,9 @@ module Cover(){ ////toplevel
       }
     }
   }
-  translate([handling_dia/2, 0, -side_height])
-    rotate([0,0,-90])
-    rotate([-45, 0,0])
+  translate(strap_hoop_thick * [-0.5, 0, +1])
+    translate([handling_dia/2, 0, -side_height])
+    rotate([0,180,0]) rotate([0,0,90])
     StrapHoop();
 }
 
-- 
2.30.2