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.

Sign up for GitHub

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

Jump to bottom

Bottom navbar #49

Merged
merged 5 commits into from
Dec 15, 2023
Merged

Bottom navbar #49

merged 5 commits into from
Dec 15, 2023

Conversation

alessioC42
Copy link
Owner

Es ist jetzt so, dass es standartmäßig die vier einträge

  • vertretungsplan
  • kalender
  • nachrichten
  • mein unterricht
    gibt. Was nicht unterstützt wird, wird aus der Navigationbar konmplett gestrichen.

Die Navigationbar in "Mein Unterricht" (main page) wurde durch eine TabView ausgetauscht.

@alessioC42 alessioC42 self-assigned this Dec 15, 2023
@kurwjan kurwjan mentioned this pull request Dec 15, 2023
Closed
7 tasks
kurwjan
kurwjan requested changes Dec 15, 2023
View reviewed changes
Copy link
Collaborator

@kurwjan kurwjan left a comment •
edited

Choose a reason for hiding this comment

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

ich muss noch für nachrichten hinzufügen

@alessioC42
Copy link
Owner Author

alessioC42 commented Dec 15, 2023 •
edited

Das sollte eigentlich für nachrichten Funktionieren.

@alessioC42
Copy link
Owner Author

alessioC42 commented Dec 15, 2023 •
edited

Ok, sehe gerade, dass es nicht geht.
edit: es geht doch, ich muss in meinem Testcase nur überall und nicht nur an der ersten Stelle doesSupportFeature auf true setzen

kurwjan
kurwjan previously approved these changes Dec 15, 2023
View reviewed changes
Copy link
Collaborator

@kurwjan kurwjan left a comment •
edited

Choose a reason for hiding this comment

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

Nachrichten hat jetzt auch eine TabBar (hat bisschen länger gebraucht als ich wollte)

@kurwjan kurwjan dismissed their stale review December 15, 2023 16:29

ein problem

kurwjan
kurwjan approved these changes Dec 15, 2023
View reviewed changes
Copy link
Collaborator

@kurwjan kurwjan left a comment •
edited

Choose a reason for hiding this comment

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

Jetzt kann man auch swipen bei nachrichten, unnötiger und schwammiger refactor (bei nachrichten), aber funktioniert

@alessioC42 alessioC42 merged commit b100006 into main Dec 15, 2023
@alessioC42 alessioC42 deleted the bottom_navbar branch December 15, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kurwjan kurwjan kurwjan approved these changes

Assignees

@alessioC42 alessioC42

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@alessioC42 @kurwjan