Toolbar View and Structure cleaning

[PaddleStroke]

This PR does several things :

• Put Link in the command group

• Give Link a proper tooltip (screenshot outdated: tooltips changed).
  

• Put sel_back and sel_forward in the treeview actions group rather than in root of view toolbar. This way they now appear in the view menu (they weren't before) and they don't appear in the root of view toolbar.

• Give them proper names and tooltips (screenshot outdated: tooltips changed)
  

• Remove 'Link Navigation' from the view toolbar

• Put 'Link Navigation' in the view menu where it was missing.

• Remove 'Bouding box' from the view toolbar (its still in view menu)

• Reorder the view toolbar so that the order makes sense.

Lastly it improves the tooltips of std_part and std_group.

Fixes #10755
Fixes #10756
Fixes #10757
Fixes #10758

[Roy-043]

The texts for the new tooltips seem to be directly copied from the Wiki. This does not necessarily work well. Some consitency with the other tooltips in the same menu/toolbar may also be desirable. I also think that a tooltip should be relatively short.

Example:

Wiki text (and current tooltip proposal):
"The Std SelBack command restores the previous recorded Tree view selection. Note that selections are only recorded if Tree RecordSelection mode is switched on."

Shorter alternative:
"Restore the previous Tree view selection. Only works if Tree RecordSelection mode is switched on."

[PaddleStroke]

I don't think tooltips need to be particularly short.
Having said that you are right in your example that the command name is not necessary.

[Roy-043]

My suggestion for the other tooltips (fixing some language issues and shortening a few):

A Link is an object that references or links to another object in the same document, or in another document. Unlike Clones, Links reference the original Shape directly, making them more memory efficient which helps with the creation of complex assemblies.

A Part is a general purpose container to keep together a group of objects so that they act as a unit in the 3D view. It is meant to arrange objects that have a Part TopoShape, like Part Primitives, PartDesign Bodies, and other Parts."

A Group is a general purpose container to group objects in the Tree view, regardless of their data type. It is a simple folder to organize the objects in a model.

[PaddleStroke]

Sounds good to me. I updated them.

[mwganson]

I prefer having the selection arrows on the toolbar. Is there a preference to keep them there?

[qewer33]

I prefer having the selection arrows on the toolbar. Is there a preference to keep them there?

You can customize the toolbars from Tools > Customize and you can even create custom toolbars for functions that you use regularly.

[PaddleStroke]

I have not added a preference to keep them in as :

• custom toolbar let you do this very easily.
• no one of the forum (https://forum.freecad.org/viewtopic.php?t=81319) seem to even know what these were doing.
  So it feels like very few people actually use them. IMO custom toolbars are specifically designed for such niche tools.

[adrianinsaval]

I have not added a preference to keep them in as :

* custom toolbar let you do this very easily.

* no one of the forum (https://forum.freecad.org/viewtopic.php?t=81319) seem to even know what these were doing.
  So it feels like very few people actually use them. IMO custom toolbars are specifically designed for such niche tools.

Yes, let's not keep polluting FreeCAD with very niche config options that serve no real purpose than a minor convenience for a small subset of users.

[wwmayer]

After each start of FreeCAD these error messages appear in the console:

<Command> Command.cpp(1828): duplicate command Std_SelBack
<Command> Command.cpp(1828): duplicate command Std_SelForward

The extra creation of StdCmdSelBack and StdCmdSelForward should be removed from CreateViewStdCommands()

[PaddleStroke]

Ah yes sorry about this. I will fix it asap.

[wwmayer]

I have a PR: #10819