fix(3156126186): align transactions modals to Figma

galuzan-starkware commented 2 years ago

Not include Steppers icon and title design

Changes:

Adding margin-top Alert - .message class to make space between title and message.
CSS changes to align to Figma design
Adding style prop to ModalText (used to centralize the text in my case)
Create new folders TransactionSubmittedModal and ProgressModal and separate them to header and body instead of passing only the title in modal-hooks functions (in order to give each modal his own CSS properties)
Adding buttonProp to TransactionSubmittedModalButton and passing the same styling to it and to the close modal button (to avoid code duplication)
Align strings to be like the design, on strings.js I changed the waitForConfirm object to be with strings that match the design on Figma

[x] Manually tests of the main Application flows are done and passed.
[ ] New unit / functional tests have been added (whenever applicable).
[ ] Docs have been updated (whenever relevant).
[x] PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is

netlify[bot] commented 2 years ago

Name	Link
Latest commit	fa384d7eab9d7175b7206509d96113be2c9ea8df
Latest deploy log	https://app.netlify.com/sites/goerli-starkgate/deploys/631f29197dee220009999138
Deploy Preview	https://deploy-preview-313--goerli-starkgate.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] commented 2 years ago

Name	Link
Latest commit	fa384d7eab9d7175b7206509d96113be2c9ea8df
Latest deploy log	https://app.netlify.com/sites/integration-starkgate/deploys/631f2919f0895a000945f7d0
Deploy Preview	https://deploy-preview-313--integration-starkgate.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

dan-ziv commented 2 years ago

:tada: This PR is included in version 1.9.0-dev.28 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

dan-ziv commented 1 year ago

:tada: This PR is included in version 1.9.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

dan-ziv commented 1 year ago

:tada: This PR is included in version 1.9.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

starknet-io / starkgate-frontend