mam/mamweb
11
0
Fork 0
Code Issues Pull requests 9 Projects Wiki Activity Actions

Výroční sraz M&M #32

Merged
zelvuska merged 12 commits from vyroci into master 2023-06-15 16:57:27 +02:00
Conversation 1 Commits 12 Files changed 13 +3 -3
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 6f3fe7f0da - Show all commits

6
vyroci/templates/vyroci/vyroci_list.html
View file

@ -17,9 +17,9 @@
{% for obj in ucastnikvyroci_list %} {% for obj in ucastnikvyroci_list %}
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-13 19:10:03 +02:00
Outdated
Review
Copy link

Tohle funguje? Měl jsem za to, že ListView vyrábí v kontextu object_list… (A možná bych to malinko preferoval, kvůli konzistenci; byť by asi rozumné naopak bylo plošně všechny object_list přepsat na to, seznamy čeho to reálně jsou. Asi spíš výhledově, do Kanboardu…)

Jo, a když už jsme u toho, tak bych neobjektifikoval účastníky a tu iterační proměnou pojmenoval třeba u. Ale taky je to spíš taková poznámka, která by se spíš týkala nějaké dlouhodobější stránky, než téhle…

Tohle funguje? Měl jsem za to, že `ListView` vyrábí v kontextu `object_list`… (A možná bych to malinko preferoval, kvůli konzistenci; byť by asi rozumné naopak bylo plošně všechny `object_list` přepsat na to, seznamy čeho to reálně jsou. Asi spíš výhledově, do Kanboardu…) Jo, a když už jsme u toho, tak bych neobjektifikoval účastníky a tu iterační proměnou pojmenoval třeba `u`. Ale taky je to spíš taková poznámka, která by se spíš týkala nějaké dlouhodobější stránky, než téhle…
<tr> <tr>
<td style="word-break: break-all">{{ obj.jmeno }}</td> <td>{{ obj.jmeno }}</td>
<td style="word-break: break-all">{{ obj.prezdivka }}</td> <td>{{ obj.prezdivka }}</td>
<td style="word-break: break-all">{{ obj.email }}</td> <td>{{ obj.email }}</td>
<td style="word-break: break-all">{{ obj.kdy }}</td> <td style="word-break: break-all">{{ obj.kdy }}</td>
<td style="word-break: break-all">{{ obj.cojemam }}</td> <td style="word-break: break-all">{{ obj.cojemam }}</td>
<td style="word-break: break-all">{{ obj.cislo }}</td> <td style="word-break: break-all">{{ obj.cislo }}</td>