Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

1.12.0 - Revert Fee Breakdown Tooltip #2596

Merged
merged 2 commits into from
Mar 28, 2022

Conversation

W3stside
Copy link
Contributor

Summary

Part of the last 3 checkboxes in #2573

Reverts the changes to fee breakdown that weren't desired - now only shows the fee and the discounted amount (if any)

image

@W3stside W3stside requested review from a team March 28, 2022 09:24
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@elena-zh
Copy link

elena-zh commented Mar 28, 2022

I vote for showing fee with discount label in green. Or at least a discount value.
image

Copy link

@elena-zh elena-zh left a comment

Choose a reason for hiding this comment

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

Changes LGTM, besides the nitpick mentioned above.

@W3stside
Copy link
Contributor Author

image

@W3stside W3stside merged commit f40a576 into release/1.12.0 Mar 28, 2022
@W3stside W3stside deleted the revert-fee-breakdown-line branch March 28, 2022 10:05
@anxolin
Copy link
Contributor

anxolin commented Mar 28, 2022

concerned if this would be perceived as the fee is 5%, 10%, 20%, 40%, which would be a super big fee

@fairlighteth @W3stside what do you think?

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