jquery-simple-context-menu icon indicating copy to clipboard operation
jquery-simple-context-menu copied to clipboard

Added optional highlighting of right-clicked element; changed .bind to .on with contextMenuPlugin namespace; converted mixed tab+space indentation to tabs

Open buu700 opened this issue 11 years ago • 1 comments

buu700 avatar Nov 29 '12 21:11 buu700

Hi Ryan

Thanks for this change. It's hard to review because the reformatting means the GitHub diff shows that virtually every line has changed, and I the individual commit comments don't point me to which commits have the relevant changes.

Please can you give me some pointers as to which bits have changed, other than formatting.

Thanks -Joe

On Thu, Nov 29, 2012 at 3:25 PM, Ryan Lester [email protected]:


You can merge this Pull Request by running:

git pull https://github.com/buu700/jquery-simple-context-menu master

Or view, comment on, or merge it at:

https://github.com/joewalnes/jquery-simple-context-menu/pull/9 Commit Summary

  • Update jquery.contextmenu.css
  • Update jquery.contextmenu.js
  • Update jquery.contextmenu.js
  • Update jquery.contextmenu.js

File Changes

  • M jquery.contextmenu.css (157)
  • M jquery.contextmenu.js (189)

Patch Links

  • https://github.com/joewalnes/jquery-simple-context-menu/pull/9.patch

  • https://github.com/joewalnes/jquery-simple-context-menu/pull/9.diff

    — Reply to this email directly or view it on GitHubhttps://github.com/joewalnes/jquery-simple-context-menu/pull/9.

joewalnes avatar Nov 29 '12 23:11 joewalnes