Fix compare by time / by distance bug

.. silly typo mean't allplot was being used as a refrence
   when generating plots instead of fullplot

.. this manifests itself as using the wrong plot for the
   'bydist' boolean -- this is fine, unless you switch
   between by time and by distance whilst in compare mode.
UI Nits: Colorblind

.. tweaked color sequencing in compare to avoid using
   red/green as it should be avoided -- red/green color
   blindness is relatively common.

.. this is just a hack in the absence of a good algorithm
   to generate distinct hues automatically.
Fix Ride plot zone shading for series stack

.. the setting wasn't being set/unset according to
   user preferences properly.
AllPlot Paint Zones in Compare Mode

.. we should actually paint a zoned background if
   power shade zones is selected, when in compare mode.

.. but /only/ for charts that have power on them!
Hide empty series plots in compare mode

.. check the curves are empty!
Fixed Estimated VO2MAX metric

.. to use latest ACSM formula.

.. see: http://blue.utb.edu/mbailey/handouts/pdf/MetCalnew.pdf
UI Nits: Mac Athlete Tab Bar

.. now looks a bit more 'Mac-like'.
UI Nits: Prettify Mac Toolbar for QT5

.. colors match document mode tabbar.
Forgot to increment the DB Schema!
UI Nits: Qt5 Toolbar color when inactive
Don't auto open athlete that crashed

.. instead offer the cyclist menu
Fix WKO import - another optpad constant

.. amazingly there are still some constant values we have
   not seen before. This time from files provided by
   Patrick McNally.

.. optpad values we have seen now range from 0x8007 through
   0x801a and crucially with only one value missing 0x8008

.. its just a matter of time before we see one of those :)
Fix Mass Import Memory Exhaustion

.. don't signal when adding a ride, which kicks off
   all manner of updates across the code, instead
   we let the metric refresh kick in at the end.

.. this is great for an initial load and mass import
   of large numbers of rides, but sucks for a small number
   of rides when maybe downloading

.. we could make the signalling dependent on the number of
   rides being imported, so small number still updates the
   CP charts etc.

THIS IS A TEMPORARY FIX TO THE MORE SIGNIFICANT PERFORMANCE
ISSUE -- BUT WE DO NEED TO CONSIDER THE ISSUE OF CHART UPDATES
ANT+ Don't write 5 blank bytes

.. it doesn't do anything except create errors
Compromise on import

.. signal when importing a relatively small number
   of items (less than 20)

.. don't signal when importing a lot of items (more
   than 20).
What's in a name ?

The 4 views have been shown to be confusing -- it is not clear
to new users what their purpose is, and hence how or why they
are different.

So in an effort to reduce the confusion I've renamed a few
things to be much more explicit and specific.

.. We embrace the fact we are a cycling program and use the term
   RIDES not ACTIVITIES for all files

.. HOME is confusing, renamed to TRENDS

.. the ANALYSIS view name is confusing, its all about ANALYSIS
   to renamed to RIDES

There will no doubt be areas that are not renamed properly we
can fix those when the arise.
RideNavigator - activities become rides
Fixup renaming of Views

.. missed in a few minor places.
CP chart settings into tabs

.. there were too many settings on one page
   so split into tabs