From 1000e6f000c54ec14099d6c462c5b3b61d7f746b Mon Sep 17 00:00:00 2001
From: Sander Roosendaal <roosendaalsander@gmail.com>
Date: Tue, 2 Mar 2021 11:52:36 +0100
Subject: [PATCH] better use of icons on PlannedSession management pages

---
 rowers/templates/plannedsessioncreate.html     | 12 +++++++-----
 rowers/templates/plannedsessionedit.html       | 10 +++++-----
 rowers/templates/plannedsessions.html          | 16 ++++++++--------
 rowers/templates/plannedsessionteamcreate.html | 13 +++++++------
 rowers/templates/plannedsessionteamedit.html   | 13 +++++++------
 rowers/templates/plannedsessionview.html       |  2 +-
 6 files changed, 35 insertions(+), 31 deletions(-)

diff --git a/rowers/templates/plannedsessioncreate.html b/rowers/templates/plannedsessioncreate.html
index 7f27f935..1258113a 100644
--- a/rowers/templates/plannedsessioncreate.html
+++ b/rowers/templates/plannedsessioncreate.html
@@ -80,14 +80,16 @@
           <td> {{ ps.sessionvalue }} </td>
           <td> {{ ps.sessionunit }} </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}/?when={{ timeperiod }}">Edit</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}/?when={{ timeperiod }}">
+							<i class="fas fa-pencil-alt fa-fw"></i></a>
           </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/user/{{ rower.user.id }}/?when={{ timeperiod }}/?when={{ timeperiod }}">Clone</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/user/{{ rower.user.id }}/?when={{ timeperiod }}/?when={{ timeperiod }}">
+							<i class="fas fa-clone fa-fw"></i></a>
           </td>
 
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/">Delete</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/"><i class="fas fa-trash-alt fa-fw"></i></a>
           </td>
 					<td>
 						<a class="small" href="/rowers/sessions/{{ ps.id }}/maketemplate/"><i class="fas fa-books fa-fw"></i></a>
@@ -138,10 +140,10 @@
           <td> {{ ps.sessionvalue }} </td>
           <td> {{ ps.sessionunit }} </td>
 					<td>
-						<a class="small" href="/rowers/sessions/{{ ps.id }}/templateedit/">Edit Template</a>
+						<a class="small" href="/rowers/sessions/{{ ps.id }}/templateedit/"><i class="fas fa-pencil-alt fa-fw"></i></a>
 					</td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/?next={{ request.path|urlencode }}">Delete</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/?next={{ request.path|urlencode }}"><i class="fas fa-trash-alt fa-fw"></i></a>
           </td>
 	</tr>
 	{% endfor %}
diff --git a/rowers/templates/plannedsessionedit.html b/rowers/templates/plannedsessionedit.html
index b3443795..96050549 100644
--- a/rowers/templates/plannedsessionedit.html
+++ b/rowers/templates/plannedsessionedit.html
@@ -53,8 +53,8 @@
 
       </div>
       <p>
-        <a href="/rowers/sessions/{{ thesession.id }}/deleteconfirm">Delete</a>
-        <a href="/rowers/sessions/{{ thesession.id }}/clone/?when={{ timeperiod }}">Clone</a>
+        <a href="/rowers/sessions/{{ thesession.id }}/deleteconfirm"><i class="fas fa-trash-alt fa-fw"></i> Delete</a>
+        <a href="/rowers/sessions/{{ thesession.id }}/clone/?when={{ timeperiod }}"><i class="fas fa-clone fa-fw"></i> Clone</a>
       </p>
       <input class="button"
              action="/rowers/sessions/{{ thesession.id }}/edit/user/{{ rower.user.id }}/" type="submit" value="Save">
@@ -96,14 +96,14 @@
           <td> {{ ps.sessionvalue }} </td>
           <td> {{ ps.sessionunit }} </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}">Edit</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}"><i class="fas fa-pencil-alt fa-fw"></i></a>
           </td>
           <td>
             <a class="small"
-               href="/rowers/sessions/{{ ps.id }}/clone/user/{{ rower.user.id }}/?when={{ timeperiod }}">Clone</a>
+               href="/rowers/sessions/{{ ps.id }}/clone/user/{{ rower.user.id }}/?when={{ timeperiod }}"><i class="fas fa-clone fa-fw"></i></a>
           </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm">Delete</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm"><i class="fas fa-trash-alt fa-fw"></i></a>
           </td>
         </tr>
         {% endfor %}
diff --git a/rowers/templates/plannedsessions.html b/rowers/templates/plannedsessions.html
index 0ac86245..1b583cc6 100644
--- a/rowers/templates/plannedsessions.html
+++ b/rowers/templates/plannedsessions.html
@@ -60,7 +60,7 @@
                href="/rowers/sessions/manage/session/{{ ps.id }}/user/{{ rower.user.id }}/?when={{ timeperiod }}">
               &nbsp;</a>
             {% else %}
-            <a 
+            <a
                class="{{ sessioncolor|lookup:ps.id }} dot"
                title="{{ completeness|lookup:ps.id }}"
                href="/rowers/virtualevent/{{ ps.id }}/submit">&nbsp;</a>
@@ -92,7 +92,7 @@
           <td>
             {% if ps.manager == request.user %}
             <a class="small"
-               href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}">Edit</a>
+               href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}"><i class="fas fa-pencil-alt fa-fw"></i></a>
             {% else %}
             &nbsp;
             {% endif %}
@@ -115,7 +115,7 @@
     planned workouts by purchasing the "Coach" or "Self-Coach" plans.
     {% endif %}
     <p>
-      <a 
+      <a
         href="/rowers/sessions/print/user/{{ rower.user.id }}/?when={{ timeperiod }}">
         Print View</a>
       &nbsp;
@@ -128,7 +128,7 @@
 	Actual vs Plan chart
 	</a>
     </p>
-    
+
   </li>
   <li class="grid_4">
     {% if unmatchedworkouts %}
@@ -162,7 +162,7 @@
             {% else %}
             <td>
               <a href={% url rower.defaultlandingpage id=workout.id|encode %}>No Name
-            </a></td> 
+            </a></td>
             {% endif %}
             {% else %}
             {% if workout.name != '' %}
@@ -176,12 +176,12 @@
             <td> {{ workout.duration |durationprint:"%H:%M:%S.%f" }} </td>
             <td> {{ workout.averagehr }} </td>
             <td> {{ workout.maxhr }} </td>
-          </tr>        
+          </tr>
           {% endfor %}
         </tbody>
       </table>
-      
-      
+
+
       {% endif %}
     </p>
   </li>
diff --git a/rowers/templates/plannedsessionteamcreate.html b/rowers/templates/plannedsessionteamcreate.html
index fd0befc0..1089acfc 100644
--- a/rowers/templates/plannedsessionteamcreate.html
+++ b/rowers/templates/plannedsessionteamcreate.html
@@ -66,6 +66,7 @@
               <th>Edit</th>
               <th>Clone</th>
               <th>Delete</th>
+							<th>To Library</th>
             </tr>
           </thead>
           <tbody>
@@ -85,17 +86,17 @@
               <td> {{ ps.sessionvalue }} </td>
               <td> {{ ps.sessionunit }} </td>
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/">Edit</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/"><i class="fas fa-pencil-alt fa-fw"></i></a>
               </td>
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/">Clone</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/"><i class="fas fa-clone fa-fw"></i></a>
               </td>
 
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/">Delete</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/"><i class="fas fa-trash-alt fa-fw"></i></a>
               </td>
 							<td>
-							 <a class="small" href="/rowers/sessions/{{ ps.id }}/maketemplate/">Save to Library</a>
+							 <a class="small" href="/rowers/sessions/{{ ps.id }}/maketemplate/"><i class="fas fa-books fa-fw"></i></a>
 						 </td>
             </tr>
             {% endfor %}
@@ -134,10 +135,10 @@
           <td> {{ ps.sessionvalue }} </td>
           <td> {{ ps.sessionunit }} </td>
 					<td>
-						<a class="small" href="/rowers/sessions/{{ ps.id }}/templateedit/">Edit Template</a>
+						<a class="small" href="/rowers/sessions/{{ ps.id }}/templateedit/"><i class="fas fa-pencil-alt fa-fw"></i></a>
 					</td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/">Delete</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm/"><i class="fas fa-trash-alt fa-fw"></i></a>
           </td>
 	</tr>
 	{% endfor %}
diff --git a/rowers/templates/plannedsessionteamedit.html b/rowers/templates/plannedsessionteamedit.html
index 160a374e..db7dabf5 100644
--- a/rowers/templates/plannedsessionteamedit.html
+++ b/rowers/templates/plannedsessionteamedit.html
@@ -61,10 +61,10 @@
         {{ form.as_table }}
       </table>
       <p>
-        <a href="/rowers/sessions/{{ plannedsession.id }}/deleteconfirm">Delete</a>
+        <a href="/rowers/sessions/{{ plannedsession.id }}/deleteconfirm"><i class="fas fa-trash-alt fa-fw"></i> Delete</a>
       </p>
       <p>
-        <a href="/rowers/sessions/{{ plannedsession.id }}/clone">Clone</a>
+        <a href="/rowers/sessions/{{ plannedsession.id }}/clone"><i class="fas fa-clone fa-fw"></i> Clone</a>
       </p>
       <p>
         <input type="submit" value="Save">
@@ -88,6 +88,7 @@
               <th>Edit</th>
               <th>Clone</th>
               <th>Delete</th>
+              <th>To Library</th>
             </tr>
           </thead>
           <tbody>
@@ -107,17 +108,17 @@
               <td> {{ ps.sessionvalue }} </td>
               <td> {{ ps.sessionunit }} </td>
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/edit">Edit</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/edit"><i class="fas fa-pencil-alt fa-fw"></i></a>
               </td>
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/clone">Clone</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/clone"><i class="fas fa-clone fa-fw"></i></a>
               </td>
 
               <td>
-                <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm">Delete</a>
+                <a class="small" href="/rowers/sessions/{{ ps.id }}/deleteconfirm"><i class="fas fa-trash-alt fa-fw"></i></a>
               </td>
               <td>
-    						<a class="small" href="/rowers/sessions/{{ ps.id }}/maketemplate/">Save</a>
+    						<a class="small" href="/rowers/sessions/{{ ps.id }}/maketemplate/"><i class="fas fa-books fa-fw"></i></a>
     					</td>
             </tr>
             {% endfor %}
diff --git a/rowers/templates/plannedsessionview.html b/rowers/templates/plannedsessionview.html
index a95a619f..5a3ccf5b 100644
--- a/rowers/templates/plannedsessionview.html
+++ b/rowers/templates/plannedsessionview.html
@@ -17,7 +17,7 @@
   <a href="/rowers/sessions/{{ psdict.id.1 }}/edit/user/{{ rower.user.id }}">
     Edit Session</a>
 		/
-	<a href="/rowers/sessions/{{ psdict.id.1 }}/maketemplate/?next={{ request.path|urlencode }}">Save to template</a>
+	<a href="/rowers/sessions/{{ psdict.id.1 }}/maketemplate/?next={{ request.path|urlencode }}">Save to Library</a>
 </p>
 {% endif %}
 <h1>Session {{ psdict.name.1 }}</h1>