fix(cornelius): Side-step path splitting issue
There's an edge-case in Cornelius where things fall apart when a path is split exactly on the endpoint of a curve that is part of said path. In a perfect world, core would handle this in a more robust way. Until that time, this implements a workaround by checking for this edge-case. Closes #934
This commit is contained in:
Joost De Cock
parent
9e1104690f
commit
a779b3a539
3 changed files with 313 additions and 1 deletions
|
|
@ -24,6 +24,8 @@
|
||||||
- Crotch- and Cross seam have been redrawn using the new angle options
|
- Crotch- and Cross seam have been redrawn using the new angle options
|
||||||
|
|
||||||
Fixed:
|
Fixed:
|
||||||
|
cornelius:
|
||||||
|
- Handle edge-case where splitting a path on and endpoint causes things to break
|
||||||
titan:
|
titan:
|
||||||
- Adapt seat control point when waist is dropped below the hip line
|
- Adapt seat control point when waist is dropped below the hip line
|
||||||
- Removed lingering console.log statements
|
- Removed lingering console.log statements
|
||||||
|
|
|
||||||
303
packages/cornelius/example/src/layout.css
Normal file
303
packages/cornelius/example/src/layout.css
Normal file
|
|
@ -0,0 +1,303 @@
|
||||||
|
div.layout-wrapper {
|
||||||
|
width: 100%;
|
||||||
|
margin: 0;
|
||||||
|
padding: 0;
|
||||||
|
background-color: red;
|
||||||
|
background: #f8f9fa;
|
||||||
|
background: linear-gradient(90deg, #f1f3f5 0%, #f1f3f5 25%, #f8f9fa 26%, #f8f9fa 100%);
|
||||||
|
}
|
||||||
|
div.layout-wrapper div.layout {
|
||||||
|
display: flex;
|
||||||
|
max-width: 1600px;
|
||||||
|
margin: auto;
|
||||||
|
padding: 0;
|
||||||
|
flex-direction: row;
|
||||||
|
flex-wrap: nowrap;
|
||||||
|
justify-content: space-between;
|
||||||
|
background-color: #f8f9fa;
|
||||||
|
min-height: calc(100vh - 64px);
|
||||||
|
}
|
||||||
|
div.layout-wrapper div.layout > aside {
|
||||||
|
width: 33%;
|
||||||
|
background: #f1f3f5;
|
||||||
|
border-right: 2px solid #dee2e6;
|
||||||
|
}
|
||||||
|
div.layout-wrapper div.layout > section {
|
||||||
|
margin: 0;
|
||||||
|
padding: 1rem;
|
||||||
|
}
|
||||||
|
div.layout-wrapper div.layout > section > div.content {
|
||||||
|
max-width: 66ch;
|
||||||
|
min-width: 340px;
|
||||||
|
}
|
||||||
|
div.layout-wrapper div.layout > section > div.content.wide {
|
||||||
|
max-width: 100%;
|
||||||
|
margin: auto;
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.dark header {
|
||||||
|
background-color: #1a1d21;
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.dark div.layout-wrapper {
|
||||||
|
background: #f8f9fa;
|
||||||
|
background: linear-gradient(90deg, #1a1d21 0%, #1a1d21 25%, #212529 26%, #212529 100%);
|
||||||
|
}
|
||||||
|
.theme-wrapper.dark div.layout-wrapper div.layout {
|
||||||
|
background-color: #212529;
|
||||||
|
}
|
||||||
|
.theme-wrapper.dark div.layout-wrapper div.layout > aside {
|
||||||
|
background-color: #1a1d21;
|
||||||
|
border-right: 2px solid #343a40;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a svg {
|
||||||
|
color: #ced4da;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a:first-of-type svg {
|
||||||
|
color: #f8f9fa;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a:hover svg {
|
||||||
|
color: #b197fc;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a span,
|
||||||
|
header button span {
|
||||||
|
color: #ced4da;
|
||||||
|
}
|
||||||
|
header a span svg,
|
||||||
|
header button span svg {
|
||||||
|
color: #dee2e6;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a:hover span,
|
||||||
|
header button:hover span {
|
||||||
|
color: #f8f9fa;
|
||||||
|
}
|
||||||
|
header a:hover span svg,
|
||||||
|
header button:hover span svg {
|
||||||
|
color: #b197fc;
|
||||||
|
}
|
||||||
|
|
||||||
|
header a,
|
||||||
|
header button {
|
||||||
|
padding: 0 1vw !important;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* monitor */
|
||||||
|
@media (min-width: 1200px) {
|
||||||
|
div.layout > section {
|
||||||
|
width: 63%;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/* slate */
|
||||||
|
@media (max-width: 1199px) and (min-width: 960px) {
|
||||||
|
div.layout > aside {
|
||||||
|
width: 298px;
|
||||||
|
}
|
||||||
|
div.layout > section {
|
||||||
|
width: calc(100% - 300px - 4rem);
|
||||||
|
max-width: none;
|
||||||
|
margin: 0 1rem 0 3rem;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/* tablet */
|
||||||
|
@media (max-width: 959px) {
|
||||||
|
div.layout > aside {
|
||||||
|
width: 218px;
|
||||||
|
}
|
||||||
|
div.layout > section {
|
||||||
|
width: calc(100% - 220px - 4rem);
|
||||||
|
max-width: none;
|
||||||
|
margin: 0;
|
||||||
|
padding: 0 2rem;
|
||||||
|
}
|
||||||
|
div.layout > section div.content {
|
||||||
|
min-width: inherit;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/* mobile */
|
||||||
|
@media (max-width: 599px) {
|
||||||
|
div.layout > aside {
|
||||||
|
display: none;
|
||||||
|
}
|
||||||
|
div.layout > section {
|
||||||
|
width: calc(100%);
|
||||||
|
margin: 0 auto;
|
||||||
|
padding: 0 1.5rem;
|
||||||
|
max-width: none;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
div.gatsby-highlight {
|
||||||
|
margin-bottom: 1rem;
|
||||||
|
}
|
||||||
|
|
||||||
|
@media (max-width: 599px) {
|
||||||
|
#mobile-menu {
|
||||||
|
position: fixed;
|
||||||
|
top: 0;
|
||||||
|
left: 0;
|
||||||
|
width: 100%;
|
||||||
|
height: 100vh;
|
||||||
|
padding: 0 0 1rem;
|
||||||
|
max-width: 600px;
|
||||||
|
z-index: -10;
|
||||||
|
transition: opacity 0.25s ease 0s;
|
||||||
|
opacity: 0;
|
||||||
|
overflow: scroll;
|
||||||
|
}
|
||||||
|
#mobile-menu > ul,
|
||||||
|
#mobile-menu > div {
|
||||||
|
transform: translate(0px, 10px);
|
||||||
|
transition: transform 0.25s ease 0s;
|
||||||
|
}
|
||||||
|
.theme-wrapper.show-menu #mobile-menu {
|
||||||
|
opacity: 1;
|
||||||
|
z-index: 10;
|
||||||
|
}
|
||||||
|
.theme-wrapper.show-menu #mobile-menu > div {
|
||||||
|
transform: translate(0px, 0px);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.light div.draft-ui-menu,
|
||||||
|
.theme-wrapper.light div.menu {
|
||||||
|
background: #f1f3f5;
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.dark div.draft-ui-menu,
|
||||||
|
.theme-wrapper.dark div.menu {
|
||||||
|
background: #343a40;
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.show-menu div.menu {
|
||||||
|
opacity: 1;
|
||||||
|
z-index: 10;
|
||||||
|
}
|
||||||
|
.theme-wrapper.show-menu div.menu > div {
|
||||||
|
transform: translate(0px, 0px);
|
||||||
|
}
|
||||||
|
|
||||||
|
div.spaced-buttons > button {
|
||||||
|
margin: 0 0.5rem 0.5rem 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
div.spaced > * {
|
||||||
|
margin: 0 0.5rem 0.5rem 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
ul#pre-main-menu {
|
||||||
|
margin: 0;
|
||||||
|
padding: 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.boldish {
|
||||||
|
font-weight: 500;
|
||||||
|
}
|
||||||
|
|
||||||
|
.freesewing.draft {
|
||||||
|
padding: 1rem;
|
||||||
|
}
|
||||||
|
|
||||||
|
li.action {
|
||||||
|
clear: both;
|
||||||
|
}
|
||||||
|
|
||||||
|
li.action span.MuiSwitch-root {
|
||||||
|
float: right;
|
||||||
|
}
|
||||||
|
|
||||||
|
.theme-wrapper.light ul#draft-config li.action.toggle.off,
|
||||||
|
.theme-wrapper.dark ul#draft-config li.action.toggle.off {
|
||||||
|
color: #868e96;
|
||||||
|
}
|
||||||
|
.theme-wrapper.light ul#draft-config li.action.toggle.off > span svg,
|
||||||
|
.theme-wrapper.dark ul#draft-config li.action.toggle.off > span svg {
|
||||||
|
color: #868e96;
|
||||||
|
}
|
||||||
|
|
||||||
|
footer {
|
||||||
|
background-color: #1a1d21;
|
||||||
|
color: #adb5bd;
|
||||||
|
padding: 3rem 0 6rem;
|
||||||
|
}
|
||||||
|
footer a {
|
||||||
|
color: #dee2e6 !important;
|
||||||
|
font-weight: 400;
|
||||||
|
}
|
||||||
|
footer a:hover {
|
||||||
|
color: #d0bfff !important;
|
||||||
|
}
|
||||||
|
footer div.cols {
|
||||||
|
display: flex;
|
||||||
|
flex-direction: row;
|
||||||
|
justify-content: space-between;
|
||||||
|
max-width: 1600px;
|
||||||
|
margin: auto;
|
||||||
|
padding: 0 1.5rem;
|
||||||
|
}
|
||||||
|
footer div.cols > div {
|
||||||
|
min-width: 150px;
|
||||||
|
max-width: calc(20% - 4rem);
|
||||||
|
padding: 0 2rem 0 0;
|
||||||
|
width: 100%;
|
||||||
|
}
|
||||||
|
footer ul {
|
||||||
|
text-align: left;
|
||||||
|
font-size: 1.1rem;
|
||||||
|
margin: 0;
|
||||||
|
padding: 0;
|
||||||
|
width: 100%;
|
||||||
|
}
|
||||||
|
footer ul li:first-of-type {
|
||||||
|
padding: 0.35rem 0.75rem;
|
||||||
|
}
|
||||||
|
footer ul li {
|
||||||
|
display: block;
|
||||||
|
}
|
||||||
|
footer ul li a:hover {
|
||||||
|
text-decoration: none !important;
|
||||||
|
}
|
||||||
|
footer ul li.heading {
|
||||||
|
font-weight: bold;
|
||||||
|
border-bottom: 3px solid #adb5bd;
|
||||||
|
margin-bottom: 0.5rem;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* XL screens */
|
||||||
|
@media (min-width: 1200px) {
|
||||||
|
footer div.cols > div:last-of-type {
|
||||||
|
min-width: 350px;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/* SM screens */
|
||||||
|
@media (min-width: 600px) and (max-width: 959px) {
|
||||||
|
footer div.cols {
|
||||||
|
flex-wrap: wrap;
|
||||||
|
}
|
||||||
|
footer div.cols > div {
|
||||||
|
width: calc(30% - 4rem);
|
||||||
|
padding: 0 1rem;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/* XS screens */
|
||||||
|
@media (max-width: 599px) {
|
||||||
|
footer div.cols {
|
||||||
|
display: block;
|
||||||
|
}
|
||||||
|
footer div.cols > div {
|
||||||
|
margin: 2rem auto 0;
|
||||||
|
max-width: calc(100% - 4rem);
|
||||||
|
}
|
||||||
|
footer div.cols > div:first-of-type {
|
||||||
|
margin-top: 0;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
@ -22,7 +22,14 @@ export default function (part) {
|
||||||
points.pocketFacingBR = paths.sideSeam.shiftAlong(
|
points.pocketFacingBR = paths.sideSeam.shiftAlong(
|
||||||
paths.sideSeam.length() - (waist / 2 / 4.5) * 3.5 - halfInch * 3 + sa
|
paths.sideSeam.length() - (waist / 2 / 4.5) * 3.5 - halfInch * 3 + sa
|
||||||
)
|
)
|
||||||
paths.sideSeam = paths.sideSeam.split(points.pocketFacingBR)[1].setRender(false)
|
// If we split at the exact point where the curves join, things fall apart
|
||||||
|
// So let's work around that until this is handled in core
|
||||||
|
if (points.pocketFacingBR.sitsOn(points.pAextra))
|
||||||
|
paths.sideSeam = new Path()
|
||||||
|
.move(points.pAextra)
|
||||||
|
.curve(points.pAextraCPu, points.pUcpA, points.pU)
|
||||||
|
.setRender(false)
|
||||||
|
else paths.sideSeam = paths.sideSeam.split(points.pocketFacingBR)[1].setRender(false)
|
||||||
|
|
||||||
points.brCPtl = points.pocketFacingBR.shift(
|
points.brCPtl = points.pocketFacingBR.shift(
|
||||||
points.pocketFacingBR.angle(points.pocketSide) + 90,
|
points.pocketFacingBR.angle(points.pocketSide) + 90,
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue