feature/ArticleViewMobileAnsicht#27-first_submission#35
feature/ArticleViewMobileAnsicht#27-first_submission#35LuckyForce wants to merge 6 commits intomainfrom
Conversation
Maxi1324
left a comment
There was a problem hiding this comment.
Allgemeine Anmerkungen
Wenn du einen branch zu einem Issue erstellst, musst du ihn auch beim Issue referenzieren, dann bewegt sich das Issue im Board automatisch und es wir automatisch geschlossen beim mergen des Pull Requests und es ist besser erkenntlich, wo der Code zu dem Issue herkommt.

Bei Development kann man einen branch referenzieren oder halt direkt dort erstellen. Das zweite empfehle ich dir, weil es am schnellsten ist.
Achte immer darauf, dass der Code den du pushst auch immer zu dem Issue gehört und nicht random was anderes ist. Macht das debuggen bei Fehlern 1000 mal einfacher. Die ganzen Readme änderungen waren nicht Teil des Issues
Inhaltliches

Der Abstand zwischen den Pages fehlt in der Desktop View(Damit das nicht passiert hätte ich ein ngIf genutzt. Jetzt lappt das alles aufeinander und beim Ändern des einen Ändert man aus versehen das andere) Dadruch sieht man den Schatten der ItemCards der nächsten Page. Das ist nicht gut

was ist das für ein File? Wenn unnötig ist, weg damit
Die Dummy ItemCards werden in der Desktop view nicht mehr anzeigt. Wenn man ans Ende gelangt, ist da keiner freier Space mehr. Das bringt das ganzer ItemCard Deck auser Sync und das folgende Entsteht:
Außerdem macht das noch ganz viel anderen Müll, den ich jetzt nicht ausformulieren will.
.../user-frontend/src/app/Components/Pages/home-page/item-card-deck/item-card-deck.component.ts
Show resolved
Hide resolved
…com/ZID-Automat/Frontend-ZIDAutomat into feature/ArticleViewMobileAnsicht#27
a660ba4 to
9c68a9a
Compare

#27 ArticleView Mobile Ansicht
Card Deck has now a different behaviour on small screens (640px)