WIP: [android-auto] Route Planning and other screens #4936

Closed
AndrewShkrob wants to merge 7 commits from android-auto/routing into aa
Member

Links to:

Links to: * Refactoring for master - https://git.omaps.dev/organicmaps/organicmaps/pulls/4987 * Refactoring of AA code - https://git.omaps.dev/organicmaps/organicmaps/pulls/4989
biodranik (Migrated from github.com) reviewed 2023-04-08 21:39:26 +00:00
biodranik (Migrated from github.com) left a comment
  1. Тут много мелких изменений, которые можно было бы смело мержить отдельно, без внимательного ревью.
  2. Есть рефакторинги, которые лучше было бы сразу в мастер, верно?
  3. Надо чтобы @rtsisyk посмотрел тоже внимательно.
1. Тут много мелких изменений, которые можно было бы смело мержить отдельно, без внимательного ревью. 2. Есть рефакторинги, которые лучше было бы сразу в мастер, верно? 3. Надо чтобы @rtsisyk посмотрел тоже внимательно.
@ -319,3 +318,4 @@
// }
}
JNIEXPORT void JNICALL Java_app_organicmaps_search_SearchEngine_nativeRunSearchMaps(
biodranik (Migrated from github.com) commented 2023-04-08 21:25:03 +00:00

Зачем это комментировать? А если потом забудем раскомментить, когда будет большой мерж в мастер?

Зачем это комментировать? А если потом забудем раскомментить, когда будет большой мерж в мастер?
@ -1208,2 +1247,3 @@
private boolean isFullscreen()
@Override
public void onPlacePageSlide(int top)
{
biodranik (Migrated from github.com) commented 2023-04-08 21:26:01 +00:00

Может какое-то более понятное сообщение?

Может какое-то более понятное сообщение?
@ -1258,9 +1293,8 @@ public class MwmActivity extends BaseMwmFragmentActivity
return;
biodranik (Migrated from github.com) commented 2023-04-08 21:27:27 +00:00

А может быть такое, что onStart вызвался, когда isDeviceDisplayUsed == true, а onStop, когда false? Или наоборот?

А может быть такое, что onStart вызвался, когда isDeviceDisplayUsed == true, а onStop, когда false? Или наоборот?
@ -157,3 +171,4 @@
if (newDisplayType == DisplayType.Car)
{
LocationState.nativeSetListener(this);
onMyPositionModeChanged(LocationState.nativeGetMode());
biodranik (Migrated from github.com) commented 2023-04-08 21:29:25 +00:00

ditto, это нужно вообще?

ditto, это нужно вообще?
@ -171,0 +193,4 @@
@Override
public void onPlacePageActivated(@NonNull PlacePageData data)
{
biodranik (Migrated from github.com) commented 2023-04-08 21:29:32 +00:00

ditto

ditto
biodranik (Migrated from github.com) commented 2023-04-08 21:29:08 +00:00

Очень много логов везде, они действительно нужны? Не усложнят потом нам отладку багов у пользователей по логам?

Очень много логов везде, они действительно нужны? Не усложнят потом нам отладку багов у пользователей по логам?
@ -0,0 +97,4 @@
final CarIcon icon = new CarIcon.Builder(IconCompat.createWithResource(context, drawableRes)).setTint(tintColor).build();
builder.setIcon(icon);
builder.setOnClickListener(() -> {
LocationState.nativeSwitchToNextMode();
biodranik (Migrated from github.com) commented 2023-04-08 21:30:16 +00:00

Это в каком например случае?

Это в каком например случае?
biodranik (Migrated from github.com) commented 2023-04-08 21:31:36 +00:00

Разве это нормально, когда вызывается метод, и ничего не происходит? В каких это случаях будет? Пользователи с этим столкнутся?

Разве это нормально, когда вызывается метод, и ничего не происходит? В каких это случаях будет? Пользователи с этим столкнутся?
@ -0,0 +151,4 @@
if (routingInfo == null)
{
throw new IllegalStateException("routingInfo == null");
}
biodranik (Migrated from github.com) commented 2023-04-08 21:32:31 +00:00
      throw new IllegalStateException("routingInfo == null");
```suggestion throw new IllegalStateException("routingInfo == null"); ```
@ -32,1 +32,3 @@
public class SearchOnMapScreen extends BaseMapScreen implements NativeSearchListener
import java.util.List;
// TODO: finish implementation
biodranik (Migrated from github.com) commented 2023-04-08 21:33:44 +00:00

nit: можно писать точнее, что надо закончить. Вдруг кто-то другой будет доделывать )

nit: можно писать точнее, что надо закончить. Вдруг кто-то другой будет доделывать )
@ -83,0 +101,4 @@
coordinates[2 * i] = location.getLatitude();
coordinates[2 * i + 1] = location.getLongitude();
}
// Framework.nativeShowPoints(coordinates);
biodranik (Migrated from github.com) commented 2023-04-08 21:34:03 +00:00

?

?
@ -129,0 +151,4 @@
System.nanoTime(), true /* isMapAndTable */, true /* hasLocation */,
location.getLat(), location.getLon());
else
SearchEngine.INSTANCE.searchInteractive(
biodranik (Migrated from github.com) commented 2023-04-08 21:34:48 +00:00

Может в одну строку всё-таки тут и выше? Коряво смотрится.

Может в одну строку всё-таки тут и выше? Коряво смотрится.
biodranik (Migrated from github.com) commented 2023-04-08 21:37:11 +00:00

А когда и почему возможен null? Пользователи никогда не увидят дисклеймер?

А когда и почему возможен null? Пользователи никогда не увидят дисклеймер?
This repo is archived. You cannot comment on pull requests.
No reviewers
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
2 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#4936
No description provided.