Remove jargon from interface
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 5.5k
- Forks
- 5.4k
- PR merge metrics
- No merged PRs in 30d
Description
Bug Description
The summary modal box when swapping assets reads "Price Impact", and its tooltip (when hovered over) reads "The impact your trade has on the market price of this pool." No, my trade will not have an "impact" on the market price; i.e., my trade is not crashing into market price. Even in the jargon sense of "strong/violent/marked effect", having it say that makes no sense if the magnitude is anything less than, say, 10%, since it would be an oxymoron. In my screenshotted example below, that is indeed the case: a 0.00% "impact" on the price.
Steps to Reproduce
- Go to the Uniswap App.
- Choose a from asset and an amount.
- Choose a to asset and an amount.
- Click Swap.
Expected Behaviour
The word should be effect; the trade might have an effect on the price, and the number listed there will describe the magnitude of this effect. Consequently, the label should read "Price Effect" instead. I'm not sure where this jargon is coming from or why it's becoming prevalent these days.
Additional Context


Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in the swap summary modal by searching the interface for the displayed "Price Impact" label and its tooltip text. Update the user-facing wording to "Price Effect" and verify the swap flow shows the revised label and explanation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100