ux: improve some user-facing texts

controllers/deleteOfficeHourHandler.go

@@ -45,7 +45,7 @@ func (b *BaseHandler) DeleteOfficeHourHandler(w http.ResponseWriter, req *http.R
 			templating.ServeTemplate(w, "deleteMailForm", struct {
 				Error      string
 				OfficeHour models.OfficeHour
-			}{"Die Sprechstunde wurde nicht mit dieser Mailadresse angegeben.", officeHour})
+			}{"Die Sprechstunde wurde nicht mit dieser Mailadresse angelegt.", officeHour})
 			return
 		}
 		_, err = b.requestRepo.Add(officeHour, models.RequestDelete)

controllers/timetable.go

@@ -51,6 +51,7 @@ func (b *BaseHandler) GetTimetable(officeHours []models.OfficeHour) (timetable m
 
 func (b *BaseHandler) printTimetable(timetable map[models.Date]map[int]models.OfficeHour, slots []int, deleteIcons bool) template.HTML {
 	var tableBody string
+	var anyOfficeHourExists bool = false // do not show table if no entry in there
 	for hour := 8; hour < 19; hour += 1 {
 		for minute := 0; minute < 60; minute += b.config.Date.MinuteGranularity {
 			tableBody += "<tr>"
@@ -93,6 +94,7 @@ func (b *BaseHandler) printTimetable(timetable map[models.Date]map[int]models.Of
 								// TODO: better error wrapping up to top-level request handler
 							}
 							tableBody += celldata.String()
+							anyOfficeHourExists = true
 						}
 					} else {
 						if slot+1 == slots[day] {
@@ -106,6 +108,9 @@ func (b *BaseHandler) printTimetable(timetable map[models.Date]map[int]models.Of
 			tableBody += "</tr>\n"
 		}
 	}
+	if !anyOfficeHourExists {
+		return template.HTML("Aktuell sind keine passenden Sprechstunden eingetragen.")
+	}
 	var table bytes.Buffer
 
 	tableData := struct {

templating/templates/addSuccess.html

@@ -2,6 +2,8 @@
 
 {{define "content"}}
 <div class="col-md-8 offset-md-2">
-  Die Sprechstunde wurde angelegt. Du solltest eine Mail mit einem Aktivierungslink erhalten haben.
+  Die Sprechstunde wurde angelegt.
+  Du solltest eine Mail mit einem Aktivierungslink erhalten haben.
+  Klicke auf diesen, um die Sprechstunde öffentlich anzuzeigen.
 </div>
 {{end}}

templating/templates/deleteSuccess.html

@@ -2,7 +2,7 @@
 
 {{define "content"}}
 <div class="col-md-8 offset-md-2">
-  Du solltest eine Mail mit einem Bestätigungslink erhalten haben. <br>
+  Du solltest eine Mail mit einem Bestätigungslink erhalten haben.
-  Sie wurde an die Adresse geschickt, mit der die Sprechstunde angelegt wurde.<br>
+  Klicke auf diesen, um die Sprechstunde endgültig zu löschen.
 </div>
 {{end}}

templating/templates/td.html

@@ -4,7 +4,8 @@
   {{if eq .OfficeHour.Date.Week 1}}in ungeraden Vorlesungswochen<br>{{end}}{{if eq .OfficeHour.Date.Week 2}}in geraden Vorlesungswochen<br>{{end}}
   {{.OfficeHour.Course.Name}}<br>
   {{.OfficeHour.Tutor.Name}}<br>
-  {{.OfficeHour.Room.Name}}<br>
+  {{.OfficeHour.Room.Name}}
-  {{if ne .OfficeHour.Info ""}}{{.OfficeHour.Info}}<br>{{end}}
+  {{- if ne .OfficeHour.RoomName ""}}: {{.OfficeHour.RoomName}}{{end -}}
-  {{if ne .OfficeHour.RoomName ""}}{{.OfficeHour.RoomName}}<br>{{end}}
+  <br>
+  {{if ne .OfficeHour.Info ""}}{{.OfficeHour.Info}}{{end}}
 </td>