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

[#173363018] ActivateBonusRequestScreen graphical review #1922

Merged

Conversation

fabriziofff
Copy link
Contributor

@fabriziofff fabriziofff commented Jun 18, 2020

This pr depends on (#1909)

Short description:
This pr introduces some graphical adjustment for the ActivateBonusRequestScreen.

List of changes proposed in this pull request:

  • Remove horizontal spacing for the KeyValueTable rows.
  • Change vertical Align to top for the KeyValueTable rows.
  • The subtitle now is correcly aligned with the image.
  • The bonus image is loaded from remote url, instead of local asset (also for the ActivateBonusCompletedScreen).
  • The Alert (cancel bonus or confirm activation) now can be dismissed when tap outside (Android only)

fabriziofff and others added 30 commits June 10, 2020 13:01
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jun 18, 2020

Affected stories

  • 🌟 #173363018: Rework grafico per la schermata ActivateBonusRequestScreen

Generated by 🚫 dangerJS

@Undermaken Undermaken merged commit f157673 into master Jun 19, 2020
@Undermaken Undermaken deleted the 173363018-activatebonusrequestscreen-graphical-review branch June 19, 2020 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants