Skip to content

feature/ArticleViewMobileAnsicht#27-first_submission#35

Open
LuckyForce wants to merge 6 commits intomainfrom
feature/ArticleViewMobileAnsicht#27
Open

feature/ArticleViewMobileAnsicht#27-first_submission#35
LuckyForce wants to merge 6 commits intomainfrom
feature/ArticleViewMobileAnsicht#27

Conversation

@LuckyForce
Copy link
Copy Markdown
Member

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

@LuckyForce LuckyForce added enhancement New feature or request UserFrontend Is part of the UserFrontend labels Nov 14, 2022
@LuckyForce LuckyForce requested a review from Maxi1324 November 14, 2022 23:40
Copy link
Copy Markdown
Member

@Maxi1324 Maxi1324 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
image
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

image
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

image
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:

image

Außerdem macht das noch ganz viel anderen Müll, den ich jetzt nicht ausformulieren will.

@Maxi1324 Maxi1324 linked an issue Nov 16, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request UserFrontend Is part of the UserFrontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ArticleView Mobile Ansicht

2 participants