Skip to content
This repository has been archived by the owner on Jan 17, 2024. It is now read-only.

Task 180 Einführung #34

Merged
merged 5 commits into from
Dec 15, 2023
Merged

Task 180 Einführung #34

merged 5 commits into from
Dec 15, 2023

Conversation

moverval
Copy link
Contributor

OP#180

Der LogoScreen sollte direkt bei Start der App auftauchen. Eine Introduction zur Demo kann angeschaut werden indem unter der Map das Popup geöffnet wird und dann auf nicht speichern geklickt wird 👍

Copy link
Contributor

@IsAvaible IsAvaible left a comment

Choose a reason for hiding this comment

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

Sieht super aus! Habe noch kleine Anmerkungen für den Feinschliff gemacht :)

Allgemein:

  • Dart Fix ausführen, teilweise noch Widgets die als const markiert werden könnten.


controller!.forward();

Future.delayed(const Duration(seconds: 3), () {
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich finde 3 Sekunden "Künstlicher" Delay sind zu lang, wie wären 2 / 2.5?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, kann sein aber so stand das in OpenProject :/

Copy link
Contributor

Choose a reason for hiding this comment

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

Habe die Frage an Sophia weitergegeben.

Noch als Idee, die mir eingefallen ist:
Ab Android 12 gibt es native Unterstützung von Splash Screens (momentan wird der Splash Ccreen ja über den Router verwaltet). Dann würden wir uns auch den kurzen weißen Hintergrund beim Start der App sparen.

https://pub.dev/packages/flutter_native_splash

https://github.com/FH-SWE-2023/biersommelier/assets/76125864/60edc022-1b11-4ba2-bf61-628951bf41f1
(Insgesamt vergehen im Moment 6 Sekunden bis zum App-Start :0)

lib/pages/LogoScreen.dart Outdated Show resolved Hide resolved
@moverval
Copy link
Contributor Author

Ich würde das jetzt noch einmal so hochladen ohne ändern des Delays. Am Ende wenn alle einverstanden sind können wir das gerne auf 2.5 Sekunden stellen, mir ist das relativ egal

@moverval
Copy link
Contributor Author

Dinge sind gefixt, bitte gleich nochmal reviewen 👍

@Mats4k Mats4k merged commit 5d09417 into develop Dec 15, 2023
@IsAvaible
Copy link
Contributor

@Mats4k Warum gemerged? Es stand noch aus ob wir auf native Splash Screens wechseln.
Aber das können wir auch nachtragen

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

Successfully merging this pull request may close these issues.

3 participants