Forward-port 3 to 6

[AzulRadio]

➡️ Forward port

Port ign-gui3 to ign-gui6

Branch comparison: ign-gui6...ign-gui3

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

[chapulina]

LGTM, just the example that needs to be updated. You can do that as a new commit to this branch, suffixed with (#434) so we can trace it back to this PR.

[jennuine]

This test is failing:

gz-gui/test/integration/minimal_scene.cc

Line 158 in 2ba68b8

EXPECT_DOUBLE_EQ(60, camera->HFOV().Degree());

@chapulina should Haoyuan update the test to use EXPECT_NEAR?

[chapulina]

Yeah I see it's very close, EXPECT_NEAR sounds good, thanks!

45: Expected equality of these values:
45:   60
45:   camera->HFOV().Degree()
45:     Which is: 60.000001368151501

[chapulina]

LGTM with happy CI. Remember to merge with commit, don't squash or rebase 😉

[codecov]

Codecov Report

Merging #434 (15f131a) into ign-gui6 (2ba68b8) will increase coverage by 35.34%.
The diff coverage is 74.66%.

❗ Current head 15f131a differs from pull request most recent head 24bb4b3. Consider uploading reports for the commit 24bb4b3 to get more accurate results

@@              Coverage Diff              @@
##           ign-gui6     #434       +/-   ##
=============================================
+ Coverage     30.54%   65.89%   +35.34%     
=============================================
  Files            41       39        -2     
  Lines          1748     5257     +3509     
=============================================
+ Hits            534     3464     +2930     
- Misses         1214     1793      +579     

Impacted Files	Coverage Δ
include/ignition/gui/MainWindow.hh	100.00% <ø> (ø)
src/Application.cc	86.52% <0.00%> (ø)
src/Dialog.cc	77.17% <76.71%> (ø)
src/plugins/camera_tracking/qrc_CameraTracking.cpp
include/ignition/gui/qrc_resources.cpp
src/plugins/world_stats/moc_WorldStats.cpp
include/ignition/gui/moc_Application.cpp
src/plugins/teleop/moc_Teleop.cpp
src/plugins/world_control/qrc_WorldControl.cpp
src/plugins/scene3d/moc_Scene3D.cpp
... and 73 more

---

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 2ba68b8...24bb4b3. Read the comment docs.