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

Feature/basal account page gsi 1296 #55

Merged
merged 6 commits into from
Feb 5, 2025

Conversation

SilverLinings89
Copy link
Member

As per issue 1296. Layout different than in old ui. May need some work still.

@SilverLinings89 SilverLinings89 self-assigned this Feb 3, 2025
@SilverLinings89 SilverLinings89 requested a review from Cito February 3, 2025 14:03
Copy link
Member

@Cito Cito left a comment

Choose a reason for hiding this comment

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

This page would look a bit less boring with some contrasting colors and icons.

E.g. the heading could have a larger "user profile" icon and the individual sections smaller icons on the top left corner.

We can also experiment with giving the card headers an orange background with white text or something like that.

But we can also do this in an additional "polish account page" task.

src/app/app.routes.ts Outdated Show resolved Hide resolved
src/app/app.routes.ts Outdated Show resolved Hide resolved
src/app/portal/features/account/account.component.html Outdated Show resolved Hide resolved
src/app/portal/features/account/account.component.ts Outdated Show resolved Hide resolved
Cito
Cito previously approved these changes Feb 3, 2025
@Cito's suggestions.

Co-authored-by: Christoph Zwerschke <[email protected]>
Signed-off-by: Pascal Kraft <[email protected]>
@SilverLinings89 SilverLinings89 merged commit cea8122 into main Feb 5, 2025
3 checks passed
@SilverLinings89 SilverLinings89 deleted the feature/basal-account-page-gsi-1296 branch February 5, 2025 11:07
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.

2 participants