Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Startseite news #187

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Startseite news #187

wants to merge 6 commits into from

Conversation

duoistwo
Copy link
Contributor

@duoistwo duoistwo commented Sep 5, 2022

Close #139

Typ

Was für eine Art von Veränderungen wird hier vorgenommen?
Lösche die nicht zutreffenden

  • man hat jetzt news auf der startseite

Implementation

Welche Ideen gab es zur Implementation? Was galt es zu verbessern? Bitte möglichst detailliert die Implementation erklären

Checklist

  • das ganze basiert auf dem code für die termine

Alle erfüllten Boxen ankreuzen. Diese Liste kann auch nach Erstellung der Pull Request vervollständigt werden. Unter diesen Gesichtspunkten wird die Implementation begutachtet.

  • Ich habe die Tests erweitert um zu zeigen, dass der Bugfix/das Feature funktioniert
  • Der neue Code hält sich an die Coding Standards
  • Im Code befinden sich wichtige Kommentare, falls angemessen

@strifel
Copy link
Member

strifel commented Sep 5, 2022

   info • Unused import: '../Views/News.dart' • lib/api/api_helpers.dart:6:8 • unused_import
   info • Unused import: 'dart:html' • lib/components/home.dart:1:8 • unused_import
   info • Sort directive sections alphabetically • lib/components/home.dart:4:1 • directives_ordering
   info • Sort directive sections alphabetically • lib/components/home.dart:16:1 • directives_ordering

@themathman2
Copy link

@RobinArlinghaus Wenn ich das auf verschiedenen Endgeräten begutachten soll, dann muss der Testbild gebaut werden. Dafür muss aber der Branche aktuell genug sein.

@strifel
Copy link
Member

strifel commented Sep 7, 2022

Der ist gebaut
https://app.test-deploy.kag-langenfeld.de/187/

Copy link

@themathman2 themathman2 left a comment

Choose a reason for hiding this comment

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

Ich finde, dass die grauen Border nicht zum Design der Seite passen. Die Border an sich sind, glaube ich, für mich das Problem. Der Rest der Seite ist relativ offen gestaltet.
Evt. könnte ein leicht grauer Kasten, der sich etwas vom hellen Hintergrund absetzt und auch einen kleinen Abstand zum nächsten Artikel hat, eine Lösung sein. Aber sicher bin ich mir nicht.

@strifel
Copy link
Member

strifel commented Sep 11, 2022

Eventuell das Artikelbild auch mit anzeigen.
Und ich könnte mir vorstellen das zur Seite Scrollbar OK aussieht

@themathman2
Copy link

Bei mir auf dem Handy kann ich mit etwas Geschick zum zweiten Bild/Artikel scrollen. Sobald nur noch die Artikelbilder auf dem Screen sind, kann ich nicht mehr scrollen. Ich vermute, dass man nicht mehr in die Box kommt, die die Scrollbar enthält, sondern von den Artikelbildboxen "gefangen" wird.

@strifel
Copy link
Member

strifel commented Oct 28, 2022

Eventuell ein Column statt einer ListView verwenden

@themathman2
Copy link

Testdeploy hängt bei mir.

@strifel
Copy link
Member

strifel commented Oct 29, 2022

Eventuell ein Column statt einer ListView verwenden

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

News auf der Startseite
3 participants