I would like to revert r11101. To me, the now extremely wide numberput boxes look very ugly and the tiny number is almost lost in them. And the arrows are very far apart for easy mouse changing.
I appreciate the fact that they are aligned and have the same size, but I have to agree they are too wide (at least the second ones). Ideally they would take the same size of the widest input number.
For that, one would need to add a third row of dummy objects ... can be done, but would need to be done for each of the affected dialogues. (Also the display settings i.e for traffic or map are now very wide.)
IMO the stretchable numberinputs look much nicer, this is the primary reason why I changed it. Also, other components like comboboxes or textinputs are stretchable too, so it is more consistent if numberinputs are stetchable, too.
Quote from: prissi on March 26, 2024, 02:22:12 PMFor that, one would need to add a third row of dummy objects ... can be done, but would need to be done for each of the affected dialogues. (Also the display settings i.e for traffic or map are now very wide.)
It would indeed be a larger change to check and change all the dialogues containing numberinputs. I think it would be best to revert the change right now (since reverting to the previous behaviour does not break anything), and then redo the change properly after the release since I want to focus on GUI improvements after the release anyway.
In my opinion, aligned input fields look much nicer than irregular ones.
(https://i.imgur.com/akafEY6.png)
I think that by properly combining table and gui_fill_t, you can maintain alignment while preventing the width from becoming too wide.
The red frame is a table with two horizontal rows.
Then put gui_fill_t in the blue frame.
I think this is the way ceeac wants to go and as long term goal this is certainly the best way. Alternatively, one could give the numberinput a default minimum width of 4 digits, and then they would align nicer, at least the smaller numbers.
r11105 enforces five digits minimum width. Maybe this should be a themeable parameter?