ducdo79 / core-plot

Automatically exported from code.google.com/p/core-plot
0 stars 0 forks source link

CPTBarPlot barPlot:barWasSelectedAtRecordIndex: called even if bar is not selected. #544

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Create graph with CPTBarPlot having positive and negative value (Bars are 
Vertical). 
2. Have zoom and scrolling to graph.
3. Enable interaction to plot space and implement 
barPlot:barWasSelectedAtRecordIndex: or - 
barPlot:barWasSelectedAtRecordIndex:withEvent: delegate method.
4. Tap on plot space near to bar (above x-axis and not on the bar) whose tip 
value is negative. Graph can be zoomed-in.

What is the expected output? What do you see instead?

Expected output:
The delegate method barPlot:barWasSelectedAtRecordIndex: should not be called.

Actual output: 
barPlot:barWasSelectedAtRecordIndex: or 
barPlot:barWasSelectedAtRecordIndex:withEvent: (whichever implemented) delegate 
method was called.

What version of the product are you using? On what operating system?
I am using core-plot 1.2 on ios 6.1

Please provide any additional information below.

The delegate method was called event if user taps on plot space near the bar.
This behaviour is clearly observed if bar have negative tip value and graph is 
zoomed-in.
This happens when user taps above the bar (not on the bar) near x-axis.

Original issue reported on code.google.com by projectc...@gmail.com on 31 May 2013 at 6:23

GoogleCodeExporter commented 8 years ago

Original comment by eskr...@mac.com on 1 Jun 2013 at 4:23

GoogleCodeExporter commented 8 years ago
Can you post screenshots and/or code showing the problem? The bar selection 
code appears to work correctly in the Core Plot example apps on both Mac and 
iOS.

Original comment by eskr...@mac.com on 2 Jun 2013 at 12:24

GoogleCodeExporter commented 8 years ago
- I have attached VerticalBarChart.m from CorePlotGallery, I have made some 
changes to have bar with negative values.

- I have also attached screen shot for reference.
- I have not added zoom and scroll to graph.

Original comment by projectc...@gmail.com on 3 Jun 2013 at 2:14

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by eskr...@mac.com on 9 Jun 2013 at 12:09

GoogleCodeExporter commented 8 years ago
This issue was closed by revision cc264c65608b.

Original comment by eskr...@mac.com on 9 Jun 2013 at 12:49