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

Chiusa
  • Qualcosa non va?
#184 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
50/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp, unity
Ambito
game-dev

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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).

Lingua principale
C#
Stelle
135
Fork
15
Merge medio
24m
PR unite (30g)
112

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di RadicalCSG/com.chisel

Tutte le issue di RadicalCSG/com.chisel

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.