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

Refactor app layout #230

Open
daniel-mader opened this issue May 14, 2024 · 1 comment
Open

Refactor app layout #230

daniel-mader opened this issue May 14, 2024 · 1 comment
Labels
Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality.

Comments

@daniel-mader
Copy link
Collaborator

daniel-mader commented May 14, 2024 •
edited
Loading

Description

The app layout needs refactoring to reduce technical debt. A generic "root layout" should be defined that covers the following aspects:

  • handles y-overflow correctly (no cut-offs, no unnecessary white space)
  • some pages have a top navigation (but not all pages)
  • some pages have a bottom navigation (but not all pages)
  • respect "safe areas" (iOS)

Motivation

Reduce technical debt

Requirements

see Description

Open Questions

No response

Are you planning to contribute this in a PR?

Yes
@daniel-mader daniel-mader added the Chore Non-functional change, e.g., changing a config. label May 14, 2024
@maiertech maiertech added Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality. and removed Chore Non-functional change, e.g., changing a config. labels Jul 3, 2024
@M-Adam-Hus
Copy link

Refactor app layout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality.
Projects
Impierce SSI Stack
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@maiertech @daniel-mader @M-Adam-Hus