Page MenuHome

Improve readability of Normal menu in 3d view edit mode.
ClosedPublic

Authored by George Vogiatzis (Gvgeo) on Apr 7 2019, 7:56 PM.

Details

Summary

T61007 Improve readability of Normal menu in 3d view edit mode.
And rename operators for consistency.

Diff Detail

Repository
rB Blender

Event Timeline

George Vogiatzis (Gvgeo) edited the summary of this revision. (Show Details)
William Reynish (billreynish) requested changes to this revision.

What you have done is more correct, but also makes this menu slower to access.

As a general rule, we should try not to add more than two levels inside the pulldown menus. More than two makes it very hard to navigate.

Instead, I think we should try to solve this without adding more nesting.

Here's an idea:

We could just use separators, and use more descriptive names. So:

  • Normals
    • Recalculate Outside
    • Recalculate Inside
    • ---------------
    • Flip
    • Set from Faces
    • Rotate
    • Point to Target
    • Merge
    • Split
    • Average
    • ---------------
    • Copy Vectors
    • Paste Vectors
    • Multiply Vectors
    • Add Vectors
    • Smoothen Vectors
    • Reset Vectors
    • ---------------
    • Get Face Strength
    • Set Face Strength

By using separators and better/clearer naming, we communicate the features and keep the separation clear, but can avoid requiring sub-menus. And even though we have some word duplication (Vectors) I think it's a better overall way to solve the issue, for the above reasons.

This revision now requires changes to proceed.Apr 7 2019, 8:06 PM
George Vogiatzis (Gvgeo) edited the summary of this revision. (Show Details)Apr 7 2019, 8:48 PM
George Vogiatzis (Gvgeo) updated this revision to Diff 14662.

Faster and improved.

This revision is now accepted and ready to land.Apr 7 2019, 8:59 PM

Even tho it's approved, I would do a few tiny changes:

  1. Remove redundant 'normal' word in Point to Target - all these items have 'normal' implied.
  2. Put Reset Vectors last in the list of vector-related items
  3. Remove the icons from Get/Set Face Strength. I think they add nothing, other than confusion - they don't reflect the meaning at all really.
George Vogiatzis (Gvgeo) retitled this revision from Create menu for Normal Vector and Face Strength. to Improve readability of Normal menu in 3d view edit mode..Apr 8 2019, 9:12 AM
George Vogiatzis (Gvgeo) edited the summary of this revision. (Show Details)
George Vogiatzis (Gvgeo) updated this revision to Diff 14667.

Done, switched Add with Multiply too.

Imo "Vectors" word is redundant too.

Also...
I looked to add the missing ops, like spin, screw, select. But it turns it not that simple.
Some operations are in context menu but missing on the header menus, like knife topology tool.

They should all be, at very minimum, in a sub-menu 'Specials' in their relevant menu. Even if they need more verts/edges/faces selected, to serve as a map of all operations.

They are even more ops that were missing in 2.79 that can go there. There is no need to read a manual, to find hidden keywords to search.

Just a thought.

Pablo Vazquez (pablovazquez) added inline comments.
release/scripts/startup/bl_ui/space_view3d.py
3402

Instead of changing the UI, why not changing the name of the operator itself?

If we use a different name in Python it makes it hard to search for it later, since search shows the operator name.

This is good point, also for consistency with the tool's settings panel at the bottom.
But in the search menu popup there is not enough info(missing popup info too). If someone see 'smooth vectors', will not know about what vectors, this tool is.

What about rename this menu's tools like that(deleted lines keep as is):
~~~~~~~~~~
Make Normals Consistent -> Recalculate Normals
Make Normals Consistent -> Recalculate Normals

Flip Normals
Set Normals From Faces

Normal Rotate-> Rotate Normals
Point Normals To Target
Merge Normals
Split Normals
Average Normals

Normals Vector Tools
Normals Vector Tools
Normals Vector Tools
Normals Vector Tools

Smoothen Normals -> Smooth Normals Vectors
Normals Vector Tools

Face Strength -> Face Normal Strength
Face Strength -> Face Normal Strength
~~~~~~~~~
And keep text in the menu as planned.

I don't follow this latest suggestion. Inside the Normals menu, the term 'normal' is implicit, so we don't have to add it to every entry.

@William Reynish (billreynish) My suggestion is a solution to the search problem, mentioned from

If we use a different name in Python it makes it hard to search for it later, since search shows the operator name.

The suggestion:

  1. In menu omit the 'Normals' wording, as is currently in this patch.
  2. Change the operators names to match the names in the menu, and including the 'Normals' word at the end(where applicable).

This will make these tools explicit in the search popup, while keeping the menu simple and clear.

George Vogiatzis (Gvgeo) set the repository for this revision to rB Blender.
George Vogiatzis (Gvgeo) updated this revision to Diff 14987.

Rename operators in C

Nvm, already accepted.
I didn't get a response for a while.
I updated the patch, so you can see for yourselves, what I was trying to explain.
I believe you can use the old diff, if you don't like it.

With this update,
it is easy to search, using the name displayed in the normal menu.
And by typing 'normals' in search will get all the normals operators.


While keeping the menu simple.

This revision was automatically updated to reflect the committed changes.