gazebosim / gazebo-classic

Gazebo classic. For the latest version, see https://github.com/gazebosim/gz-sim
http://classic.gazebosim.org/
Other
1.17k stars 477 forks source link

Qt5: Clicking to select models in the scene doesn't work every time #2135

Open osrf-migration opened 7 years ago

osrf-migration commented 7 years ago

Original report (archived issue) by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


I inserted two simple shapes and then started selecting one and then the other by clicking on them in the scene.

More than half the times, the click doesn't select the shape, I have to click a few times until it is selected.

I tried the same with gazebo7 and every click works

osrf-migration commented 7 years ago

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


osrf-migration commented 7 years ago

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


I think I may have seen this problem before but I can't reproduce it anymore. Can you try commenting out the Qt::OpaquePainEvent attribute in GLWidget.cc?

osrf-migration commented 7 years ago

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Yes, that solves the issue for me!

osrf-migration commented 7 years ago

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


@iche033 , do you think there's a downside to removing that line? I can go ahead and open a PR if you want

osrf-migration commented 7 years ago

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


it's supposed to be an optimization flag but if that's causing a bug we should just remove it. Yes go ahead and open a PR

osrf-migration commented 7 years ago

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Actually I just tested the latest gazebo8 branch and the issue seems not to be happening anymore. Maybe we can leave this open for a while and see if other people complain...