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

Rating control does not return results #3

Closed
mathepton opened this issue Apr 14, 2016 · 6 comments
Closed

Rating control does not return results #3

mathepton opened this issue Apr 14, 2016 · 6 comments

Comments

@mathepton
Copy link

Rating control selections are not added to the survey result. This occurs using the default values (1 to 5), and when setting custom values.

If you make a Rating control 'Required' surveyjs will always identify it as unselected ("Please answer the question" alert appears).

Example survey:
var survey = new Survey.Survey(
{pages:[{name:"page1",questions:[{type:"rating",isRequired:true,name:"question1"}]}]});

@andrewtelnov
Copy link
Member

Hi,
I just tried with the latest source and it works fine. I have tried bootstrap and nobootstrap versions. I did not try npm version. I am going to update it shortly.
Could you please try with this links?
<script src="https://cdnjs.cloudflare.com/ajax/libs/knockout/3.4.0/knockout-min.js"></script> <script src="http://surveyjs.org/js/survey.min.js"></script> <link rel="stylesheet" href="http://surveyjs.org/css/survey.css" />
Thank you,
Andrew

@andrewtelnov
Copy link
Member

andrewtelnov commented Apr 15, 2016

I have published a new version into npm. I hope the issue is resolved.

Thank you,
Andrew

@mathepton
Copy link
Author

Hi Andrew. I'm still experiencing the same issue (using the most recent cut). I've done a bit more digging and found that it only occurs with the bootstrap version. I can replicate it using Windows desktop Chrome, Internet Explorer, and mobile Chrome on Android.

Importantly, I can see the issue when I use the Visual Designer and run the survey from there (so I've eliminated my setup from the equation).

I'm traveling right now so I don't have a ton of time to debug, but I'll try to take a closer look when I have a chance..

-mat

@andrewtelnov andrewtelnov reopened this Apr 15, 2016
@andrewtelnov
Copy link
Member

Yes, I have managed to reproduced this issue. I guess it is related to the Visual Editor.
I will work on the fix. I will keep you inform.

Thank you,
Andrew

@andrewtelnov
Copy link
Member

ok, Here is the reason of the issue: http://stackoverflow.com/questions/20077475/knockout-bootstrap-3-radio-buttons
I will try to find the workaround.

@andrewtelnov
Copy link
Member

Fixed-
Andrew-

OlgaLarina pushed a commit that referenced this issue Aug 29, 2023
tsv2013 pushed a commit that referenced this issue Oct 9, 2023
* fix cover

(cherry picked from commit 6a7b6e2)

* Cover remarks #3  resolve surveyjs/private-tasks#302

* fix for unit tests

* fix maximum call stack

---------

Co-authored-by: OlgaLarina <olga.larina.dev@gmail.com>
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