Add internal notes to grants recap

Author: estyxx

backend/reviews/admin.py

@@ -292,6 +292,12 @@ def _review_grants_recap_view(self, request, review_session):
                 if key.startswith("reimbursementcategory-")
             }
 
+            notes_updates = {
+                int(key.split("-")[1]): value
+                for [key, value] in data.items()
+                if key.startswith("notes-")
+            }
+
             grants = list(
                 review_session.conference.grants.filter(id__in=decisions.keys()).all()
             )
@@ -393,6 +399,17 @@ def _review_grants_recap_view(self, request, review_session):
                             change_message=f"[Review Session] Reimbursement {reimbursement.category.name} added.",
                         )
 
+            # Update internal notes for all grants that have notes changes
+            if notes_updates:
+                grants_to_update_notes = review_session.conference.grants.filter(
+                    id__in=notes_updates.keys()
+                ).all()
+                for grant in grants_to_update_notes:
+                    new_notes = notes_updates.get(grant.id, "")
+                    if grant.internal_notes != new_notes:
+                        grant.internal_notes = new_notes
+                        grant.save(update_fields=["internal_notes"])
+
             messages.success(
                 request, "Decisions saved. Check the Grants Summary for more info."
             )

backend/reviews/templates/grants-recap.html

@@ -181,6 +181,93 @@
   tr:nth-of-type(odd) {
     background-color: var(--body-bg);
   }
+
+  /* Tooltip */
+  .tooltip {
+    position: relative;
+    display: inline-block;
+  }
+
+  .tooltip .tooltiptext {
+    visibility: hidden;
+    opacity: 0;
+    width: 220px;
+    background-color: #333;
+    color: #fff;
+    text-align: left;
+    text-transform: none;
+    padding: 8px 10px;
+    border-radius: 6px;
+    position: absolute;
+    z-index: 1;
+    top: 125%;
+    left: 50%;
+    margin-left: -110px;
+    font-size: 12px;
+    font-weight: normal;
+    line-height: 1.4;
+    transition: opacity 0.2s ease-in-out 0.5s, visibility 0.2s ease-in-out 0.5s;
+  }
+
+  .tooltip .tooltiptext::after {
+    content: "";
+    position: absolute;
+    bottom: 100%;
+    left: 50%;
+    margin-left: -5px;
+    border-width: 5px;
+    border-style: solid;
+    border-color: transparent transparent #333 transparent;
+  }
+
+  .tooltip:hover .tooltiptext {
+    visibility: visible;
+    opacity: 1;
+  }
+
+  /* Sortable columns */
+  .sortable {
+    cursor: pointer;
+    user-select: none;
+  }
+
+  .sortable:hover {
+    text-decoration: underline;
+  }
+
+  .notes-cell {
+    position: relative;
+    min-width: 220px;
+  }
+
+  .notes-textarea {
+    position: absolute;
+    top: 8px;
+    left: 8px;
+    right: 8px;
+    bottom: 8px;
+    width: calc(100% - 16px);
+    min-height: 80px;
+    resize: none;
+    font-size: 12px;
+    padding: 5px;
+    box-sizing: border-box;
+  }
+
+  /* Center-align columns */
+  .results-table th:nth-child(1), /* number column */
+  .results-table th:nth-child(3), /* score column */
+  .results-table th:nth-child(4), /* std dev column */
+  .results-table th:nth-child(6), /* current status column */
+  .results-table th:nth-child(7), /* pending status column */
+  .results-table td:nth-child(1), /* number column */
+  .results-table td:nth-child(3), /* score column */
+  .results-table td:nth-child(4), /* std dev column */
+  .results-table td:nth-child(6), /* current status column */
+  .results-table td:nth-child(7) /* pending status column */
+  {
+    text-align: center;
+  }
 </style>
 <script type="application/javascript">
   const grantsById = {};
@@ -462,6 +549,12 @@ <h3>
                   </div>
                   <div class="clear"></div>
                 </th>
+                <th scope="col">
+                  <div class="text">
+                    <span>Notes</span>
+                  </div>
+                  <div class="clear"></div>
+                </th>
               </tr>
             </thead>
             <tbody>
@@ -659,6 +752,17 @@ <h3>
                   </ul>
                   {% else %} No permission to change. {% endif %}
                 </td>
+                <td class="notes-cell">
+                  {% if perms.reviews.decision_reviewsession %}
+                  <textarea
+                    name="notes-{{ item.id }}"
+                    class="notes-textarea"
+                    placeholder="Internal notes..."
+                  >{{ item.internal_notes }}</textarea>
+                  {% else %}
+                  {{ item.internal_notes|default:"-" }}
+                  {% endif %}
+                </td>
               </tr>
               {% endfor %}
             </tbody>