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

Remove useClick feature? #47

Closed
joeldbirch opened this issue Apr 5, 2013 · 15 comments
Closed

Remove useClick feature? #47

joeldbirch opened this issue Apr 5, 2013 · 15 comments
Labels

Comments

@joeldbirch
Copy link
Owner

@joeldbirch joeldbirch commented Apr 5, 2013

I'm regretting adding the useClick feature as it increases the complexity of the code and eats up development time that I'd prefer to spend focussing on hover events (plus touch and keyboard support, of course), which is truer to the mission of Superfish.

I think it would be a better idea to have a separate click menu plugin, allowing me to remove some complexity and bloat from Superfish and making room for further enhancements. I'm not seeing much benefit of having both click and hover capability in the one plugin, and the amount of if (o.useClick) {… forks that keep needing to be added is telling.

I'd really appreciate hearing some opinions on this.

@steffen-maass
Copy link

@steffen-maass steffen-maass commented Apr 7, 2013

Hi,

I'm using your plugin on a number of sites and never used the click-feature, the recent improvements for touch-devices makes the click-feature even more redundant for me...

Long story short: I'm all for removing this... ;)

Best regards,
s-m

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Apr 8, 2013

Thanks for chiming in. Music to my ears!

@Spuds
Copy link

@Spuds Spuds commented Apr 8, 2013

The unruly nature of the onclick is the reason I have not upgraded to the latest version .. and as you noted trying to support both and touch devices is causing some bloat and development pain.

I'd be fine seeing it removed and done as a separate plug-in for those folks that prefer click over hover (I let them choose since it was less work than arguing with em!)

@OxyFuse
Copy link

@OxyFuse OxyFuse commented Apr 9, 2013

I'm in favor of this as long as touch is supported.

@allthosehumans
Copy link

@allthosehumans allthosehumans commented Apr 10, 2013

I've found that even with the hoverIntent plugin people still find clicking easier than hovering especially on laptops. There are a few UX articles floating around about the caveats of the hover interaction.

If you can bare it, I wouldn't remove it. I do think moving it to a separate plugin would be fine though.

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Apr 12, 2013

Thanks very much for the feedback. useClick will be removed in version 1.7 (which is pretty much ready for release).

@Pushplaybang
Copy link

@Pushplaybang Pushplaybang commented Apr 16, 2013

Know this is closed, but would be great to see this at least moved to a separate plugin, the click was one of my main reasons for my excitement in seeing superfish brought back into active development.

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Apr 16, 2013

Removing useClick has really cleared some roadblocks for me so I'm glad I did it in that regard, but I knew there would be some disappointment, so I'm sorry about that. I agree that it should be a separate plugin. I'm hoping someone interested enough will fork Superfish and rewrite the jQuery to solely use click. Maybe it could stay in sync with Superfish as much as possible and benefit from sharing CSS and such. Implementing the click functionality would be simple enough given that hovers and touch events won't need to be juggled.

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Apr 16, 2013

Okay, so I've created Superclick. I'll get it up on GitHub in the next day or so.

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Apr 16, 2013

Here is the Superclick repo.

@Pushplaybang
Copy link

@Pushplaybang Pushplaybang commented Apr 26, 2013

nice.

@dalin-
Copy link

@dalin- dalin- commented May 27, 2015

One downside of hover is with a non-touch device that has a small viewport, it's tricky to navigate a mobile-optimized menu with hovers. You have to be really precise with the pointer and things open/close like wild.

@chriskuku
Copy link

@chriskuku chriskuku commented Jun 2, 2016

Hi,
I have a problem with a mobile-menu which might stem from code developed and discussed here.
Have a look at http://krischu.bplaced.net/test.html

When the mobile-menu appears I would like to be able to navigate to a node point. At the moment the menu is constructed in a way that clicking on the first level (node) only the submenu opens.

Would it be possible to modify the code such that with a click on the left side the node's URL opens
(in the case of the example: /ueber-uns/test.html) , with a click on the arrow in the right, the submenu drops down and can be navigated to by clicking on the link there (/ueber-uns/history/test.html)?

Thanks for helping.

Christoph

@joeldbirch
Copy link
Owner Author

@joeldbirch joeldbirch commented Jun 2, 2016

Hello,

Sorry, that site doesn't use Superfish.

@chriskuku
Copy link

@chriskuku chriskuku commented Jun 2, 2016

Oh, sorry. Just thought because it was using class names like sf-sub-indicator and other class names starting with sf-. And, last not least, since it contains (in functions.js)

/*
 * jQuery Superfish Menu Plugin - v1.7.4
 * Copyright (c) 2013 Joel Birch
 *
 * Dual licensed under the MIT and GPL licenses:
 *  http://www.opensource.org/licenses/mit-license.php
 *  http://www.gnu.org/licenses/gpl.html
 */

Just make the browser window smaller. Then the hamburger menu will appear. Isn't that SF?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
8 participants
You can’t perform that action at this time.