vexed changed priority from normal to trivial

vexed commented

What other menu option would it be used on ?

moriarty commented

Everything longer than two options. And to be honest, there's no reason not to do it on two options either. So everything really.
But it's pretty much mandatory for languages too. (actually, lists as long as resolutions and languages should probably be real drop-down lists).
Texture size and FSAA would also benefit, and that's just from the other video options.

vexed commented

Replying to Warzone2100/old-trac-import#3318 (comment:2):

Everything longer than two options. And to be honest, there's no reason not to do it on two options either. So everything really.
But it's pretty much mandatory for languages too. (actually, lists as long as resolutions and languages should probably be real drop-down lists).
Texture size and FSAA would also benefit, and that's just from the other video options.

The problem is the way it is done is more of a hack than anything else. We don't even have drop-down lists. The GUI just wasn't made with this stuff in mind...

If you feel like making a patch to fix things, then please do so.

moriarty commented

Sorry, but my skills lie in finding bugs not fixing them (not everyone can code C(++) with proficiency, I sure can't).
Maybe a re-write of the GUI code is in order for a future major release.

Cyp <cyp@...> changed status from new to closed

Cyp <cyp@...> changed owner from `` to Cyp <cyp@wz2100.net>

Cyp <cyp@...> changed resolution from `` to fixed

Cyp <cyp@...> committed [95]

In Warzone2100/warzone2100@95d5bb5:

#CommitTicketReference repository="" revision="95d5bb5483e0919a2bb1dffa82ed85b0b3423501"
widget: Cycle through menu lists consistently.

Together with b6baadb80ae21684ea48856c635830ca9764d691 and 630c624433ead2663c1576a8491f409ea5a5766e, fixes #3318.