From 1c885d9c6d3e9cdc6b42d590bfd77784f5b41680 Mon Sep 17 00:00:00 2001 From: Thomas FORGIONE Date: Fri, 4 Sep 2015 16:31:21 +0200 Subject: [PATCH] Some modifications --- controllers/feedback/views/index.jade | 33 +++++++++--- controllers/prototype/views/user_study.jade | 18 ++++++- .../apps/prototype/tutorial/TutorialSteps.js | 52 +++++++++---------- views/base.jade | 2 +- 4 files changed, 70 insertions(+), 35 deletions(-) diff --git a/controllers/feedback/views/index.jade b/controllers/feedback/views/index.jade index 6494b91..cdbc2cd 100644 --- a/controllers/feedback/views/index.jade +++ b/controllers/feedback/views/index.jade @@ -1,5 +1,10 @@ extends ../../../views/base.jade +mixin message + span(style={'margin-bottom':'10px', 'margin-right': '10px'}) + block + br + mixin question(id, qu, info) label(for="#{id}", style={'margin-right':'10px'}) #{qu} if (info !== undefined) @@ -7,16 +12,30 @@ mixin question(id, qu, info) input.form-control(name="answer#{id}", type="text", placeholder="Answer") mixin rate(id, qu, info) - label(for="anwser#{id}", style={'margin-right':'10px'}) #{qu} - if (info !== undefined) - span.glyphicon.glyphicon-question-sign(type='button', data-toggle='tooltip', data-placement='top', title='#{info}') - input.rating(id="answer#{id}", name="answer#{id}", type="number", min='0', max='5', step='1', default='3') + div + label(for="anwser#{id}", style={'margin-right':'10px'}) #{qu} + if (info !== undefined) + span.glyphicon.glyphicon-question-sign(type='button', data-toggle='tooltip', data-placement='top', title='#{info}') + //-input.rating(id="answer#{id}", name="answer#{id}", type="number", min='0', max='5', step='1', default='3') + + br + input(type='radio', name='answerItem#{id}', value='1', style={'margin-right':'10px'}) + +message Very easy + input(type='radio', name='answerItem#{id}', value='2', style={'margin-right':'10px'}) + +message Easy + input(type='radio', name='answerItem#{id}', value='3', style={'margin-right':'10px'}) + +message Medium + input(type='radio', name='answerItem#{id}', value='4', style={'margin-right':'10px'}) + +message Hard + input(type='radio', name='answerItem#{id}', value='5', style={'margin-right':'10px'}) + +message Very hard + div(style={'margin-bottom':'10px'}) block extrahead link(rel="stylesheet", href="/static/css/feedback.css") link(rel="stylesheet", href="/static/css/star-rating.min.css") -block extrajs +//-block extrajs script(src="/static/js/star-rating.min.js") script $(function () { $('[data-toggle="tooltip"]').tooltip() }) script. @@ -30,8 +49,8 @@ block content form.form-signin(method="POST", action='/feedback-target') h2 Please give us your feedback - +rate(1, "What was the difficulty level without recommendation ?", "1 star if it was very easy, 5 stars if it was very difficult") - +rate(2, "What was the difficulty level with recommendation ?", "1 star if it was very easy, 5 stars if it was very difficult") + +rate(1, "What was the difficulty level without recommendation ?") + +rate(2, "What was the difficulty level with recommendation ?") +question(3, "Did the recommendations help you to find the coins ?") +question(4, "Did the recommendations help you to browse the scene ?") +question(5, "Do you think recommendations can be helpful ?") diff --git a/controllers/prototype/views/user_study.jade b/controllers/prototype/views/user_study.jade index b0772c3..c064a02 100644 --- a/controllers/prototype/views/user_study.jade +++ b/controllers/prototype/views/user_study.jade @@ -24,7 +24,18 @@ block extrabody Error : this id is already used ! block content - form.form-signin(method="POST", action='/identification') + script. + function validateForm() { + if (document.getElementById('sel1').selectedIndex === -1) { + alert('Select a correct age please'); + return false; + } + if (document.getElementById('sel2').selectedIndex === -1) { + alert('Select a correct date for the last time you played please'); + return false; + } + } + form.form-signin(method="POST", action='/identification', onsubmit='return validateForm()') h2 Please sign in label(for='inputId').sr-only Id input#inputId.form-control(name="inputId", type="text", placeholder='Id', required, autofocus) @@ -53,6 +64,9 @@ block content option(value='50-55') Between 50 and 55 option(value='55-60') Between 55 and 60 option(value='60-') More than 60 + script. + document.getElementById('sel1').selectedIndex = -1; + .form-group label(for='se21', style={'margin-right':'10px'}) When is the last time you played a 3D video game ? @@ -62,6 +76,8 @@ block content option(value='2') This month option(value='1') This year option(value='0') I never played a 3D video game + script. + document.getElementById('sel2').selectedIndex = -1; label(for='3dgames', style={'margin-right':'10px'}) Rate your 3D game skills diff --git a/js/l3d/apps/prototype/tutorial/TutorialSteps.js b/js/l3d/apps/prototype/tutorial/TutorialSteps.js index 5d15ac6..2fa3dfa 100644 --- a/js/l3d/apps/prototype/tutorial/TutorialSteps.js +++ b/js/l3d/apps/prototype/tutorial/TutorialSteps.js @@ -11,71 +11,71 @@ var TutorialSteps = function(tutoCamera, scene, coins, onWindowResize, container this.instructions = [ { - text:"Welcome to this tutorial ! Click on the canvas to go start !", + text:"Welcome to the tutorial. Click anywhere on the canvas to start. You will enter into the 'pointer lock' mode", justclick:false }, { - text: "You can use your mouse to move around, and press the escape key to unlock the pointer", + text: "Move your mouse to look around. Press the escape key to unlock the pointer.", justclick: false }, { - text: "You can also uncheck the lock pointer otion at the bottom of the page to rotate the camera via drag'n'drop !", + text: "You can also uncheck the lock pointer option at the bottom of the page to unlock the pointer.", justclick: false }, { - text:"You can use your mouse (drag'n'drop) to rotate the camera", + text:"Now, the pointer is unlocked. Click and drag your mouse to look around.", justclick:false }, { - text:"Nice ! You can also use (2,4,6 and 8) keys or (k,j,l and i)", + text:"Nice! You can also use the (2,4,6 and 8) keys or (k,j,l and i) keys to look around.", justclick: true }, { - text: "Here is a red coin, click on it !", + text: "Here is a red coin, click on it!", justclick: false }, { - text: "Nice, there are a 3 more red coins around you, try to get them ! Check the coin counter !", + text: "Nice, there are a 3 more red coins around you, try to get them! Check the gauge (your score)! You can lock the pointer if you wish.", justclick: false }, { - text: "Two more ! Feel free to select the pointer lock option you prefer", + text: "Two more!", justclick: false }, { - text: "One more !", + text: "One more!", justclick: false }, { - text:"Nice ! You will now learn to translate the camera", + text:"Nice! You will now learn to navigate in the scene.", justclick: true }, { - text: "Try pressing an arrow key on your keyboard to translate the camera !", + text: "Press one of the arrow keys on your keyboard to move!", justclick: false }, { - text: "There is a red coin on the roof of the castle, just in front of you ! Go and get it !", + text: "There is a red coin on the roof of the castle, just in front of you! Go and get it!", justclick: false }, { - text: "You got it ! Try to click on reset camera !", + text: "You got it! Now, click on the 'reset camera' button!", justclick: false }, { - text: "Nice ! Let me introduce you to recommendations", + text: "Nice! Let me introduce you to recommendations", justclick: true }, { - text: "This is a recommendation, by hovering it, you should see a preview, and by clicking on it, you should go to the recommended viewpoint", + text: "This is a recommendation, by hovering it, you should see a preview, and by clicking on it, you will automatically move towards the recommended viewpoint. Click on this recommendation to try.", justclick: false }, { - text: "The recommendation will change color once you clicked on it, just like a web link", + text: "The recommendation will change color once you clicked on it, just like a web link.", justclick:true }, { - text: "Recommendations can be displayed as 3D arrows like the one you just saw, or as viewports like this one", + text: "Recommendations can be displayed as 3D arrows like the one you just saw, or as viewports like this one. Click on this recommendation to proceed.", justclick: false }, { @@ -83,35 +83,35 @@ var TutorialSteps = function(tutoCamera, scene, coins, onWindowResize, container justclick:false }, { - text:"Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text:"Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick: false }, { - text: "Tip : you can use the previous / next buttons in the bar at the bottom of the screen to go to the previous / next position", + text: "Tip: You can use the previous / next buttons in the control bar at the bottom of the screen to go to the previous / next position", justclick:false }, { - text: "Congratulations ! You've successfully finished the tutorial ! Click on the green button to continue !", + text: "Congratulations! You've successfully collected all the coins and completed the tutorial ! Click on the green button to continue!", justclick: false } ]; @@ -266,7 +266,7 @@ TutorialSteps.prototype.notify = function(myString, justclick) { '' + myString + (justclick ? - ' (next)' : '' ) + + ' Click here to continue the tutorial. ' : '' ) + '' + '' ); diff --git a/views/base.jade b/views/base.jade index 15fc3d0..8d62811 100644 --- a/views/base.jade +++ b/views/base.jade @@ -10,7 +10,7 @@ html(lang='fr') link(rel="stylesheet", href="http://fonts.googleapis.com/css?family=Ubuntu:400,700,400italic") link(rel="icon", type="image/x-icon", href="/favicon.ico") block title - title Hello + title 3DUI body nav#nav.navbar.navbar-inverse.navbar-fixed-top(role="navigation")