api-issue-tracker icon indicating copy to clipboard operation
api-issue-tracker copied to clipboard

[Request] ComponentDefinition#manifold?

Open Fredosixx opened this issue 1 year ago • 2 comments

This method ComponentDefinition#manifold? does not exist and would make sense.

It would avoid to either rely on an existing instance of the definition, or worse, create an temporary instance of the definition, just to check if the definition is manifold.

Note that there are cases where we can manipulate a definition that does not have an instance (yet).

  • when creating geometry (cdef = Sketchup.active_model.definitions.add, then adding entities to cdef.entities)
  • importing a definition from a skp file (definition#load)

The existing Group#manifold? and ComponentInstance#manifold? can continue to exist.

Fredosixx avatar Aug 26 '24 12:08 Fredosixx

That's a good point, having to have an instance to make that query.

thomthom avatar Aug 27 '24 08:08 thomthom

Logged as: SKEXT-4287

sketchup[bot] avatar Sep 02 '24 06:09 sketchup[bot]