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

This Version Did Not Work For Me #129

Closed
SolStis86 opened this issue Sep 22, 2015 · 21 comments
Closed

This Version Did Not Work For Me #129

SolStis86 opened this issue Sep 22, 2015 · 21 comments

Comments

@SolStis86
Copy link

Im not sure if theres some deeper issues with bootstrap core js but this version didnt work... The new version you posted did however so Ive used that and it works great...

I added some additional

$(current).find('.dropdown-menu').fadeIn('fast');

$(current).find('.dropdown-menu').fadeOut('fast');

Within the mouseenter and leave event handlers for fading in and out.. looks and works great so thanks!

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

I'm not sure what you're talking about? What version didn't work? What version did?

@SolStis86
Copy link
Author

OH sorry it was the revised version posted by redavis in this issue: #111!

http://mediaflydesign.com/plugins/bs-hover.revised.js

The one from this repo didnt work at all for some reason....

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

What do you mean by "didn't work?"

What browser (and version) and OS (and version)?

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

What version of Bootstrap are you using?

@SolStis86
Copy link
Author

3.3.5 latest

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

What do you mean by "didn't work?"

What browser (and version) and OS (and version)?

@SolStis86
Copy link
Author

The menu didnt appear on hover basically... i'd added the data-hover attribute as instructed and included the code as instructed....

CHrome Version 45.0.2454.93 m
W10

@SolStis86
Copy link
Author

ive emailed you a working example of the code

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

I'd like to see the version that wasn't working, if possible.

@SolStis86
Copy link
Author

ill swap out the include, bear with

@SolStis86
Copy link
Author

k live

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

Hmm... strange, it's working for me (it's the same as the other one, but no fade).

I'm on a Mac (10.10.5), tho, Chrome version 45.0.2454.99.

@SolStis86
Copy link
Author

The demo here: http://cameronspear.com/demos/bootstrap-hover-dropdown/ isnt working on mine either... odd...

@SolStis86
Copy link
Author

does the fade version work on yours?

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

Yes

@SolStis86
Copy link
Author

Fk knows... theres no console errors showing at all

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

I just tried on my PC which is also running Windows 10. It's the same version of Chrome as my Mac (except the m at the end). My demo page works there.

Do you have any of that touch-enabled/tablet stuff active?

@SolStis86
Copy link
Author

Yes i think thats the Issue... i just commented out the code for the ontouchenabled check and it worked... I have touchscreen laptop...

@SolStis86
Copy link
Author

Emailed u version with fade func built in

@CWSpear
Copy link
Collaborator

CWSpear commented Sep 22, 2015

Ok, that officially makes this issue a duplicate of #68.

@CWSpear CWSpear closed this as completed Sep 22, 2015
@SolStis86
Copy link
Author

sorry lol

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

No branches or pull requests

2 participants