Skip to content

Conversation

@DSCaskey
Copy link
Contributor

The PR does the following:

  • Extends the bounding rect of a pattern piece to the cutline path if a seam allawance exists, otherwise the bounding rect remain the main path of the piece.

  • Allows selection of a pattern piece by including the seam allowance area as part of the selection target area.

  • Allows the Grainline to extend into the seam allowance upto the cutline.

Screenshot 2025-10-22 083404
  • Allows the text of labels to appear within the seam allowance.
Screenshot 2025-10-22 084215

Closes issue #1418

@what-the-diff
Copy link

what-the-diff bot commented Oct 23, 2025

🌟 PR Summary 🌟

Hello Team! Here's a brief, easy-to-understand summary of the latest pull request, making our project even better! 🌈

  • Improvement of m_pieceRect Setting Process 🎯
    We've enhanced the way m_pieceRect is set up. Now, it precisely captures the bounding rectangle of m_cutLine right after we modify its path.

  • Elimination of Duplicate m_pieceRect Settings 🚫🔁
    We've smartly removed a repetitive setting process where m_pieceRect was previously getting assigned twice within the same function.

All these changes makes our code cleaner and more efficient! 🚀 Keep the good work team! 👏

@slspencer slspencer merged commit ffd37d0 into develop Oct 23, 2025
12 checks passed
@slspencer slspencer deleted the extend-pattern-piece-bounding-rect branch October 23, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment