Grid config: set values from startup and improve layout

[chapulina]

🎉 New feature

Summary

• Make widgets expand to fill the entire plugin
• Increase maximum values
• Make use of sceneFromFirstRenderEngine to reduce duplication
• Choose number of decimals according to widget width
• Make it possible to set initial values from XML
• Add example

Note to self: backport to ign-gazebo3

Test it

ign gui -v 4 -c ign-gui/examples/config/grid_config.config

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

[ahcorde]

Suggested change

	<pose>1 2 3 0 0 0</pose>
	<pose>1 2 3 0 0 0</pose>

[chapulina]

I've been trying to get in the habit of separating position and rotation with an extra space as suggested on Specifying pose: Proposal for a better pose:

libsdformat and SDFormat tutorials should encourage additional whitespace, e.g. to separate translation and rotation, use 3 spaces (instead of 1) as a delimiter between values if they fit on one line, or use a newline (possibly with hanging indents) if they do not fit on one line.

[jennuine]

LGTM, I left a minor comment & suggestion but nonblocking

Comments addressed

[chapulina]

I'm working on addressing @jennuine 's comments, and I'll also take the opportunity to consolidate GridConfig and Grid3d once and for all. This should close #183.

[chapulina]

I updated this PR to incorporate Grid3D's functionality, and removed the code for Grid3D. The functionality added mainly consists of managing multiple grids:

ign gui -v 4 -c examples/config/grid_config.config

I also deleted the original Grid3D and now GridConfig is installed with both names.

@jennuine , I made quite a lot of changes from the original PR, mind taking another look? Thanks!

[chapulina]

While looking at the Grid3D code, I noticed a lot of it was still commented out because we never completed the migration from QWidget to QtQuick. Here's a gif with the original Grid3D. I miss having multiple views.

So I'll just note here that the functionality that I migrated to GridConfig is just what was working on ign-gui6, not all the functionality from ign-gui0. Specifically, the addition and removal of grids from the plugin can come at a later time.

[codecov]

Codecov Report

Merging #324 (e144763) into ign-gui6 (3b57ac2) will decrease coverage by 1.49%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##           ign-gui6     #324      +/-   ##
============================================
- Coverage     65.97%   64.48%   -1.50%     
============================================
  Files            34       36       +2     
  Lines          4782     4953     +171     
============================================
+ Hits           3155     3194      +39     
- Misses         1627     1759     +132     

Impacted Files	Coverage Δ
src/plugins/grid_config/GridConfig.hh	100.00% <ø> (ø)
src/plugins/grid_config/GridConfig.cc	22.35% <25.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b57ac2...e144763. Read the comment docs.