Enable distant mountains, wire by default
#1
Posted 12 September 2014 - 11:55 AM
Given that both are toggling key visual functionality, I believe they should both be on by default. I am not suggesting we remove either option, only that we change the default value from "off" to "on".
What do people think?
#3
Posted 12 September 2014 - 01:36 PM
Regards,
Rob Roeterdink
#4
Posted 12 September 2014 - 01:46 PM
#5
Posted 12 September 2014 - 01:49 PM
I want control to choose and don't want to be forced, as Rob suggests.
:oldstry:
#6
Posted 12 September 2014 - 01:58 PM
#7
Posted 12 September 2014 - 02:23 PM
roeter, on 12 September 2014 - 01:36 PM, said:
disc, on 12 September 2014 - 01:46 PM, said:
If you enable distant mountains and the current route has none, a small bit of unnecessary work is done in the loader process looking for distant mountains tiles and the renderer checks for any distant mountains to render separately, but as neither of these find anything there is no performance impact that I'm aware of. I don't see any point in having the setting vary dynamically (practically, that means either we have 3 states for the setting [on, off, auto] or we have no setting at all).
thegrindre, on 12 September 2014 - 01:49 PM, said:
I want control to choose and don't want to be forced, as Rob suggests.
Coonskin, on 12 September 2014 - 01:58 PM, said:
Yes, you're understanding correctly. Anyone who dislikes distant mountains or doesn't wish to use performance on them (on routes with distant mountains) will be able to deselect the option.
#8
Posted 12 September 2014 - 03:38 PM
I would support "on by default" since it would fall under "best practices."
Robert
#9
Posted 12 September 2014 - 07:42 PM
#10
Posted 13 September 2014 - 07:15 AM
BTW, you wouldn´t necessarily need three states, James. If auto reliably turns DM on if they are present, OFF and AUTO would be enough. That could go with a simple change of ON to be the AUTO mode, IE to say AUTO mode be labelled as ON, which will also avoid question why there´s no separate ON state.
Cheers, Markus