target scaling menu bar logic:
*Options->Ships_>AIS Targets->Allow scaling down targets..... is not ticked, even though the feature is active.
I corrected a couple of logic holes in the code. In github now.
Now we should think of the two checkboxes in Options->Ships->AIS Targets as "enablers" of the related functions. If those boxes are unticked, then the feature (hiding moored targets, or scaling targets) is disabled.
Also, the current
logic does not allow both hiding of moored targets, and scaling of unimportant targets, at the same time. This is a consequence of the toolbar tool "cycling" action. I don't know if this is going to be a functional problem, or not.
We could of course change the code so, for instance, the selections are cumulative. That is, "scaling" of targets implies automatically suppression of moored targets as well. I see no other clear way to do this if we really want the toolbar "cycling" action.
So, the cycle would go:
0. AIS ON
1. AIS OFF
2. Suppress moored targets, if enabled.
3. Suppress moored targets, if enabled, and scale unimportant targets, if enabled.
4. and then back to 0.
Also, we will need to develop a crisp, "new-user-compatible" definition of "unimportant targets". I'm sure it's mother would thing it is important, so why would OCPN not think so?
Such definition should include no
reference to any tweak values in the config file, since they are not really user-space accessible.