From 4f776f3268beafb6c89dc5b615e426ad08c3be66 Mon Sep 17 00:00:00 2001 From: Alexander Sytnyk Date: Fri, 2 Feb 2018 19:14:45 +0200 Subject: [PATCH] Redesign "add waypoint" menu; remove some legacy code --- ...gment_add_waypoint_bottom_sheet_dialog.xml | 263 ++++++++++++++++++ OsmAnd/res/values/strings.xml | 4 + .../search/SearchByNameAbstractActivity.java | 4 - .../base/MenuBottomSheetDialogFragment.java | 23 +- .../AddWaypointBottomSheetDialogFragment.java | 160 +++++++++++ .../plus/dialogs/DirectionsDialogs.java | 43 +-- 6 files changed, 457 insertions(+), 40 deletions(-) create mode 100644 OsmAnd/res/layout/fragment_add_waypoint_bottom_sheet_dialog.xml create mode 100644 OsmAnd/src/net/osmand/plus/dialogs/AddWaypointBottomSheetDialogFragment.java diff --git a/OsmAnd/res/layout/fragment_add_waypoint_bottom_sheet_dialog.xml b/OsmAnd/res/layout/fragment_add_waypoint_bottom_sheet_dialog.xml new file mode 100644 index 0000000000..4fccb2edb8 --- /dev/null +++ b/OsmAnd/res/layout/fragment_add_waypoint_bottom_sheet_dialog.xml @@ -0,0 +1,263 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/OsmAnd/res/values/strings.xml b/OsmAnd/res/values/strings.xml index d7355cd6a5..28cb806bd8 100644 --- a/OsmAnd/res/values/strings.xml +++ b/OsmAnd/res/values/strings.xml @@ -9,6 +9,10 @@ 3. All your modified/created strings are in the top of the file (to make easier find what\'s translated). PLEASE: Have a look at http://code.google.com/p/osmand/wiki/UIConsistency, it may really improve your and our work :-) Thx - Hardy --> + Current + Adds the last stop along the route + Adds the first stop on the route + Move destination up, and create destination Show closed notes Show/Hide OSM Notes on the map. GPX - suitable for export to JOSM or other OSM editors. diff --git a/OsmAnd/src/net/osmand/plus/activities/search/SearchByNameAbstractActivity.java b/OsmAnd/src/net/osmand/plus/activities/search/SearchByNameAbstractActivity.java index f359311486..ec9534021a 100644 --- a/OsmAnd/src/net/osmand/plus/activities/search/SearchByNameAbstractActivity.java +++ b/OsmAnd/src/net/osmand/plus/activities/search/SearchByNameAbstractActivity.java @@ -90,7 +90,6 @@ public abstract class SearchByNameAbstractActivity extends OsmandListActivity private static final Log log = PlatformUtil.getLog(SearchByNameAbstractActivity.class); private static final int NAVIGATE_TO = 3; - private static final int ADD_WAYPOINT = 4; private static final int SHOW_ON_MAP = 5; private static final int ADD_TO_FAVORITE = 6; @@ -571,9 +570,6 @@ public abstract class SearchByNameAbstractActivity extends OsmandListActivity } else if (mode == NAVIGATE_TO) { DirectionsDialogs.directionsToDialogAndLaunchMap(getActivity(), searchPoint.getLatitude(), searchPoint.getLongitude(), ai.getHistoryName()); - } else if (mode == ADD_WAYPOINT) { - DirectionsDialogs.addWaypointDialogAndLaunchMap(getActivity(), searchPoint.getLatitude(), - searchPoint.getLongitude(), ai.getHistoryName()); } else if (mode == SHOW_ON_MAP) { showOnMap(searchPoint, ai); } diff --git a/OsmAnd/src/net/osmand/plus/base/MenuBottomSheetDialogFragment.java b/OsmAnd/src/net/osmand/plus/base/MenuBottomSheetDialogFragment.java index 0a3687b8e3..59d3d52d56 100644 --- a/OsmAnd/src/net/osmand/plus/base/MenuBottomSheetDialogFragment.java +++ b/OsmAnd/src/net/osmand/plus/base/MenuBottomSheetDialogFragment.java @@ -80,14 +80,12 @@ public abstract class MenuBottomSheetDialogFragment extends BottomSheetDialogFra } if (AndroidUiHelper.isOrientationPortrait(activity)) { - AndroidUtils.setBackground(activity, mainView, nightMode, R.drawable.bg_bottom_menu_light, R.drawable.bg_bottom_menu_dark); + mainView.setBackgroundResource(getPortraitBgResId()); } else { if (screenHeight - statusBarHeight - mainView.getHeight() >= getResources().getDimension(R.dimen.bottom_sheet_content_padding_small)) { - AndroidUtils.setBackground(activity, mainView, nightMode, - R.drawable.bg_bottom_sheet_topsides_landscape_light, R.drawable.bg_bottom_sheet_topsides_landscape_dark); + mainView.setBackgroundResource(getLandscapeTopsidesBgResId()); } else { - AndroidUtils.setBackground(activity, mainView, nightMode, - R.drawable.bg_bottom_sheet_sides_landscape_light, R.drawable.bg_bottom_sheet_sides_landscape_dark); + mainView.setBackgroundResource(getLandscapeSidesBgResId()); } } @@ -101,6 +99,21 @@ public abstract class MenuBottomSheetDialogFragment extends BottomSheetDialogFra }); } + @DrawableRes + protected int getPortraitBgResId() { + return nightMode ? R.drawable.bg_bottom_menu_dark : R.drawable.bg_bottom_menu_light; + } + + @DrawableRes + protected int getLandscapeTopsidesBgResId() { + return nightMode ? R.drawable.bg_bottom_sheet_topsides_landscape_dark : R.drawable.bg_bottom_sheet_topsides_landscape_light; + } + + @DrawableRes + protected int getLandscapeSidesBgResId() { + return nightMode ? R.drawable.bg_bottom_sheet_sides_landscape_dark : R.drawable.bg_bottom_sheet_sides_landscape_light; + } + protected boolean isNightMode() { if (usedOnMap) { return getMyApplication().getDaynightHelper().isNightModeForMapControls(); diff --git a/OsmAnd/src/net/osmand/plus/dialogs/AddWaypointBottomSheetDialogFragment.java b/OsmAnd/src/net/osmand/plus/dialogs/AddWaypointBottomSheetDialogFragment.java new file mode 100644 index 0000000000..777c454622 --- /dev/null +++ b/OsmAnd/src/net/osmand/plus/dialogs/AddWaypointBottomSheetDialogFragment.java @@ -0,0 +1,160 @@ +package net.osmand.plus.dialogs; + +import android.app.Activity; +import android.content.Context; +import android.content.DialogInterface; +import android.graphics.drawable.Drawable; +import android.graphics.drawable.LayerDrawable; +import android.os.Bundle; +import android.support.annotation.DrawableRes; +import android.support.annotation.Nullable; +import android.support.v4.content.ContextCompat; +import android.view.ContextThemeWrapper; +import android.view.LayoutInflater; +import android.view.View; +import android.view.ViewGroup; +import android.widget.ImageView; +import android.widget.TextView; + +import net.osmand.data.LatLon; +import net.osmand.data.PointDescription; +import net.osmand.plus.R; +import net.osmand.plus.TargetPointsHelper; +import net.osmand.plus.activities.MapActivity; +import net.osmand.plus.base.MenuBottomSheetDialogFragment; + +public class AddWaypointBottomSheetDialogFragment extends MenuBottomSheetDialogFragment { + + public static final String TAG = "AddWaypointBottomSheetDialogFragment"; + public static final String LAT_KEY = "latitude"; + public static final String LON_KEY = "longitude"; + public static final String POINT_DESCRIPTION_KEY = "point_description"; + + @Nullable + @Override + public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { + Bundle args = getArguments(); + final LatLon latLon = new LatLon(args.getDouble(LAT_KEY), args.getDouble(LON_KEY)); + final PointDescription name = PointDescription.deserializeFromString(args.getString(POINT_DESCRIPTION_KEY), latLon); + final TargetPointsHelper targetPointsHelper = getMyApplication().getTargetPointsHelper(); + + final int themeRes = nightMode ? R.style.OsmandDarkTheme : R.style.OsmandLightTheme; + final View mainView = View.inflate(new ContextThemeWrapper(getContext(), themeRes), + R.layout.fragment_add_waypoint_bottom_sheet_dialog, container); + + ((TextView) mainView.findViewById(R.id.current_dest_text_view)) + .setText(getCurrentDestinationName(targetPointsHelper.getPointToNavigate())); + + ((ImageView) mainView.findViewById(R.id.current_dest_icon)) + .setImageDrawable(getBackgroundIcon(R.drawable.ic_action_point_destination)); + ((ImageView) mainView.findViewById(R.id.subsequent_dest_icon)).setImageDrawable(getSubsequentDestIcon()); + ((ImageView) mainView.findViewById(R.id.first_interm_dest_icon)).setImageDrawable(getFirstIntermDestIcon()); + ((ImageView) mainView.findViewById(R.id.last_interm_dest_icon)).setImageDrawable(getLastIntermDistIcon()); + + View.OnClickListener onClickListener = new View.OnClickListener() { + @Override + public void onClick(View v) { + int id = v.getId(); + if (id == R.id.replace_dest_row) { + targetPointsHelper.navigateToPoint(latLon, true, -1, name); + } else if (id == R.id.subsequent_dest_row) { + targetPointsHelper.navigateToPoint(latLon, true, + targetPointsHelper.getIntermediatePoints().size() + 1, name); + } else if (id == R.id.first_intermediate_dest_row) { + targetPointsHelper.navigateToPoint(latLon, true, 0, name); + } else if (id == R.id.last_intermediate_dest_row) { + targetPointsHelper.navigateToPoint(latLon, true, targetPointsHelper.getIntermediatePoints().size(), name); + } + dismiss(); + } + }; + + mainView.findViewById(R.id.replace_dest_row).setOnClickListener(onClickListener); + mainView.findViewById(R.id.subsequent_dest_row).setOnClickListener(onClickListener); + mainView.findViewById(R.id.first_intermediate_dest_row).setOnClickListener(onClickListener); + mainView.findViewById(R.id.last_intermediate_dest_row).setOnClickListener(onClickListener); + mainView.findViewById(R.id.cancel_row).setOnClickListener(onClickListener); + + if (nightMode) { + int dividerColor = ContextCompat.getColor(getContext(), R.color.route_info_bottom_view_bg_dark); + mainView.findViewById(R.id.current_dest_divider).setBackgroundColor(dividerColor); + mainView.findViewById(R.id.cancel_divider).setBackgroundColor(dividerColor); + } + + setupHeightAndBackground(mainView, R.id.scroll_view); + + return mainView; + } + + @Override + public void onDismiss(DialogInterface dialog) { + super.onDismiss(dialog); + closeContextMenu(); + } + + @Override + protected Drawable getActiveIcon(@DrawableRes int id) { + return getIcon(id, nightMode ? R.color.ctx_menu_direction_color_dark : R.color.map_widget_blue); + } + + @Override + protected int getPortraitBgResId() { + return nightMode ? R.drawable.bg_additional_menu_dark : R.drawable.bg_bottom_menu_light; + } + + @Override + protected int getLandscapeTopsidesBgResId() { + return nightMode ? R.drawable.bg_additional_menu_topsides_dark : R.drawable.bg_bottom_sheet_topsides_landscape_light; + } + + @Override + protected int getLandscapeSidesBgResId() { + return nightMode ? R.drawable.bg_additional_menu_sides_dark : R.drawable.bg_bottom_sheet_sides_landscape_light; + } + + private Drawable getBackgroundIcon(@DrawableRes int resId) { + return getIcon(resId, R.color.searchbar_text_hint_light); + } + + private LayerDrawable getLayerDrawable(@DrawableRes int bgIdRes, @DrawableRes int icIdRes) { + return new LayerDrawable(new Drawable[]{getBackgroundIcon(bgIdRes), getActiveIcon(icIdRes)}); + } + + private LayerDrawable getSubsequentDestIcon() { + return getLayerDrawable(R.drawable.ic_action_route_subsequent_destination, + R.drawable.ic_action_route_subsequent_destination_point); + } + + private LayerDrawable getFirstIntermDestIcon() { + return getLayerDrawable(R.drawable.ic_action_route_first_intermediate, + R.drawable.ic_action_route_first_intermediate_point); + } + + private LayerDrawable getLastIntermDistIcon() { + return getLayerDrawable(R.drawable.ic_action_route_last_intermediate, + R.drawable.ic_action_route_last_intermediate_point); + } + + private String getCurrentDestinationName(@Nullable TargetPointsHelper.TargetPoint finish) { + Context ctx = getContext(); + StringBuilder builder = new StringBuilder(ctx.getString(R.string.shared_string_current)); + builder.append(": "); + if (finish != null) { + if (finish.getOnlyName().length() > 0) { + builder.append(finish.getOnlyName()); + } else { + builder.append(ctx.getString(R.string.route_descr_map_location)); + builder.append(" "); + builder.append(ctx.getString(R.string.route_descr_lat_lon, finish.getLatitude(), finish.getLongitude())); + } + } + return builder.toString(); + } + + private void closeContextMenu() { + Activity activity = getActivity(); + if (activity instanceof MapActivity) { + ((MapActivity) activity).getContextMenu().close(); + } + } +} diff --git a/OsmAnd/src/net/osmand/plus/dialogs/DirectionsDialogs.java b/OsmAnd/src/net/osmand/plus/dialogs/DirectionsDialogs.java index 6e711241d9..65852a05d0 100644 --- a/OsmAnd/src/net/osmand/plus/dialogs/DirectionsDialogs.java +++ b/OsmAnd/src/net/osmand/plus/dialogs/DirectionsDialogs.java @@ -2,7 +2,9 @@ package net.osmand.plus.dialogs; import android.app.Activity; import android.content.DialogInterface; +import android.os.Bundle; import android.support.v7.app.AlertDialog; +import android.support.v7.app.AppCompatActivity; import android.support.v7.widget.PopupMenu; import net.osmand.data.LatLon; @@ -45,38 +47,17 @@ public class DirectionsDialogs { MapActivity.launchMapActivityMoveToTop(act); } } - - public static void addWaypointDialogAndLaunchMap(final Activity act, final double lat, final double lon, final PointDescription name) { - final OsmandApplication ctx = (OsmandApplication) act.getApplication(); - final TargetPointsHelper targetPointsHelper = ctx.getTargetPointsHelper(); + + public static void addWaypointDialogAndLaunchMap(final AppCompatActivity act, final double lat, final double lon, final PointDescription name) { + final TargetPointsHelper targetPointsHelper = ((OsmandApplication) act.getApplication()).getTargetPointsHelper(); if (targetPointsHelper.getPointToNavigate() != null) { - AlertDialog.Builder builder = new AlertDialog.Builder(act); - builder.setTitle(R.string.new_destination_point_dialog); - builder.setItems( - new String[] { act.getString(R.string.replace_destination_point), - act.getString(R.string.keep_and_add_destination_point), - act.getString(R.string.add_as_first_destination_point), act.getString(R.string.add_as_last_destination_point) }, - new DialogInterface.OnClickListener() { - @Override - public void onClick(DialogInterface dialog, int which) { - if (which == 0) { - targetPointsHelper.navigateToPoint(new LatLon(lat, lon), true, -1, name); - closeContextMenu(act); - } else if (which == 1) { - targetPointsHelper.navigateToPoint(new LatLon(lat, lon), true, - targetPointsHelper.getIntermediatePoints().size() + 1, name); - closeContextMenu(act); - } else if (which == 2) { - targetPointsHelper.navigateToPoint(new LatLon(lat, lon), true, 0, name); - closeContextMenu(act); - } else { - targetPointsHelper.navigateToPoint(new LatLon(lat, lon), true, targetPointsHelper.getIntermediatePoints().size(), name); - closeContextMenu(act); - } - MapActivity.launchMapActivityMoveToTop(act); - } - }); - builder.show(); + Bundle args = new Bundle(); + args.putDouble(AddWaypointBottomSheetDialogFragment.LAT_KEY, lat); + args.putDouble(AddWaypointBottomSheetDialogFragment.LON_KEY, lon); + args.putString(AddWaypointBottomSheetDialogFragment.POINT_DESCRIPTION_KEY, PointDescription.serializeToString(name)); + AddWaypointBottomSheetDialogFragment fragment = new AddWaypointBottomSheetDialogFragment(); + fragment.setArguments(args); + fragment.show(act.getSupportFragmentManager(), AddWaypointBottomSheetDialogFragment.TAG); } else { targetPointsHelper.navigateToPoint(new LatLon(lat, lon), true, -1, name); closeContextMenu(act);