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

Add context menus #42

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add context menus #42

wants to merge 3 commits into from

Conversation

jovannic
Copy link

@jovannic jovannic commented May 9, 2016

This is really great.

I wanted to use the animated visualizations in a small presentation on git usage for non-command-line users. Not sure if you'll want this, but I added context menus for the basics on commit nodes and branches (commit (if head), reset, checkout, merge, rebase) and on origin (fetch, push, pull).

The node/tag menu uses historyview functions directly because historyview doesn't reference commandbox, and so you can do things on origin too. The origin menus use commandbox.command, mostly because they fail more frequently and I wasn't sure what to do with those errors. Didn't want to refactor or change the dependency graph on my own. Thoughts?

Used https://github.com/patorjk/d3-context-menu

@onlywei
Copy link
Owner

onlywei commented May 13, 2016

This is interesting. I will try to find time to code review this, but I don't find that I have a lot of time these days to do so. Please do not be offended if this takes a long time.

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