Skip to content

Commit

Permalink
Fixed issue #07867: 5 point scale question type is missing CSS classe…
Browse files Browse the repository at this point in the history
…s for slider or star rating layout

Dev Added classes to the slider elements
  • Loading branch information
tpartner committed Sep 22, 2013
1 parent bcc257f commit 9c3fc5b
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions application/helpers/qanda_helper.php
Expand Up @@ -1022,14 +1022,14 @@ function do_5pointchoice($ia)
$answer.="
<script type=\"text/javascript\">
document.write('";
$answer.="<div style=\"float:left;\">'+
'<div style=\"text-align:center; margin-bottom:6px; width:370px;\"><div style=\"width:2%; float:left;\">1</div><div style=\"width:46%;float:left;\">2</div><div style=\"width:4%;float:left;\">3</div><div style=\"width:46%;float:left;\">4</div><div style=\"width:2%;float:left;\">5</div></div><br/>'+
'<div id=\"{$id}sliderBg\" style=\"background-image:url(\'{$imageurl}/sliderBg.png\'); text-align:center; background-repeat:no-repeat; height:22px; width:396px;\">'+
$answer.="<div style=\"float:left;\" class=\"slider-wrapper slider-5\">'+
'<div style=\"text-align:center; margin-bottom:6px; width:370px;\" class=\"slider-labels\"><div style=\"width:2%; float:left;\" class=\"slider-label-1\">1</div><div style=\"width:46%;float:left;\" class=\"slider-label-2\">2</div><div style=\"width:4%;float:left;\" class=\"slider-label-3\">3</div><div style=\"width:46%;float:left;\" class=\"slider-label-4\">4</div><div style=\"width:2%;float:left;\" class=\"slider-label-5\">5</div></div><br/>'+
'<div id=\"{$id}sliderBg\" style=\"background-image:url(\'{$imageurl}/sliderBg.png\'); text-align:center; background-repeat:no-repeat; height:22px; width:396px;\" class=\"slider-background\">'+
'<center>'+
'<div id=\"{$id}slider\" style=\"width:365px;\"></div>'+
'<div id=\"{$id}slider\" style=\"width:365px;\" class=\"slider slider-rating\"></div>'+
'</center>'+
'</div></div>'+
'<div id=\"{$id}emoticon\" style=\"text-align:left; margin:10px; padding-left:10px;\"><img id=\"{$id}img1\" style=\"margin-left:10px;\" src=\"{$imageurl}/emoticons/{$value}.png\"/><img id=\"{$id}img2\" style=\"margin-left:-31px;margin-top:-31px;\" src=\"{$imageurl}/emoticons/{$value}.png\" />'+
'<div id=\"{$id}emoticon\" style=\"text-align:left; margin:10px; padding-left:10px;\" class=\"slider-emoticon-wrapper\"><img id=\"{$id}img1\" style=\"margin-left:10px;\" src=\"{$imageurl}/emoticons/{$value}.png\" class=\"slider-emoticon-1\" /><img id=\"{$id}img2\" style=\"margin-left:-31px;margin-top:-31px;\" src=\"{$imageurl}/emoticons/{$value}.png\" class=\"emoticon-2\" />'+
'</div>";
$answer.="');
</script>
Expand Down

5 comments on commit 9c3fc5b

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

205 have a completely different system ;)

e5765fb

@tpartner
Copy link
Collaborator Author

@tpartner tpartner commented on 9c3fc5b Sep 23, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we have to talk for "litle fix" (don't broke survey but can be cool). Here, i think we have to do it only for 2.05 ;)

@tpartner
Copy link
Collaborator Author

@tpartner tpartner commented on 9c3fc5b Sep 23, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh no,

Misunderstanding, and bad explanation ....

I mean : your fix don't broke survey, but add a "new feature" ;). The "no css class" for 5-slider is more a feature than a bug :)

Please sign in to comment.