Allow event query after plotSymbolWasSelectedAtRecordIndex is called

[GoogleCodeExporter]

As an example, in my app, if the user clicks a plot, 
plotSymbolWasSelectedAtRecordIndex is called. However, when the user 
holds the shiftkey, I'd like to zoom in, not select. I solved this by 
calling [NSApp currentEvent] in plotSymbolWasSelectedAtRecordIndex to 
test for the modifier key, and process accordingly: 
-(void)scatterPlot:(CPTScatterPlot *)plot 
plotSymbolWasSelectedAtRecordIndex:(NSUInteger)index 
{ 
    NSEvent *currentEvent = [NSApp currentEvent]; 
    if (!([currentEvent modifierFlags] & NSShiftKeyMask)) 
    { 
        self.selectedIndex = index; 
    } 
    else 
    { 
        zoomStart = [currentEvent locationInWindow]; 
        zoominProgress = YES; 
    } 
} 

It's a hack and it works, but I think it would be great if something 
similar were to built in into the framework. 

Original issue reported on code.google.com by koenvand...@gmail.com on 30 Dec 2011 at 2:22

[GoogleCodeExporter]

Original comment by eskr...@mac.com on 30 Dec 2011 at 3:44

• Changed state: Accepted
• Added labels: Priority-Medium, Type-Enhancement

[GoogleCodeExporter]

What is the preferred way to resolve this issue? I see at least three options:

1. Make new plot delegate methods similar to the existing ones that include the 
event as an additional parameter. The old delegate methods would continue to be 
supported for backwards compatibility.

2. Store the event in the graph (accessible via a property) before calling the 
-pointingDevice... methods on the plots, axis set, and plot spaces.

3. Store the event in the plot (accessible via a property) before calling the 
delegate.

My preference is option #1. Anyone else have an opinion?

Eric

Original comment by eskr...@mac.com on 22 Apr 2012 at 11:13

[GoogleCodeExporter]

I agree, passing the event with the delegate method seems the most logical.

Original comment by koenvand...@gmail.com on 23 Apr 2012 at 12:34

[GoogleCodeExporter]

Original comment by eskr...@mac.com on 28 Apr 2012 at 8:21

• Changed state: Started

[GoogleCodeExporter]

This issue was closed by revision 7888d057a237.

Original comment by eskr...@mac.com on 29 Apr 2012 at 1:53

• Changed state: Fixed