New API doc - typos and questions

Jim

Any ideas about a way to mark these errors as they are fixed?

thomthom

We could ask Coen to have a strike-through BBC tag. [deleted] or something...
Or, put a big fat FIXED in bold green. Might be easier.

avariant

From Image.explode:

entitiesarray     an Array object of entities if successful

which I interpret as the face and edges created in the explosion. However, the result from my testing always returns an empty array.

thomthom

I see that behaviour as well: http://www.sketchucation.com/forums/scf/viewtopic.php?f=11&t=17150&hilit=+explode

Chris Fullmer

http://code.google.com/apis/sketchup/docs/ourdoc/polygonmesh.html#add_polygon

add polygon detail just repeats the add_point example.

scottlininger

@thomthom said:

Seems that InstanceObserver.onClose is dead as well. http://code.google.com/apis/sketchup/docs/ourdoc/instanceobserver.html#onClose
But onOpen works.

Thomas,

I'm about to push an update with all of the recent changes above. This one puzzled me, though. The ToolsObserver examples work fine for me on the onOpen and onClose. Can you clarify what "dead as well" means?

@unknownuser said:

Does this list work 'across-platforms' - my understanding was that PCs and Macs differ ?

Tig, these action codes only work on the PC, but I still think I'll add them to the docs. There are some cool things in there that PC-only shops might need.

Thanks, guys! New update should be live in the next day or so.

scottlininger

@thomthom said:

We could ask Coen to have a strike-through BBC tag. [deleted] or something...
Or, put a big fat FIXED in bold green. Might be easier.

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

thomthom

@unknownuser said:

I'm about to push an update with all of the recent changes above. This one puzzled me, though. The ToolsObserver examples work fine for me on the onOpen and onClose. Can you clarify what "dead as well" means?

Dead as in: I never see it trigger.

thomthom

ToolObservers? How are they related to onClose and onOpen?

remus

@unknownuser said:

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

I'll ask gai if he can make you a mod for the ruby forum

scottlininger

@thomthom said:

ToolObservers? How are they related to onClose and onOpen?

I meant to say, InstanceObservers.

In any case, they fire fine for me.

Jim

@remus said:

@unknownuser said:

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

I'll ask gai if he can make you a mod for the ruby forum

Scott, there's no way to give edit-rights on a single topic. So you get edit rights to the whole Ruby Forum.

If you have a list of changes, we can take care of marking them fixed. I'd hate to see you spend time doing it if you have more important work to do.

Gaieus

Hi Scott, I have sent you a PM, too, about it. The mods have also be informed to welcome you in the team.

thomthom

@unknownuser said:

@thomthom said:

ToolObservers? How are they related to onClose and onOpen?

I meant to say, InstanceObservers.

In any case, they fire fine for me.

Hmm... When I tried to implement it in my plugin the onClose never fired. I also tried Jim's observer test script: http://sketchuptips.blogspot.com/2008/03/plugin-oberver-tests.html - and that never fired either. Could you post a working sample where onClose fires? (The onOpen always works though.)

scottlininger

@thomthom said:

Could you post a working sample where onClose fires?

Give this a go... Could be that we're seeing a sneaky bug...

class MyInstanceObserver < Sketchup;;InstanceObserver
   def onOpen(instance)
     UI.messagebox("onOpen; " + instance.to_s)
   end

   def onClose(instance)
     UI.messagebox("onClose; " + instance.to_s)
   end
end

model = Sketchup.active_model
model.definitions[0].instances[0].add_observer(MyInstanceObserver.new)

thomthom

Nopes. Doesn't work.

Jim

Works for me...

thomthom

Scott just found out that it works if the script loads when SU loads. But if you put it a subfolder for instance, and then manually load it, it won't work.
Jim's Observer tests never fire the event in SU7. However, in SU6 it works fine.

Jim

Just wondering if you have Dynamic Components on, and what happens if you turn them off?

thomthom

@jim said:

Just wondering if you have Dynamic Components on, and what happens if you turn them off?

Well I'll be! After disabling DCs the onClose even works!

btw, the DefinitionObserver works in SU6. But not SU7. Noticed that when I tested this observer in SU6.