From d96a28b6973382919c84f957000d976ea8d54ec7 Mon Sep 17 00:00:00 2001
From: Leonardo Canessa <lcanessa1@udayton.edu>
Date: Tue, 17 Jan 2017 15:46:01 -0500
Subject: [PATCH] SAKAI-3187 Improve student view

---
 .../attendance/tool/Attendance.properties           |  2 ++
 .../attendance/tool/pages/EventView.java            |  2 +-
 .../attendance/tool/pages/StudentView.html          |  1 +
 .../attendance/tool/pages/StudentView.java          |  9 ++++++++-
 .../tool/panels/AttendanceRecordFormDataPanel.html  |  3 ++-
 .../tool/panels/AttendanceRecordFormDataPanel.java  | 13 +++++++++----
 tool/src/webapp/css/attendance.css                  |  3 ++-
 tool/src/webapp/javascript/attendanceRecordForm.js  | 12 ++++++++++--
 8 files changed, 35 insertions(+), 10 deletions(-)

diff --git a/tool/src/java/org/sakaiproject/attendance/tool/Attendance.properties b/tool/src/java/org/sakaiproject/attendance/tool/Attendance.properties
index 3704b823..75e16b2e 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/Attendance.properties
+++ b/tool/src/java/org/sakaiproject/attendance/tool/Attendance.properties
@@ -100,6 +100,8 @@ attendance.record.form.cancel = Cancel
 attendance.record.form.add.comment = Add Comment
 attendance.record.form.view.comment = Comment
 
+attendance.no.record = Unrecorded
+
 attendance.icon.record.form.no_comment = Comment
 attendance.icon.record.form.yes_comment = Comment Present
 
diff --git a/tool/src/java/org/sakaiproject/attendance/tool/pages/EventView.java b/tool/src/java/org/sakaiproject/attendance/tool/pages/EventView.java
index 2c5c9488..87dfbc03 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/pages/EventView.java
+++ b/tool/src/java/org/sakaiproject/attendance/tool/pages/EventView.java
@@ -263,7 +263,7 @@ public void onClick() {
                 };
                 studentLink.add(stuName);
                 item.add(studentLink);
-                item.add(new AttendanceRecordFormDataPanel("record", item.getModel(), true, returnPage, feedbackPanel));
+                item.add(new AttendanceRecordFormDataPanel("record", item.getModel(), returnPage, feedbackPanel));
             }
         });
     }
diff --git a/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.html b/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.html
index c0f58840..fbade203 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.html
+++ b/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.html
@@ -94,6 +94,7 @@ <h3 class="takeAttendanceHeader">
             });
         </script>
     </div>
+    <label wicket:id="student-view-hidden" hidden id="studentView" aria-hidden="true"/>
 </wicket:extend>
 </body>
 </html>
\ No newline at end of file
diff --git a/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.java b/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.java
index eda92c5f..adaefda5 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.java
+++ b/tool/src/java/org/sakaiproject/attendance/tool/pages/StudentView.java
@@ -73,6 +73,13 @@ private void init() {
             hideNavigationLink(this.gradingLink);
         }
 
+        add(new Label("student-view-hidden","") {
+            @Override
+            public boolean isVisible() {
+                return isStudent;
+            }
+        });
+
         add(createHeader());
         add(createGrade());
         add(createStatistics());
@@ -208,7 +215,7 @@ public void onClick() {
                 }
                 item.add(eventLink);
                 item.add(new Label("event-date", item.getModelObject().getAttendanceEvent().getStartDateTime()));
-                item.add(new AttendanceRecordFormDataPanel("record", item.getModel(), false, returnPage, feedbackPanel));
+                item.add(new AttendanceRecordFormDataPanel("record", item.getModel(), returnPage, feedbackPanel));
             }
         };
 
diff --git a/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.html b/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.html
index 72e3eb48..7c1a76bc 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.html
+++ b/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.html
@@ -28,7 +28,7 @@
                 </div>
             </wicket:container>
         </div>
-        <div class="div-table-col comment-container" style="position:relative"  wicket:id="comment-container">
+        <div class="div-table-col comment-container" wicket:id="comment-container">
             <div class="commentToggle">
                 <div wicket:id="no-comment">
                     <span class="fa fa-comment-o fa-2x" aria-hidden="true"></span>
@@ -60,6 +60,7 @@ <h3><span wicket:id="add-header" /></h3>
             </script>
         </div>
     </form>
+    <div wicket:id="no-record" class="noRecord"><wicket:message key="attendance.no.record"/></div>
 </wicket:panel>
 </body>
 </html>
\ No newline at end of file
diff --git a/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.java b/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.java
index 4b0f2e25..5dabb845 100644
--- a/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.java
+++ b/tool/src/java/org/sakaiproject/attendance/tool/panels/AttendanceRecordFormDataPanel.java
@@ -48,7 +48,6 @@
 public class AttendanceRecordFormDataPanel extends BasePanel {
     private static final    long                        serialVersionUID = 1L;
     private                 IModel<AttendanceRecord>    recordIModel;
-    private                 boolean                     isStudentView;
     private                 boolean                     restricted ;
     private                 boolean                     showCommentsToStudents;
     private                 List<Component>             ajaxTargets = new ArrayList<Component>();
@@ -60,16 +59,16 @@ public class AttendanceRecordFormDataPanel extends BasePanel {
     private                 WebMarkupContainer          noComment;
     private                 WebMarkupContainer          yesComment;
 
-    public AttendanceRecordFormDataPanel(String id, IModel<AttendanceRecord> aR, boolean iS, String rP, FeedbackPanel fP) {
+    public AttendanceRecordFormDataPanel(String id, IModel<AttendanceRecord> aR,  String rP, FeedbackPanel fP) {
         super(id, aR);
         this.recordIModel = aR;
         this.oldStatus = aR.getObject().getStatus();
         this.showCommentsToStudents = recordIModel.getObject().getAttendanceEvent().getAttendanceSite().getShowCommentsToStudents();
-        this.isStudentView = iS;
         this.restricted = this.role != null && this.role.equals("Student");
         this.returnPage = rP;
         this.pageFeedbackPanel = fP;
         this.ajaxTargets.add(this.pageFeedbackPanel);
+
         add(createRecordInputForm());
     }
 
@@ -102,11 +101,17 @@ public boolean isEnabled() {
         createStatusRadio(recordForm);
         createCommentBox(recordForm);
 
+        boolean noRecordBool = recordForm.getModelObject().getStatus().equals(Status.UNKNOWN) && restricted;
+        recordForm.setVisibilityAllowed(!noRecordBool);
+
+        WebMarkupContainer noRecordContainer = new WebMarkupContainer("no-record");
+        noRecordContainer.setVisibilityAllowed(noRecordBool);
+        add(noRecordContainer);
+
         return recordForm;
     }
 
     private void createStatusRadio(final Form<AttendanceRecord> rF) {
-
         AttendanceStatusProvider attendanceStatusProvider = new AttendanceStatusProvider(attendanceLogic.getCurrentAttendanceSite(), AttendanceStatusProvider.ACTIVE);
         DataView<AttendanceStatus> attendanceStatusRadios = new DataView<AttendanceStatus>("status-radios", attendanceStatusProvider) {
             @Override
diff --git a/tool/src/webapp/css/attendance.css b/tool/src/webapp/css/attendance.css
index 2e47a3e8..e6422930 100644
--- a/tool/src/webapp/css/attendance.css
+++ b/tool/src/webapp/css/attendance.css
@@ -618,6 +618,8 @@ div.comment-container {
 
     .comment-container {
         float: right;
+        width: auto;
+        margin-right: .5em;
     }
 }
 
@@ -653,7 +655,6 @@ div.comment-container {
     .comment-container {
         float: right;
         margin-left: 2em;
-        margin-top: 5em;
     }
 }
 
diff --git a/tool/src/webapp/javascript/attendanceRecordForm.js b/tool/src/webapp/javascript/attendanceRecordForm.js
index b4c5c091..576edaa8 100644
--- a/tool/src/webapp/javascript/attendanceRecordForm.js
+++ b/tool/src/webapp/javascript/attendanceRecordForm.js
@@ -18,13 +18,16 @@
 
 (function (attendance, $, undefined) {
     // Set the status class (which applies a background) if the input is selected
-    function processInput (inputs) {
+    function processInput (inputs, isStudentView) {
         for(var i = 0; i < inputs.length; i++) {
             if(inputs[i].checked) {
                 var statusName = $(inputs[i]).siblings("label[hidden]").text();
                 $(inputs[i]).parent().addClass(statusName);
             } else {
                 $(inputs[i]).parent().removeClass().addClass("div-table-col");
+                if(isStudentView && $(window).width() < 1205) {
+                    $(inputs[i]).parent().removeClass().addClass("skip");
+                }
             }
         }
     }
@@ -47,6 +50,11 @@
             inputs.push(this);
         });
 
-        processInput(inputs);
+        var isStudentView = false;
+        if($("#studentView").size() > 0) {
+            isStudentView = true;
+        }
+
+        processInput(inputs, isStudentView);
     };
 }(window.attendance = window.attendance || {}, jQuery ));