gnosis / cowswap

🐮 CowSwap: First Gnosis Protocol v2 UI
https://gnosis.io
GNU General Public License v3.0
113 stars 55 forks source link

[MEV Slicer] Icon for the game is incorrect on the Trade confirmation modal #1764

Closed elena-zh closed 2 years ago

elena-zh commented 3 years ago

Place an order Verify the icon on the 'Play Cow Slicer game'

AR: icon for the Cow Runner change icon.jpg

ER: icon should be for the MEV Slicer game image.png

alfetopito commented 3 years ago

I would suggest to update this with either:

  1. Play a game!
    1. leads to a page with both games OR
    2. randomly opens one game

OR

  1. Add a button for each game

Play the Cow Slicer Gamer Play the Cow Runner Gamer

elena-zh commented 2 years ago

According to the discussion in https://altoros.slack.com/archives/CSGLUC1BN/p1637180786130400 , it would be nice to return Cow Runner Game to the modal

Also, should we make the CowSwap Runner Game the default again after trade execution? It keeps you on the CowSwap site, whereas the slicer opens a new tap.

In addition, my instinct is that it makes more sense to users who don't know the full Cowswap/MEV narrative. One user was pretty confused by the slicer game this week. (It starts off with a message like "the cow doesn't run from sandwiches anymore" which doesn't make sense if you haven't played the runner game.)

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you think it shouldn't be closed, speak now or forever hold your peace.

elena-zh commented 2 years ago

I still think it would be great to fix this issue.

anxolin commented 2 years ago

cc/ @alfetopito

alfetopito commented 2 years ago

Indeed, we just got a request to have the Cow runner game be default again.

@alongoni can you add this to your queue?

Not high priority, no need to stop anything in progress, just a small task to squeeze in when you have the chance.