----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124441/ -----------------------------------------------------------
(Updated July 24, 2015, 11:41 a.m.) Review request for Marble and Gábor Péterffy. Changes ------- Fix style mistakes and disable fullsceen. Repository: marble Description ------- Move Marble Maps to QQmlApplicationEngine and QtQuick components. Add basic menubar and toolbar. Use "Holo Dark" Android theme. Diffs (updated) ----- src/apps/marble_maps/CMakeLists.txt a259af7 src/apps/marble_maps/MainScreen.qml 96238c5 src/apps/marble_maps/QmlView.h b312d9a src/apps/marble_maps/QmlView.cpp 6c4fc70 src/apps/marble_maps/main.cpp 473ff44 src/apps/marble_maps/package/AndroidManifest.xml 9125e5d Diff: https://git.reviewboard.kde.org/r/124441/diff/ Testing ------- Works on Nexus 5 & Nexus 9 (both Android 5.1.1). Noticed issues: - After clicking on menu navbar appears (see screenshot) and it doesn't hide. I tried to add simple TextField and after: 1) clicking on it; 2) closing keyboard; navbar hides. So, looks it's something focus-related. Also, do we even need to go to fullscreen (immersive mode)? For example Google Earth and Google Maps doesn't use it. File Attachments (updated) ---------------- Holo Dark 2.png https://git.reviewboard.kde.org/media/uploaded/files/2015/07/23/8c561590-4e50-47f2-83ab-44b883c4f51c__Holo_Dark_2.png Holo Dark 3.png https://git.reviewboard.kde.org/media/uploaded/files/2015/07/23/63c4a2ff-ea1d-47d9-a004-46cc056d7468__Holo_Dark_3.png Holo Dark non-fullscreen.png https://git.reviewboard.kde.org/media/uploaded/files/2015/07/24/4c96aaa9-b06e-4cdf-9b8a-09ff9f1daae3__Holo_Dark_non-fullscreen.png Holo Dark non-fullscreen 2.png https://git.reviewboard.kde.org/media/uploaded/files/2015/07/24/1fe9f00b-434d-49d2-ae4d-4341b64f13c8__Holo_Dark_non-fullscreen_2.png Thanks, Mihail Ivchenko
_______________________________________________ Marble-devel mailing list Marble-devel@kde.org https://mail.kde.org/mailman/listinfo/marble-devel