1. Fixed big issue when the track point's array indexes are used for the xAxis values instead of the actual distances. Old approach cause wrong elevation profile shape and wrong slope ratio.
2. Track start/end clipper (chartPreviewView) set to hidden because it overload the screen
3. Fix code formatting, naming and style
4. Added TapGesture to handle single taps on the chart
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>
This refactoring allows to split the data for the PP on 2 parts:
1. TrackRecordingInfo (dist, dur, acend, descend etc)
2. ElecationProfileData (only the points to draw)
Because the PP should be created only with the 1st one for the TrackRecordings (will be implemented later) and the PP shoul display all the info without the chart.
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>
`emptyInfo` is used as an initial (zero) state during the recoding process starting to notify the info observers
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>
Now it looks like this:
ios_version 2024.03.27
ios_build 1.44211.31960
And allows deploying different versions from branches into the TestFlight without clashing
Signed-off-by: Alexander Borsuk <me@alex.bio>
Fixes the ~40, GCC 14 warnings in the CI job:
````
generated_message_table_driven.h:159:20: warning: 'is_pod<google::protobuf::internal::AuxillaryParseTableField>'
is deprecated: use 'is_standard_layout && is_trivial' instead [-Wdeprecated-declarations]
159 | static_assert(std::is_pod<AuxillaryParseTableField>::value, "");
````
Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
Fixes the following GCC warning:
````
drape_frontend/circles_pack_shape.cpp:133:9: warning: ‘void* memset(void*, int, size_t)’
clearing an object of non-trivial type ‘struct df::CirclesPackDynamicVertex’;
use assignment or value-initialization instead [-Wclass-memaccess]
133 | memset(m_buffer.data(), 0, m_buffer.size() * sizeof(CirclesPackDynamicVertex));
| ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
````
Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
Fixes the following warning:
````
map/framework.cpp:2252:12: warning: variable ‘rect’ set but not used [-Wunused-but-set-variable]
````
Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
and added [railway=halt][station=funicular] to mapping as it’s nearly the same as [railway=station][station=funicular]
Signed-off-by: Jenny Em <ubahnverleih@gmail.com>
To retrieve as a `GpsTrackInfo` struct that will be used to notify the UI about the track recording process.
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>
GpsTrackCollection has two methods to add: point and points.
The first one is used only once time in the unit test and removed to simplify the adding logic.
RemoveUntil is not used.
Signed-off-by: Kiryl Kaveryn <kirylkaveryn@gmail.com>