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

Cerrado
  • ¿Algo no está bien?
#184 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
50/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
csharp, unity
Área
game-dev

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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

Lenguaje dominante
C#
Estrellas
135
Forks
15
Merge medio
24 min
PR fusionados (30 d)
112

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de RadicalCSG/com.chisel

Todos los issues de RadicalCSG/com.chisel

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.