Skip to content

Revert "Update main-quote-summary designs/styles (#9612)"#9722

Merged
Gudahtt merged 1 commit intoVersion-v8.1.3from
revert-quote-summary-redesign
Oct 27, 2020
Merged

Revert "Update main-quote-summary designs/styles (#9612)"#9722
Gudahtt merged 1 commit intoVersion-v8.1.3from
revert-quote-summary-redesign

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Oct 26, 2020

This reverts commit 5456d55 (#9612)

Explanation:

This is being temporarily reverted because it has obscured whether the current quote is the best quote. This is a partial implementation of a design that moved the "Best Quote" label to the bottom section of the screen, which was outside the scope of this PR.

Manual testing steps:

  • Click the "Swap" button on the home page
  • Select something for the source and destination token
  • Wait for quotes to be received
  • See that on the view quote screen, the current quote is marked as the "best quote", if there is a best quote.

@Gudahtt Gudahtt requested review from a team and whymarrh and removed request for a team October 26, 2020 20:36
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [ee539b4]
Page Load Metrics (390 ± 50 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint318739126
domContentLoaded26164638910450
load26364739010450
domInteractive26164638810450

Copy link
Copy Markdown
Contributor

@danjm danjm left a comment

Choose a reason for hiding this comment

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

LGTM. Checked the code diff against the PR being reverted and QA'd in the browser

@Gudahtt Gudahtt merged commit e0a35dc into Version-v8.1.3 Oct 27, 2020
@Gudahtt Gudahtt deleted the revert-quote-summary-redesign branch October 27, 2020 19:23
@github-actions github-actions bot locked and limited conversation to collaborators Oct 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants