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

Change state on hover #20

Closed
DynamicArray opened this issue Aug 29, 2014 · 7 comments
Closed

Change state on hover #20

DynamicArray opened this issue Aug 29, 2014 · 7 comments

Comments

@DynamicArray
Copy link

Hi

Great plugin, but it would be nice if the state of the buttons changed when hovering over so that the user knows to click.

It's a common feature of other rating plugins so it would be good to have here.

JC

@kartik-v
Copy link
Owner

Thanks for your feedback.

Currently hover is not included because of following reasons:

  1. The primary reason this is not done so far is that hover effect WILL NOT WORK on TOUCH enabled DEVICES or most common mobile devices and tablets. It will only work with normal desktops or PCs which have pointing input devices like a mouse.
  2. Secondly the way its implemented in the current release - you cannot achieve the hover effect with CSS -- you need additional javascript overload and validation.
    Its simple to do an hover effect with CSS if you have a whole number of fixed stars and whole fixed rating steps. This is what many other plugins do and its easy (in fact version 1.0 of this plugin had this hover feature - because it was restricted in its feature). But since this plugin now supports advanced functions like any fractional rating display, control any size of stars or change the symbols you need and related setup - its not achievable with CSS. There needs to be an additional Javascript coding just specifically to be done for creating the hover effect. So, I do not immediately intend to overload this plugin for this feature - which will not work on touch devices anyway.

I may try to include this if I get a request for this feature from many OR I find a better way to do this with minimal code size & performance overhead.

@nanonerd
Copy link

Hi Kartik,

I'm just starting to build a site from scratch so it's desktop only (mobile and tablets in the future hopefully). I found your star ratings to work really well and I like it a lot except that it lacks hover. Yeah, DynamicArray's suggestion is good. But it'd be great to display the different captions (in the spot where it says Not Rated) as the user hovered as well. Just having the hover captions alone will help a great deal because I want to use the splits and thus, with 10 possible selections from 5 stars, I want to show captions for each rating range.

Anyways, for desktop browser versions, I think hover is a must have. I know that it's not a simple and quick fix. But as it stands, I will have to begin a search for another that supports hover, the splits, etc. I'll def keep this one in mind if I ever get to mobile/tablets in the future. Thanks.

@kartik-v
Copy link
Owner

kartik-v commented Oct 6, 2014

Slated and targeting this for release v3.0.0.

@nanonerd
Copy link

nanonerd commented Oct 7, 2014

Cool. Any idea when v3 will be ready? I can wait about 2-3 months as I continue to hack away on the rest of my site ... I just tried Raty which has a lot of nice features (such as hover) but it doesn't support hover over half stars (without having to modify the original code) ... bleh. Thanks.

@kartik-v
Copy link
Owner

kartik-v commented Oct 7, 2014

Cannot commit - time permitting - should try to include this within a month - may need some performance testing.

@kartik-v
Copy link
Owner

This is resolved with upgrade to new release v3.0.0. Refer change log for details and refer updated docs and demos.

@nanonerd
Copy link

Thank you very much !

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

No branches or pull requests

3 participants