gnosis / gnosis/safe-cmm-app-react

Bug: It would be nice to change a tooltip text for the Highest price

Open
#103 0 comments 0 reactions 0 assignees View on GitHub
bug low priority
Dominant language
TypeScript
Stars
2
Forks
1
PR merge metrics
No merged PRs in 30d

Description

# Description
I'm not 100% sure, but it would be nice to change a tooltip text for the Highest price to correspond to the text in the tooltip for the Lowest price.

So, there are 2 tooltips for the Highest price in the mock-ups:

1. https://projects.invisionapp.com/share/VXXWLEE7534#/screens/428757594
2. https://projects.invisionapp.com/share/VXXWLEE7534#/screens/428757599

As the tooltip for the 'Lowest price' says 'The lowest price our strategy covers, lower than this you hold 100% tokens', it would be nice to use the text on the 1st mockup for the Highest price tooltip instead of the implemented one (the 2nd one)

# Screenshots/Videos
![incorrect tooltip highest](https://user-images.githubusercontent.com/70885163/95430215-20600a80-0954-11eb-96fc-8c8d0f642265.jpg)

# Environment
**Desktop:** *(Please complete)*
- Version - 188558105 - 6f49a59fe3d23f21996b6d1e1548b76511cf9b2

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the Highest price tooltip implementation in safe-cmm-app-react and compare it with the Lowest price tooltip and the two linked mockups. Update the wording to match the intended text shown in the first mockup, then verify that the rendered tooltip displays the corrected message.

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
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.