Proposal Details - Githubissues

protofire / omen-exchange

Omen exchange

https://omen.eth.link/

GNU Affero General Public License v3.0

66 stars 77 forks source link

Proposal Details #2208

Closed Mi-Lan closed 3 years ago

Mi-Lan commented 3 years ago

closes #2190 How to test: Because data is not connected we can only test styles so we need to make sure that designs are same as on figma:

To access page go to http://localhost:3000/#/proposal-details-page
Liquidity rewards section(mobile and desktop) https://www.figma.com/file/zVniQrYgkj8AfQs5zNeBrS/Components-FINAL?node-id=300%3A1290
Whole page(mobile and desktop https://www.figma.com/file/PhYs1uw1LgqpvriqHDPl7k/Playground-FINAL?node-id=402%3A12460
[ ] Mobile
[ ] Desktop

pimato commented 3 years ago

Also remove the box shadhow here:

Bildschirmfoto 2021-08-25 um 12 37 47

and here:

Bildschirmfoto 2021-08-25 um 12 38 01

pimato commented 3 years ago

Bildschirmfoto 2021-08-25 um 12 39 24

should be primary1

pimato commented 3 years ago

the padding at the top in the tooltip should be removed:

Bildschirmfoto 2021-08-25 um 12 41 39

Bildschirmfoto 2021-08-25 um 12 41 37

pimato commented 3 years ago

can we add hover and onClick styling for the navigation back: ArrowButton

Figma: Design Specification

pimato commented 3 years ago

looks like only the arrow has the hand icon on Hover:

https://user-images.githubusercontent.com/5337809/130804301-258c08bb-3b0f-4d5e-813f-d39f3760800b.mov

pimato commented 3 years ago

can we get the description back Current Prediction, like we have it on the market details view:

Bildschirmfoto 2021-08-26 um 11 45 37