The International Simutrans Forum

Simutrans Extended => Patches/pull requests for consideration => Simutrans-Extended development => Incorporated patches/merged pull requests => Topic started by: Ranran(retired) on December 28, 2018, 11:16:20 AM

Title: Adjustment of mini-map window
Post by: Ranran(retired) on December 28, 2018, 11:16:20 AM
I thought that it would be out of the main theme of this thread (https://forum.simutrans.com/index.php/topic,18656.0.html), so I post a new thread.



(1) About mini map dialog minimum(default) size adjustment:

Currently(400px)
(https://i.imgur.com/XXJROc8.png)


Change(432px)
(https://i.imgur.com/YOvomns.png)


Currently, there were many margins so I felt it unnatural. If expand it's width slightly, it fits into four buttons. 
So, I expanded the minimum size from 400px to 432px.

I noticed that the mini-map's button size is set slightly larger than other window. (for "Show map scale" label?)
const scr_size button_size(max(D_BUTTON_WIDTH, 100), D_BUTTON_HEIGHT);

So, it does not fit four buttons with default setting.

Is this change accepted?


(2) Bug(?) fix:
I found the bug that once pressed the "Weight limit" button then the tooltip help for "Weight limit" changed to "Passenger destinations" after that.
(You can see in the above image.)

The cause of this was on line 521 of gui\map_frame.cc.

filter_buttons[18].set_tooltip("Passenger destinations");


I took this away but does this have any meaning? (For example, can not keep up with the later change)


(3) Label color grouping, change order:
Some buttons are already grouped and it is not divided by exclusive relationship.
Since buttons increased, there were many white buttons.

I think it can group "way" and "citybuilding" and colored.



About Citybuildings :Building / Commuting / Visiting / Staffing
About Ways : Traffic / Wear / Speed limit / Weight limit / Tracks / Powerlines
NOTE: I have not add these change for (3) yet.




What do you think about these?
Title: Re: Adjustment of mini-map window
Post by: jamespetts on December 28, 2018, 12:38:35 PM
Thank you for that: this is most helpful and definitely an improvement. I have incorporated this.

One thing with the window size, however: I notice that "Total capacity" is cut off on the right. I suspect that, before, it was simply not visible at all, but now it is partly visible and does not look good. I wonder whether the window could be re-arranged so as to have "Total capacity" such that it is not partly cut off at the default size?

In terms of the colours, I agree that it is sensible to update the colour groupings, which would certainly improve the UI. Perhaps a light blue would be a good additional colour?
Title: Re: Adjustment of mini-map window
Post by: Ranran(retired) on December 29, 2018, 12:01:48 PM
How about these buttons color?

(https://i.imgur.com/ERdIGbz.png)

(https://i.imgur.com/OGKjQHK.png)

"Total capacity" - This is part of show legend and related to networks display, so  move down one line is better?
Title: Re: Adjustment of mini-map window
Post by: jamespetts on December 29, 2018, 12:36:30 PM
Yes, those seem good colours and sensible groupings, thank you. Moving "total capacity" down a line might be sensible, but this would make the window take more vertical space. Perhaps it could go to the right of "Isometric view" instead, but only be activated when "show legend" is selected?

Also, I forgot to mention above: we need a .dat file for the new translatable texts for the tooltips and "Staffing".
Title: Re: Adjustment of mini-map window
Post by: Ranran(retired) on December 29, 2018, 01:49:25 PM
QuotePerhaps it could go to the right of "Isometric view" instead, but only be activated when "show legend" is selected?
(https://i.imgur.com/zV7XVMo.png)
You mean like this?
I also thought of the same thing, so I tried it but I could not select the checkbox although it would display there. (´・ω・`)

b_overlay_networks_load_factor.set_pos(scr_coord(3*D_BUTTON_WIDTH, -D_BUTTON_HEIGHT));


I attach dat file for translator to this post. Probably existing visiting and commuting are different as initials are lowercase.
Title: Re: Adjustment of mini-map window
Post by: jamespetts on December 29, 2018, 01:59:47 PM
That is very odd - I do not know enough about the GUI code to be able to assist with that.

Thank you for the .dat file: I have now uploaded this to Simutranslator.