[Plugin] FloorGenerator ( Updated 6-Apr-2017)

sdmitch · 2 Jun 2014, 00:03

Hello Everyone !

First of all, thank you for the great job you did to bring us that tool.

However, I have an issue : I wanted to be able to run it through a toolbar button (which I successfully did, but when I close the FG window the button is still activated and if I try to run it again the FG window doesnt show up anymore (even if I use other tools after that).

Could you please help me to fix it ?

Just don't "X" out of the WebDialog window, instead Right click in space or select any other tool. This will close the WebDialog window an unselect the Toolbar Icon.

jolran · 2 Jun 2014, 07:28

@unknownuser said:

Just don't "X" out of the WebDialog window, instead Right click in space or select any other tool. This will close the WebDialog window an unselect the Toolbar Icon.

Sam. I think you could try trigger some actions during the closing of the dialog with "X", using @yourdialog.set_on_close { #set closing action here }. If you are not already using it, that is.
As usual concerning webdialogs, I don't know how reliable that method is..

unsuns · 2 Jun 2014, 11:36

I tried what you said jolran but I couldn't figure it out. How should I do it ?
Here is the code:

...

[mod=:3izmkj1d]Please don't post entire blocks of code from other authors - it's bad manners...
Extracts are fine...[/mod:3izmkj1d]

sdmitch · 2 Jun 2014, 12:00

@jolran said:

@unknownuser said:

Just don't "X" out of the WebDialog window, instead Right click in space or select any other tool. This will close the WebDialog window an unselect the Toolbar Icon.

Sam. I think you could try trigger some actions during the closing of the dialog with "X", using @yourdialog.set_on_close { #set closing action here }. If you are not already using it, that is.
As usual concerning webdialogs, I don't know how reliable that method is..

I wasn't. Another one of those things that is there but not a necessity so I just ignored it. Thanks for the suggestion.

jolran · 2 Jun 2014, 12:47

@unknownuser said:

I tried what you said jolran but I couldn't figure it out. How should I do it ?

unsuns, I meant this to be tried by Sam at developer-level. I havent dug in his script, so this suggestion was a shot in the dark.

@unknownuser said:

I wasn't. Another one of those things that is there but not a necessity so I just ignored it. Thanks for the suggestion.

Hope it helps.
/Joel

unsuns · 2 Jun 2014, 13:33

Can't I do it myself ? To help improve that script

sdmitch · 2 Jun 2014, 13:49

Posted new version.

Unsuns, I would appreciate it if you would let me take care of my plugins. Thank you.

unsuns · 2 Jun 2014, 14:04

That's alright ! Thank you !

unsuns · 2 Jun 2014, 18:40

Did you test it ?
Now when I select a pattern it closes the window, so I have to reopen to adjust settings and if I select another pattern it's the same.

guanjin · 3 Jun 2014, 14:16

@sdmitch said:

Posted new version.

Unsuns, I would appreciate it if you would let me take care of my plugins. Thank you.

Each pattern, cannot directly activate command, to repeat the icon to perform。

sdmitch · 3 Jun 2014, 15:06

@unsuns said:

Did you test it ?
Now when I select a pattern it closes the window, so I have to reopen to adjust settings and if I select another pattern it's the same.

Apparently not enough!

The .set_on_close had unintended consequences and has been removed. Although it did seem to fix the problem of X'ing out, it, for some bazaar reason, was triggered when the pattern was changed. My bad

unsuns · 3 Jun 2014, 17:18

The X-ing problem is back again !

sdmitch · 3 Jun 2014, 17:52

@unsuns said:

The X-ing problem is back again !

Yes I know and it will be until I can figure out why changing the pattern selection causes the window to close.

Why do you insist on X'ing out of the window? If it is in your way then minimize it.

sdmitch · 3 Jun 2014, 18:32

@jolran said:

Sam I'm looking at your code here. Cant remember which version this is. So you may have changed this behavior.
But in a callback you have a dialog close ?

You have a select with an event onchange:

<select onChange='patternchanged(value)'>

The function then sends the callback

function patternchanged(value){
>    window.location='skp;PatternChanged@'+value;
> }

In the callback block you have a dlg.close. So everytime select change dialog closes..

@@dlg.add_action_callback("PatternChanged") {|d,p|
> 	@@opt=p; puts p
> 	if @@opt=="Reset" then 
> 						["Brick","Tile","Wood","Tweed","Hbone","BsktWv","HpScth","Hexgon","Octgon","Two2One" ,"IrPoly"].each{|o| 
>   Sketchup.write_default("FloorGenerator",o,nil)};@@opt="Tile";
>   end
>   @@dlg.close; @@dlg=nil; self.dialog
> }

I also see you have the command self.dialog trying to reopend the dialog ?
But it is set to nil before. Don't know if that messes things up..
Maybe a timing issue.

Edit: sorry meant to post this in the devlopper forum where you posted specific question. Copied and pasted wrong...

Yes I finally figured that all out. I just had to add a variable that I set to true if the PatternChanged and the window was closed and recreated. I deleted the post to the Developer forum as soon as I figured it out so as not to embarrass my self further.

jolran · 3 Jun 2014, 18:33

Sam I'm looking at your code here. Cant remember which version this is. So you may have changed this behavior.
But in a callback you have a dialog close ?

You have a select with an event onchange:

<select onChange='patternchanged(value)'>

The function then sends the callback

function patternchanged(value){
   window.location='skp;PatternChanged@'+value;
}

In the callback block you have a dlg.close. So everytime select change dialog closes..

@@dlg.add_action_callback("PatternChanged") {|d,p|
	@@opt=p; puts p
	if @@opt=="Reset" then 
						["Brick","Tile","Wood","Tweed","Hbone","BsktWv","HpScth","Hexgon","Octgon","Two2One" ,"IrPoly"].each{|o| 
  Sketchup.write_default("FloorGenerator",o,nil)};@@opt="Tile";
  end
  @@dlg.close; @@dlg=nil; self.dialog
}

I also see you have the command self.dialog trying to reopend the dialog ?
But it is set to nil before. Don't know if that messes things up..
Maybe a timing issue ?

jolran · 3 Jun 2014, 18:36

Ha ha ! Ah ok!

I thought I was getting high or something. One minute I thought I read about it in the developper section, then it was gone. So I removed my last sentence as well, just in case I was wrong

stagelight · 8 Jun 2014, 23:07

Thank you for another great plugin !!
This is what I've been working on lately, with your script.

sdmitch · 9 Jun 2014, 00:16

@stagelight said:

Thank you for another great plugin !!
This is what I've been working on lately, with your script.

It's a rather funny floor but a very nice bridge.

djrique · 10 Jun 2014, 12:35

Thank you guy for that great plugin. I'm a brazilian and translated it for portuguese. Do you think that it's good?

sdmitch · 10 Jun 2014, 14:18

@djrique said:

Thank you guy for that great plugin. I'm a brazilian and translated it for portuguese. Do you think that it's good?

Good by me.