Freeciv

Pushing Reset button causes call to fc_shortcuts_dialog::refresh(), which calls fc_shortcuts_dialog::init(). When init() is called the first time from the constructor, the call to sizeHint() gives a reasonable value, but when called from refresh() gives bogus (22,22). I rewrote refresh() to just clear and reset the scroll_layout and not call init(). Also refactored code with new method fc_shortcuts_dialog::populate_scroll_layout().