ros-visualization / rviz

ROS 3D Robot Visualizer
BSD 3-Clause "New" or "Revised" License
847 stars 463 forks source link

toggle fullscreen button for ImageDisplay #1803

Closed rhaschke closed 7 months ago

rhaschke commented 1 year ago

implementation for #1802

peci1 commented 6 months ago

With a single display, it works well. However, it struggles on my multi-monitor setup:

https://github.com/ros-visualization/rviz/assets/182533/7a6b3a0c-3a72-4a44-aedd-6423a0c4f754

You can see the behavior is not consistent on multiple retries.

My monitor setup:

Snímek obrazovky pořízený 2024-05-09 14-39-01

xrandr excerpt:

Screen 0: minimum 320 x 200, current 3120 x 2451, maximum 16384 x 16384
eDP connected 1920x1080+1200+1371 (normal left inverted right x axis y axis) 309mm x 173mm
...
DisplayPort-5 connected primary 1920x1200+1200+171 (normal left inverted right x axis y axis) 518mm x 324mm
...
DisplayPort-9 connected 1200x1920+0+0 left (normal left inverted right x axis y axis) 518mm x 324mm
...

The same happens even when I align the top of display 2 with the top of display 3.

rhaschke commented 6 months ago

@peci1, I'm afraid that's an Qt issue (which we cannot resolve).

peci1 commented 6 months ago

Okay, it's nothing horrible ;)