Issue 1575:Refine NUMSCALE question UI at InstructorFeedbackEdit

Issue 1575:Refine NUMSCALE question UI at InstructorFeedbackEdit

[arnold.koroa, 2014-02-18 07:57:56]

https://codereview.appspot.com/65090044/diff/1/src/main/java/teammates/common...
File src/main/java/teammates/common/util/Const.java (right):

https://codereview.appspot.com/65090044/diff/1/src/main/java/teammates/common...
src/main/java/teammates/common/util/Const.java:147: public static final String
FEEDBACK_SESSION_NUMSCALE_MAX = "Maximum value of acceptable responses";
Maximum acceptable response value

https://codereview.appspot.com/65090044/diff/1/src/main/java/teammates/common...
src/main/java/teammates/common/util/Const.java:148: public static final String
FEEDBACK_SESSION_NUMSCALE_INCREMENT = "Value that can be increased each time";
Value to be increased/decreased each step

https://codereview.appspot.com/65090044/diff/1/src/main/java/teammates/common...
src/main/java/teammates/common/util/Const.java:149: public static final String
FEEDBACK_SESSION_NUMSCALE_MIN = "Minimum value of acceptable responses";
Minimum acceptable response value

https://codereview.appspot.com/65090044/diff/1/src/main/resources/feedbackQue...
File src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html (left):

https://codereview.appspot.com/65090044/diff/1/src/main/resources/feedbackQue...
src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html:27: <span
id="numScalePossibleValues-${questionNumber}">${possibleValues}</span>
You need to replace ${possibleValues} with "Based on the above settings,
acceptable responses are.." in FeedbackNumericalScaleQuestionDetails instead of
"Possible Values.." too.

https://codereview.appspot.com/65090044/diff/1/src/main/resources/feedbackQue...
File src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html (right):

https://codereview.appspot.com/65090044/diff/1/src/main/resources/feedbackQue...
src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html:24: </tr>
Add the tooltips here too?

[arnold.koroa, 2014-02-19 07:59:39]

https://codereview.appspot.com/65090044/diff/20001/src/main/java/teammates/co...
File src/main/java/teammates/common/util/Const.java (right):

https://codereview.appspot.com/65090044/diff/20001/src/main/java/teammates/co...
src/main/java/teammates/common/util/Const.java:150: 
change the variable names to "FEEDBACK_QUESTION..." and put them below together
with the other feedback question tooltip?

https://codereview.appspot.com/65090044/diff/20001/src/main/resources/feedbac...
File src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html (right):

https://codereview.appspot.com/65090044/diff/20001/src/main/resources/feedbac...
src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html:5: <td
onmouseover="ddrivetip('${Const.ParamsNames.FEEDBACK_SESSION_NUMSCALE_MIN}')"
change the tooltip variable name accordingly together with the one in Const

[arnold.koroa, 2014-02-20 02:12:30]

https://codereview.appspot.com/65090044/diff/30001/src/main/java/teammates/co...
File src/main/java/teammates/common/util/Const.java (right):

https://codereview.appspot.com/65090044/diff/30001/src/main/java/teammates/co...
src/main/java/teammates/common/util/Const.java:173: public static final String
FEEDBACK_QUESTION_NUMSCALE_MIN_VALUE = "Minimum acceptable response value";
Oh,now I realized why the name clashed. This is a tooltip and should be placed
under Tooltips, not ParamNames. Do move it there then the name will no longer
clash.

[arnold.koroa, 2014-02-21 02:28:53]

https://codereview.appspot.com/65090044/diff/40001/src/main/java/teammates/co...
File
src/main/java/teammates/common/datatransfer/FeedbackNumericalScaleQuestionDetails.java
(right):

https://codereview.appspot.com/65090044/diff/40001/src/main/java/teammates/co...
src/main/java/teammates/common/datatransfer/FeedbackNumericalScaleQuestionDetails.java:86:
"${Const.ParamsNames.FEEDBACK_QUESTION_NUMSCALE_TOOLTIP_STEP}",
Const.Tooltips.FEEDBACK_QUESTION_NUMSCALE_STEP);
Change "${Const.ParamsNames.FEEDBACK_QUESTION_NUMSCALE_TOOLTIP_*"
to "${Const.Tooltips.FEEDBACK_QUESTION_NUMSCALE_*"?

https://codereview.appspot.com/65090044/diff/40001/src/main/resources/feedbac...
File src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html (right):

https://codereview.appspot.com/65090044/diff/40001/src/main/resources/feedbac...
src/main/resources/feedbackQuestionNumScaleEditFormTemplate.html:5: <td
onmouseover="ddrivetip('${Const.ParamsNames.FEEDBACK_QUESTION_NUMSCALE_TOOLTIP_MIN}')"
"${Const.Tooltips.FEEDBACK_QUESTION_NUMSCALE_MIN"?