diff --git a/app/src/main/java/com/eveningoutpost/dexdrip/BGHistory.java b/app/src/main/java/com/eveningoutpost/dexdrip/BGHistory.java index 53128316c5..c0413bc579 100644 --- a/app/src/main/java/com/eveningoutpost/dexdrip/BGHistory.java +++ b/app/src/main/java/com/eveningoutpost/dexdrip/BGHistory.java @@ -137,7 +137,7 @@ private void setupCharts() { previewChart.setZoomType(ZoomType.HORIZONTAL); chart.setLineChartData(bgGraphBuilder.lineData()); - chart.setOnValueTouchListener(bgGraphBuilder.getOnValueSelectTooltipListener()); + chart.setOnValueTouchListener(bgGraphBuilder.getOnValueSelectTooltipListener(false)); previewChart.setLineChartData(bgGraphBuilder.previewLineData(chart.getLineChartData())); previewChart.setViewportCalculationEnabled(true); diff --git a/app/src/main/java/com/eveningoutpost/dexdrip/Home.java b/app/src/main/java/com/eveningoutpost/dexdrip/Home.java index b2f799799a..87baaab86e 100644 --- a/app/src/main/java/com/eveningoutpost/dexdrip/Home.java +++ b/app/src/main/java/com/eveningoutpost/dexdrip/Home.java @@ -1371,7 +1371,7 @@ public int getOpacity() { previewChart = (PreviewLineChartView) findViewById(R.id.chart_preview); chart.setLineChartData(bgGraphBuilder.lineData()); - chart.setOnValueTouchListener(bgGraphBuilder.getOnValueSelectTooltipListener()); + chart.setOnValueTouchListener(bgGraphBuilder.getOnValueSelectTooltipListener(true)); previewChart.setBackgroundColor(getCol(X.color_home_chart_background)); previewChart.setZoomType(ZoomType.HORIZONTAL); diff --git a/app/src/main/java/com/eveningoutpost/dexdrip/UtilityModels/BgGraphBuilder.java b/app/src/main/java/com/eveningoutpost/dexdrip/UtilityModels/BgGraphBuilder.java index e9c0c0a6ff..9b54775111 100644 --- a/app/src/main/java/com/eveningoutpost/dexdrip/UtilityModels/BgGraphBuilder.java +++ b/app/src/main/java/com/eveningoutpost/dexdrip/UtilityModels/BgGraphBuilder.java @@ -1537,13 +1537,18 @@ public String unit() { } - public OnValueSelectTooltipListener getOnValueSelectTooltipListener() { - return new OnValueSelectTooltipListener(); + public OnValueSelectTooltipListener getOnValueSelectTooltipListener(boolean interactive) { + return new OnValueSelectTooltipListener(interactive); } public class OnValueSelectTooltipListener implements LineChartOnValueSelectListener { private Toast tooltip; + private boolean interactive; + + public OnValueSelectTooltipListener(boolean interactive) { + this.interactive = interactive; + } @Override public synchronized void onValueSelected(int i, int i1, PointValue pointValue) { @@ -1574,26 +1579,25 @@ public synchronized void onValueSelected(int i, int i1, PointValue pointValue) { } else { message = timeFormat.format(time) + " " + (Math.round(pointValue.getY() * 10) / 10d) + " "+unit() + filtered; } + if (interactive) { + final View.OnClickListener mOnClickListener = new View.OnClickListener() { + @Override + public void onClick(View v) { + Home.startHomeWithExtra(xdrip.getAppContext(), Home.CREATE_TREATMENT_NOTE, time.toString(), Double.toString(ypos)); + } + }; + Home.snackBar(message, mOnClickListener); + } else { - final View.OnClickListener mOnClickListener = new View.OnClickListener() { - @Override - public void onClick(View v) { - Home.startHomeWithExtra(xdrip.getAppContext(), Home.CREATE_TREATMENT_NOTE, time.toString(), Double.toString(ypos) ); + if (tooltip != null) { + tooltip.cancel(); } - }; - Home.snackBar(message,mOnClickListener); - /* if (tooltip != null) { - tooltip.cancel(); - } - if (alternate.length()>0) { - tooltip = Toast.makeText(context, timeFormat.format(time) + ": "+alternate, Toast.LENGTH_LONG); - } else { - tooltip = Toast.makeText(context, timeFormat.format(time) + ": " + Math.round(pointValue.getY() * 10) / 10d + filtered, Toast.LENGTH_LONG); + tooltip = Toast.makeText(context, message, Toast.LENGTH_LONG); + View view = tooltip.getView(); + view.setBackgroundColor(getCol(X.color_home_chart_background)); + tooltip.show(); } - View view = tooltip.getView(); - view.setBackgroundColor(getCol(X.color_home_chart_background)); - tooltip.show();*/ } @Override