From 62a966ada07007eafc666d4ad84e8fe14aacf681 Mon Sep 17 00:00:00 2001
From: Andrew Davis <andrew@moodle.com>
Date: Sun, 4 Mar 2012 12:21:03 +0700
Subject: [PATCH] MDL-31841 grade: prevented the feedback column on the user
 report from being centered

---
 grade/report/user/lib.php    | 11 ++++++++---
 grade/report/user/styles.css |  4 ++--
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/grade/report/user/lib.php b/grade/report/user/lib.php
index 9e823cfe211..b57e8a1c251 100644
--- a/grade/report/user/lib.php
+++ b/grade/report/user/lib.php
@@ -312,6 +312,7 @@ class grade_report_user extends grade_report {
         $hidden = '';
         $excluded = '';
         $class = '';
+        $classfeedback = '';
 
         // If this is a hidden grade category, hide it completely from the user
         if ($type == 'category' && $grade_object->is_hidden() && !$this->canviewhidden && (
@@ -368,6 +369,10 @@ class grade_report_user extends grade_report {
                 /// Actual Grade
                 $gradeval = $grade_grade->finalgrade;
 
+                if ($this->showfeedback) {
+                    // Copy $class before appending itemcenter as feedback should not be centered
+                    $classfeedback = $class;
+                }
                 $class .= " itemcenter ";
                 if ($this->showweight) {
                     $data['weight']['class'] = $class;
@@ -477,13 +482,13 @@ class grade_report_user extends grade_report {
                 // Feedback
                 if ($this->showfeedback) {
                     if ($grade_grade->overridden > 0 AND ($type == 'categoryitem' OR $type == 'courseitem')) {
-                    $data['feedback']['class'] = $class.' feedbacktext';
+                    $data['feedback']['class'] = $classfeedback.' feedbacktext';
                         $data['feedback']['content'] = get_string('overridden', 'grades').': ' . format_text($grade_grade->feedback, $grade_grade->feedbackformat);
                     } else if (empty($grade_grade->feedback) or (!$this->canviewhidden and $grade_grade->is_hidden())) {
-                        $data['feedback']['class'] = $class.' feedbacktext';
+                        $data['feedback']['class'] = $classfeedback.' feedbacktext';
                         $data['feedback']['content'] = '&nbsp;';
                     } else {
-                        $data['feedback']['class'] = $class.' feedbacktext';
+                        $data['feedback']['class'] = $classfeedback.' feedbacktext';
                         $data['feedback']['content'] = format_text($grade_grade->feedback, $grade_grade->feedbackformat);
                     }
                 }
diff --git a/grade/report/user/styles.css b/grade/report/user/styles.css
index b7751e17523..f0e5a3c9c31 100644
--- a/grade/report/user/styles.css
+++ b/grade/report/user/styles.css
@@ -33,8 +33,8 @@
 .user-grade td.item {border-left: 1px solid gray;border-right: 1px solid gray;}
 .user-grade td.excluded {background-color: #666;}
 .user-grade td.hidden {color: #aaa;}
-.user-grade td.feedbacktext {max-width:600px;padding:0;}
+.user-grade td.feedbacktext {max-width:600px;padding:2px 2px;}
 .pagelayout-report .user-grade .feedbacktext .no-overflow {overflow:auto;padding:0.25em;}
 
 table.user-grade td.feedbacktext {text-align:left;width: 40%;font-size: 0.8em;white-space:normal;}
-table.user-grade td.itemcenter {text-align:center;}
\ No newline at end of file
+table.user-grade td.itemcenter {text-align:center;}