bartbutenaers / node-red-contrib-blockly

A Node Red node for visual programming a function using Blockly
Apache License 2.0
88 stars 22 forks source link

Request for testing #102

Closed bartbutenaers closed 10 months ago

bartbutenaers commented 11 months ago

Hi @cymplecy,

I wanted to avoid that you needed to beg me (again) for a new version of this node, during the next four christmas evenings ;-)

So I have quickly created a new 2.2.0 version of this node, that contains the following changes:

  1. Updated the Blockly version "6.x to 10.x".

  2. Fixed some migration issues (with some help from the Blockly community).

  3. Added a "Search" category, to allow you to search for blocks in the toolbox:

    image

    TODO: This is implemented via the (recently released) "toolbox-search" plugin. But whatever I enter in this field, no matches are displayed. I have asked for help in the Blockly forum.

  4. In the config node you can now specify whether you want to see a minimap:

    image

    I show it only when the workspace is extended (to full-screen) because otherwise it overlaps with the other icons. In extended mode it looks like this:

    image

It is all in the "maybe-later-maybe-never" branch, which you can install like this:

npm install bartbutenaers/node-red-contrib-blockly#maybe-later-maybe-never

Would be nice if you (or others of course) could play with this. Because I have only tested a few scenario's due to lack of time. But since we have skipped a large amount of (technical refactoring) Blockly releases, I expect other things will need to be fixed in our node...

Of course, there is no need to hurry with this!!!

Bart

cymplecy commented 11 months ago

Hi Bart Great work :) Will try it out in my Windows test instance :) Simon

On Sun, 10 Sept 2023 at 22:56, bartbutenaers @.***> wrote:

Hi @cymplecy https://github.com/cymplecy,

I wanted to avoid that you needed to beg me (again) for a new version of this node, during the next four christmas evenings ;-)

So I have quickly created a new 2.2.0 version of this node, that contains the following changes:

1.

Updated the Blockly version "6.x to 10.x". 2.

Fixed some migration issues (with some help from the Blockly community). 3.

Added a "Search" category, to allow you to search for blocks in the toolbox:

[image: image] https://user-images.githubusercontent.com/14224149/266869857-314bde66-fea7-4a12-be3a-9dc43c2de223.png

TODO: This is implemented via the (recently released) "toolbox-search" plugin. But whatever I enter in this field, no matches are displayed. I have asked for help in the Blockly forum https://groups.google.com/g/blockly/c/hglQG_hPy4g. 4.

In the config node you can now specify whether you want to see a minimap:

[image: image] https://user-images.githubusercontent.com/14224149/266869926-7e2fd4d4-aefd-43e5-8c3c-0f3924717f2c.png

I show it only when the workspace is extended (to full-screen) because otherwise it overlaps with the other icons. In extended mode it looks like this:

[image: image] https://user-images.githubusercontent.com/14224149/266870044-bfee4412-0d59-4a2a-bf8c-616e4f43eb66.png

It is all in the "maybe-later-maybe-never" branch, which you can install like this:

bartbutenaers/node-red-contrib-blockly/maybe-later-maybe-never

Would be nice if you (or others of course) could play with this. Because I have only tested a few scenario's due to lack of time. But since we have skipped a large amount of (technical refactoring) Blockly releases, I expect other things will need to be fixed in our node...

Of course, there is no need to hurry with this!!!

Bart

— Reply to this email directly, view it on GitHub https://github.com/bartbutenaers/node-red-contrib-blockly/issues/102, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAR7RNFIQ6URPRUEAX2TQF3XZYZQRANCNFSM6AAAAAA4SOC4SI . You are receiving this because you were mentioned.Message ID: @.***>

cymplecy commented 11 months ago

Installed it , created a small flow with it and no errors :)

Not seeing the value of the whole project thumbnail window at moment as too small for my eyesight to read anything in it - all I really see is a shape

I can't make search find anything - what search text works for you?

Simon

bartbutenaers commented 11 months ago
  1. About the minimap. It is in fact more like a gadget, similar like the Node-RED minimap. I use the Node-RED minimap sometimes when I am looking for things. And then I see in the minimap that in the middle of nowhere I forgot some leftover nodes...

  2. Yes at the moment the toolbox search does not work at all. As you can see in my original post above, I have asked this in the Blockly forum. Because it should work out of the box, but in our case their magic behind the scene doesn't do much. Hopefully I get some golden tip...

cymplecy commented 11 months ago
  1. "similar like the Node-RED minimap" Never heard of it before!

  2. "As you can see in my original post above" I see you've forgotten that I don't read details :)

bartbutenaers commented 11 months ago

RTFM ;-) I found the bug in Blockly that causes the toolbox-search plugin to fail.
Hopefully the fix is released soon, so we can finalize our own release...

bartbutenaers commented 11 months ago

Hi @cymplecy, The Github issue for our bug has been fixed by this pull-request, which has now been merged. As soon as the successor of Blockly version 10.1.3 is being released, I hope/expect that release contains this pull-request. So that we could continue with our development.

bartbutenaers commented 10 months ago

@cymplecy, I have updated the toolbox-search plugin version in the package.json of our maybe-later-maybe-never branch, to be install and test whether the issue is fixed. And it seems to be working fine to me (when you install our node again via the command in my original post above):

toolbox_search

Would be nice if you could do some tests, before we publish it on NPM...

cymplecy commented 10 months ago

Lovely - all seems good here :)

FYI - this is what I seem to have to use to install a testing branch

npm install https://github.com/bartbutenaers/node-red-contrib-blockly#maybe-later-maybe-never

bartbutenaers commented 10 months ago

Ah yes indeed. I had a typo in my command above. Have fixed it. So we are good enough for npm?

cymplecy commented 10 months ago

Yes :)

bartbutenaers commented 10 months ago

It is now available in the palette as version 2.1.0:

image

Will announce it on Discourse in a couple of minutes...