search

moveit / moveit_setup_assistant

THIS REPO HAS MOVED TO https://github.com/ros-planning/moveit
8 stars 20 forks source link

Fixed bug 82 in a quick way by reducing min size. #83

Closed hersh closed 10 years ago

hersh commented 10 years ago

This is a quick fix for #82.

If Dave Coleman was at Willow Garage when he wrote this code, I can tell you he was using a 2560x1600 resolution screen. So clearly he wasn't worried about fitting everything in compactly.

This change is just a quick patch to make things usable on a 768-vertical screen. Better changes would involve reworking the layout of the main page. There is lots of unnecessary space in there, but it is not filled by removing the min-size lines, it just squishes the widgets down and hides a bunch of information.

davetcoleman commented 10 years ago

I believe I only checked for 800px, since from my research at the time 768px was pretty small for most modern screens.