Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Context menu fix #175

Merged
merged 3 commits into from
Jun 7, 2016
Merged

Context menu fix #175

merged 3 commits into from
Jun 7, 2016

Conversation

kettch
Copy link
Contributor

@kettch kettch commented Jun 2, 2016

Fixed a case where the context menu wouldn't trigger, because backgroundView / contentView were used for cells and not properly calling [super rightMouseDown:]

@@ -187,6 +187,8 @@ - (void)mouseUp:(NSEvent *)theEvent {
}

- (void)rightMouseDown:(NSEvent *)theEvent {
[super rightMouseDown:theEvent];
Copy link
Owner

@jwilling jwilling Jun 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know the formatting in this project is kind of a mess, but for any newer code I just am trying to keep it formatted. So is there any chance you could tab this? 🎨

@kettch
Copy link
Contributor Author

kettch commented Jun 7, 2016

Of course, I usually do it, but this one must have passed through the cracks.. Pushing the fix.

@jwilling
Copy link
Owner

jwilling commented Jun 7, 2016

Thanks!

@jwilling jwilling merged commit 78ebe6b into jwilling:master Jun 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants