Previously on desktop, the always-on 3D mode was behind an odd
"easter-egg", that was only reachable with a series of
`Allow3dMode` messages:
````
#ifdef OMIM_OS_DESKTOP
if (m_enablePerspectiveInNavigation == msg->AllowPerspective() &&
m_enablePerspectiveInNavigation != screen.isPerspective())
{
AddUserEvent(make_unique_dp<SetAutoPerspectiveEvent>(m_enablePerspectiveInNavigation));
}
#endif
````
This change adds a dedicated field for that in the message,
and a dedicated checkbox on the UI.
The checkbox is labelled `experimental`, because for now `drape_frontend`
is rather unstable in this mode.
This stability issue is independent from how we enable the always-on 3D mode
(old "easter-egg" or proper UI checkbox).
The idea is that making it easier to enable, also makes it easier to improve.
Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
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>
* [Android] Possible fix for wrong ime insets on some devices
* [Android] Allow changing default insets type mask with a Builder
* [Android] Change insets type to exclude ime insets
Signed-off-by: Dzmitry Strekha <mr.choo96@gmail.com>
