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

ALFMOB-89: Error screen header not fixed to the top #30

Merged

Conversation

JoaoPinhoMinder
Copy link
Contributor

@JoaoPinhoMinder JoaoPinhoMinder commented Feb 12, 2025

Ticket

https://mindera.atlassian.net/browse/ALFMOB-89

Description

The divider in the toolbar was not fixed to the top. To resolve this, we now use frame and alignment to anchor it at the top. However, this also causes views that should be centered within the remaining space to align at the top. To address this, some error views required a Spacer at the top and bottom. As part of this fix, a generic `ErrorView was created for use in all error scenarios.

Evidences

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
The divider in the toolbar was not fixed to the top. To resolve this, we now use `frame` and `alignment` to anchor it at the top. However, this also causes views that should be centered within the remaining space to align at the top. To address this, some error views required a `Spacer` at the top and bottom. As part of this fix, a generic `ErrorView was created for use in all error scenarios.
Copy link
Member

@p4checo p4checo left a comment

Choose a reason for hiding this comment

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

nice improvement!

Copy link
Member

@p4checo p4checo left a comment

Choose a reason for hiding this comment

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

Nice! Just some minor suggestions

@JoaoPinhoMinder JoaoPinhoMinder merged commit f1fc26f into main Mar 11, 2025
2 checks passed
@JoaoPinhoMinder JoaoPinhoMinder deleted the bug/ALFMOB-89-error-screen-header-not-fixed-to-the-top branch March 11, 2025 11:25
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.

None yet

3 participants