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

Implemented fixes from other peoples pull requests into one #47

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

Conversation

andyfitch
Copy link

Implemented some other pull requests into a version which now:

  1. Works in any direction - Support for tolerance on submenus that are positioned above and below the main menu. #22
  2. Fix for activation on blank areas - Should clear any pending activation when leaving an element #29
  3. Compatible with new elements inserted into the DOM. - Work with new elements inserted into the menu #31
  4. Reset activeRow to enable hover over same element twice consecutively - Resetting activeRow when going to a non submenuSelector element #33
  5. Has bower support - Added bower.json to make this plugin installable with bower #36

I've also changed DELAY from a constant to a configurable option.

@Rio517
Copy link

Rio517 commented Feb 12, 2014

Nice. I hope @kamens will merge this in! Or let someone else.

@jescalan
Copy link

So it's quite clear that this project is now unmaintained, is there a more active fork that can be used with fixes like this? @kamens it is your responsibility to help users transition to a maintained version if you are no longer working on this, this repo has over 6,000 stars.

@mdix
Copy link

mdix commented Mar 24, 2015

@Jenius ack

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.

5 participants