Minor uilist code clean-up

[Qrox]

Summary

SUMMARY: Interface "Minor uilist code clean-up"

Purpose of change

Clean up uilist related code a bit.

Describe the solution

Removed class ui_container and merged its members into uilist. This class probably was intended to act as a super class of all in-game menus, but unfortunately it seems that never happened.

Removed uilist::redraw and uilist_callback::redraw. The former one has already been fully incorporated into uilist::show, while the latter one was never used anywhere.

Removed uilist::refresh and incorporated its code into uilist::show where it was called.

Changed all uilist_callback::select overrides to uilist_callback::refresh overrides. To be honest, I cannot for the life of me fathom why uilist_callback::select was used as a redraw callback, nor why uilist_callback::select itself was called in the display code of uilist. Its name suggests that it is a callback that fires when the selection of uilist changes, while its implementation acted like a redraw callback, so this was either a misnomer of the callback function or an extremely poor implementation of the functionality.

Changed uilist_callback::select to fire when the selection actually changes instead of on each redraw. It is not removed because the callback is still useful in future migration of some UIs to ui_adatpor.

Finally, reordered and removed some member fields and functions of uilist to improve memory alignment and reduce bloat.

Testing

Every menu that uses the affected callback classes was tested and working as intended.

[ifreund]

Thanks!