sverchok icon indicating copy to clipboard operation
sverchok copied to clipboard

fix #4705. Viewer Index+ show right text with multiple objects and indices of objects

Open satabol opened this issue 1 year ago • 8 comments

Sverchok 1.3.0-alpha, Blender 3.6.1, Windows 11

fix #4705

  • [ ] waiting for ready to blender 4.0

Now Modes are good:

  1. Only indices for vertices, edges, faces without object indices image image image

  2. Indices for vertices, edges, faces with (!!!) object indices image image image

  3. Only text for vertices, edges, faces without object indices image image image)

  4. Text for vertices, edges, faces with (!!!) object indices image image image

Blend file for tests: issue_4705.example.blend.zip

satabol avatar Sep 22 '23 22:09 satabol

Hi @zeffii ! Can you test this pull request? I force to work this node in a mode with multiple objects with custom text and property "Draw Object Index" is On. But I think this node need global refactor ))) (not now)

P.S. Also I simplify using shaders to a single shader draw_indices_2D_wbg without draw_indices_2D. draw_indices_2D is a very simple and it differ only by a background rectangle. May be draw_indices_2D is not need any more?

satabol avatar Sep 22 '23 22:09 satabol

All was hidden with that configuration of params: image

image Fixed.

satabol avatar Sep 23 '23 08:09 satabol

the ghost icon-button is for hiding backfacing indices, so only really works when user passes polygons too.

zeffii avatar Sep 23 '23 08:09 zeffii

I get this glitch after exclude draw_indices_2D. Now it is work good.

satabol avatar Sep 23 '23 08:09 satabol

Hi, @zeffii! May I ask you to view this PR?

satabol avatar Sep 29 '23 18:09 satabol

@satabol sorry , i've been traveling and have not been spending much time near computers.

zeffii avatar Oct 08 '23 12:10 zeffii

i don't see anything i disagree with, but would prefer if #4983 was pushed first.

zeffii avatar Oct 08 '23 12:10 zeffii

Ok @zeffii. No problem. I will expect it. )))

satabol avatar Oct 08 '23 16:10 satabol