Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix some styling and translations in the gas customization modal #5919

Merged
merged 3 commits into from
Dec 12, 2018
Merged

Fix some styling and translations in the gas customization modal #5919

merged 3 commits into from
Dec 12, 2018

Conversation

danjm
Copy link
Contributor

@danjm danjm commented Dec 12, 2018 •
edited
Loading

  • adds missing translations to the gas customization modal
  • fixes the width of the modal and the send screen gas buttons when viewing through the notification window
  • shows the transaction fee instead of the transaction total in the top left of the advanced tab

@danjm danjm force-pushed the gas-customization-fixes branch from 81a4fef to dabf263 Compare December 12, 2018 16:35
@metamaskbot
Copy link
Collaborator

Builds ready [dabf263]: mascara, chrome, firefox, edge, opera

@danjm danjm force-pushed the gas-customization-fixes branch from e5ac77e to 66f7102 Compare December 12, 2018 16:58
@metamaskbot
Copy link
Collaborator

Builds ready [66f7102]: mascara, chrome, firefox, edge, opera

@danjm danjm merged commit 970bdeb into develop Dec 12, 2018
@whymarrh whymarrh deleted the gas-customization-fixes branch December 12, 2018 17:10
@danjm danjm mentioned this pull request Dec 12, 2018
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@whymarrh whymarrh whymarrh approved these changes

@alextsg alextsg Awaiting requested review from alextsg

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@danjm @metamaskbot @whymarrh