[Plugin] TIG-Smart_offset

pilou

Yeeeeeeees! Works like a charm now!
You are the Ruby's king!

And lucky because I can't more test your tricky plugin before the next year!

TIG

@jim said:

in cmd.set_validation_proc (and other places) you want to give the full scope of the suitable? method since it is being called from SketchUp outside the scope of your module.

Also, combine the MF_ bits.


> cmd=UI;;Command.new('TIG.Smart_offset'){self.dialog()}
> if TIG;;Smart_offsetter.suitable?()
> MF_ENABLED
> else
> MF_DISABLED | MF_GRAYED
> end
>

Thanks, knew it was obvious... This only affected MACs though ???
This was collecting dust on a virtual dust, until Pilou reminded me that I'd written it, so sometimes things get 'forgotten'...

TIG

Here's v1.2 http://sketchucation.com/forums/viewtopic.php?p=446462#p446462
Further robustness in enabling/disabling/graying and suitable? tests [thanks to Jim!].
Seems to be especially for MACs

Jim

@tig said:

This only affected MACs though ???

I am not sure - it was just a guess and I don't have a Mac. Did you try it?

TIG

I don't have a MAC either.
Driven's reported issue was on a MAC and Jeff had other issues on a MAC too...
Hopefully this recent v1.2 [...20121221b.rbz !] will address the suitable? check and MF_ issues with the menus/toolbar...
It still works fine on a PC, but then v1.1 did too

elmer

Nice Christmas gift Tig, thanks a lot, it works perfectly on my Mac.

alvis

Thanx TIG

guanjin

@tig said:

Here's v1.2 http://sketchucation.com/forums/viewtopic.php?p=446462#p446462
Further robustness in enabling/disabling/graying and suitable? tests [thanks to Jim!].
Seems to be especially for MACs

TIG: Merry Christmas

jeff hammond

@tig said:

Here's v1.2 http://sketchucation.com/forums/viewtopic.php?p=446462#p446462
Further robustness in enabling/disabling/graying and suitable? tests [thanks to Jim!].
Seems to be especially for MACs

Hi Tig
i'm still getting the same greyed out problem.
is there anything i can type in the console or ?? to help figure out what the problem is?
thanks

TIG

Is there no difference at all - if you have nothing preselected, or one edge preselected, or one face preselected, or lots of things [including some edges] preselected ?
The menu/toolbar are only enabled if there's at least one face in the current selection.
The context-menu only appears if there's at least one face in the current selection.

Copy/paste+<enter> these into the Ruby Console and report the results:

with nothing preselected TIG::Smart_offsetter.suitable?()
with one edge preselected TIG::Smart_offsetter.suitable?()
with one face preselected TIG::Smart_offsetter.suitable?()
with lots of things [including some edges] preselected TIG::Smart_offsetter.suitable?()

with one edge preselected TIG::Smart_offsetter.dialog()
with one face preselected TIG::Smart_offsetter.dialog()

jeff hammond

oops
it actually is working now..
it's just the toolbar that remains greyed out at all times but the menu item becomes available once an appropriate selection has been made..

(which, for me personally, is fine.. i more likely than not wont want the toolbar anyway.. i think id use it with a modifier key added to my offset shortcut)

jeff hammond

here are these results.. (since you took the time to write them.. i better take the time to report )

@tig said:

Copy/paste+<enter> these into the Ruby Console and report the results:

with nothing preselected TIG::Smart_offsetter.suitable?()
false

with one edge preselected TIG::Smart_offsetter.suitable?()
false

with one face preselected TIG::Smart_offsetter.suitable?()
true

with lots of things [including some edges] preselected TIG::Smart_offsetter.suitable?()
true (drew two squares and selected all)

with one edge preselected TIG::Smart_offsetter.dialog()
warning- PreSelect Face[s !]
ruby - nil

with one face preselected TIG::Smart_offsetter.dialog()
(works as expected)

jeff hammond

uh.. not sure what the deal is but..

upon entering all of that stuff in the console, the toolbar is now working

[edit] but then.. i quit sketchup then relaunched and the toolbar is greyed out again regardless of selection.. the menu item is working though

Chris Fullmer

voodoo. definitely voodoo at work here.

jeff hammond

@chris fullmer said:

voodoo. definitely voodoo at work here.

ha.. that darn voodoo!

some more info:

so here's what the deal with the toolbar is.. (in this example, i have 3 squares with faces drawn)

•cmmdA (select all)
•toolbar remains greyed out (menu item is live)
•do something else (rotate, pan, etc)
•toolbar becomes active
•deselect all
•toolbar remains active (menu item greys out)
•do something else (switch tool, orbit, etc)
•toolbar greys out

so it appears, on mac, that something has to happen other than simply selecting in order to jump start the toolbar into updating it's state

ccbiggs

Hey Tig,
I am getting the same thing Jeff has. But since Jeff actually knows what he is doing, he can do the ruby troubleshooting. Will stay tuned to see what you fellas figure out.
It works great on my windows machine at home.
Thanks Tig

Update
Jeff says his works. It does not work on my mac.
with nothing preselected TIG::Smart_offsetter.suitable?()

TIG::Smart_offsetter.suitable?()
false

with one edge preselected TIG::Smart_offsetter.suitable?()

TIG::Smart_offsetter.suitable?()
false

with one face preselected TIG::Smart_offsetter.suitable?()

TIG::Smart_offsetter.suitable?()
true

with lots of things [including some edges] preselected TIG::Smart_offsetter.suitable?()

TIG::Smart_offsetter.suitable?()
true

with one edge preselected TIG::Smart_offsetter.dialog()
TIG.Smart_offset:

PreSelect Face[s] !

with one face preselected TIG::Smart_offsetter.dialog()
Offset [+/-ve=out/

My drop down "Tools" Menu works with a face preselected. The toolbar does not.
Nothing works with just edges selected.
If you need me to check anything else, and I do not get right back to ya, I am not ignoring you I am just here there and everywhere today.

TIG

Here's v1.3 http://sketchucation.com/forums/viewtopic.php?p=446462#p446462
Removed 'suitable?' method, to suit MACs, more error traps etc.
Try this and report back...

ccbiggs

It works the same for me. With a face selected the drop down tools menu item will work but the single icon toolbar will not.

TIG

@ccbiggs said:

It works the same for me. With a face selected the drop down tools menu item will work but the single icon toolbar will not.
Which is weird... because they BOTH use 'cmd' and share the same enabled/grayed code...
To check that the toolbar WILL work somehow... can you please edit the file 'TIG-Smart_offset.rb' on your MAC with a plain text editor [BBedit???] and find the three lines starting at #117 reading:

		cmd.set_validation_proc{
			Sketchup.active_model.selection.grep(Sketchup;;Face)[0] ? MF_ENABLED ; MF_GRAYED
		}

and change them to read

		#cmd.set_validation_proc{
			#Sketchup.active_model.selection.grep(Sketchup;;Face)[0] ? MF_ENABLED ; MF_GRAYED
		#}

i.e. add a # in front of each of the three lines to disable them.
Now the tool's command should always be 'enabled' in the menu & the toolbar button, if you try to use it without a Face in the Selection you'll get an error messagebox...
Let me know how this goes...

Incidentally, BEFORE [& after] you make the changes... do you get the tool's context-menu item IF there's a Face in the Selection ?

jeff hammond

@tig said:

To check that the toolbar WILL work somehow... can you please edit the file 'TIG-Smart_offset.rb' on your MAC with a plain text editor [BBedit???] and find the three lines starting at #117 reading:

		cmd.set_validation_proc{
> 			Sketchup.active_model.selection.grep(Sketchup;;Face)[0] ? MF_ENABLED ; MF_GRAYED
> 		}

and change them to read

		#cmd.set_validation_proc{
> 			#Sketchup.active_model.selection.grep(Sketchup;;Face)[0] ? MF_ENABLED ; MF_GRAYED
> 		#}

i.e. add a # in front of each of the three lines to disable them.

that does the trick.. the toolbar is always active now and i'll just receive an error if i try to run the tool without an appropriate selection..

[context menu works also.. but it worked prior to disabling the above lines]