Fix box: zero size, and dragging a side past its opposite

Đã đóng
  • Có gì không ổn?
#184 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
50/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp, unity
Lĩnh vực
game-dev

Hướng nghiên cứu

The payload names no files, so start by finding the box generator in the Chisel codebase and the check that raises the brushMeshID error. Reproduce it in the Unity editor by sizing a box up and back down to zero height. Done means three things: a zero-size box cancels creation, dragging a side past its opposite gives a negative size with the sides swapped under the hood, and the brush stays valid.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Problems when drawing a box:

  1. Error at zero size while sizing. While building the outline (before the shape is committed), sizing the box upwards and then back downwards until its height is zero gives an error:

    Brush with ID (Chisel.Core.CompactNodeID) has its brushMeshID set to (0), which is invalid.
The brushMeshID is invalid: a Generator created/didn't update a TreeBrush correctly.

  2. A zero-sized box can be made. Finishing with a zero size should cancel the creation instead of making the box.

  3. Dragging a side past its opposite moves the whole box. When a side is moved beyond the opposite side, the opposite side starts following the current position, so the entire box moves. Instead, the opposite side stays where it is and the box gets a negative size; the generator swaps its sides under the hood so the brush is valid.

Rule for all generators: negative sizes are allowed, and every generator builds a valid shape from them (swapping sides under the hood, so the brush is the right way out). A zero size is never allowed: creating at zero size cancels, and an existing generator can't be made zero sized.

Related: capsule (#185), cylinder (#186), inspector values (#187), hemisphere (#188), sphere (#189), extruded shape (#190), linear stairs (#191), spiral stairs (#192).

Ngôn ngữ chính
C#
Star
135
Fork
15
Merge trung bình
24 phút
Pull request đã merge (30 ngày)
112

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của RadicalCSG/com.chisel

Tất cả issue của RadicalCSG/com.chisel

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.