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

Add governance card banner. #2566

Merged
merged 2 commits into from
Mar 21, 2022
Merged

Add governance card banner. #2566

merged 2 commits into from
Mar 21, 2022

Conversation

fairlighteth
Copy link
Contributor

Summary

Part of a waterfall PR. Adds the governance styled card banner:
Screen Shot 2022-03-18 at 17 39 56
Screen Shot 2022-03-18 at 17 41 20

@fairlighteth fairlighteth requested review from a team March 18, 2022 17:42
@fairlighteth
Copy link
Contributor Author

Issue to handle: Light mode, on hover of text links, the text is black (cc @elena-zh )

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@elena-zh
Copy link

elena-zh commented Mar 21, 2022

Hey @fairlighteth , great!

Could we add a 'hovered' effect to the banner as we do on the Claim page? https://watch.screencastify.com/v/fyqv1ml5vsX7EiYQAHb6

Also, I noticed that in this PR tooltip icons on the affiliate form have became changed in the light mode.
image.png

Hovered links in the light mode you have already mentioned above=)

Thanks!

@fairlighteth
Copy link
Contributor Author

@elena-zh Thanks for reviewing.

Could we add a 'hovered' effect to the banner as we do on the Claim page?

In this case we have 2 distinct click actions. The difference with the banner on the claim page is that the whole banner is a 'link'. I would therefore not add this hover effect on the whole governance banner.

Also, I noticed that in this PR tooltip icons on the affiliate form have became changed in the light mode.

Yes, this todo point was mentioned in the base PR and will be addressed there.

@fairlighteth
Copy link
Contributor Author

Updated the text hover styles.

@fairlighteth fairlighteth merged commit 340a812 into convert-vcow Mar 21, 2022
@alfetopito alfetopito deleted the convert-vcow-2 branch March 21, 2022 22:55
@anxolin
Copy link
Contributor

anxolin commented Mar 22, 2022

Great! it looks amazing

fairlighteth added a commit that referenced this pull request Mar 22, 2022
* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color
fairlighteth added a commit that referenced this pull request Mar 22, 2022
* Profile page: Add vCOW swap feature cards (#2564)

* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color

* Styling of vesting breakdown tooltip.
fairlighteth added a commit that referenced this pull request Mar 23, 2022
* Profile page: Add vCOW swap feature cards (#2564)

* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color

* Styled governance banner and inner card container.

* Fix code style issues with Prettier

* Style update (trigger build)

Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
nenadV91 added a commit that referenced this pull request Mar 23, 2022
* Profile page: Add vCOW swap feature cards (#2564)

* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color

* Connect UI and vCow data

* Added PR changes

* Remove some previous change

* Added swapAll abi (#2576)

Co-authored-by: fairlight <31534717+fairlighteth@users.noreply.github.com>
nenadV91 pushed a commit that referenced this pull request Mar 30, 2022
* Update FAQ to reflect Token Launch

This PR pushes a new content section from lines 559 - 627 that addresses questions related to the token launch. I have tried format everything as good as I possibly know/can, apologies if it needs to be reworked. 

Questions are: 
562 - Does CowSwap Have a token? 
580 - Is there a proposal to make vCOW transferrable?
585 - what is the purspose of the COW tokens?
605 - what was the criteria for the token airdrop?

* Fix code style issues with Prettier

* Fix building error

* Add space

* Profile page: Add vCOW swap feature cards (#2564)

* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color

* Moving custom components to mod file to avoid circular dependencies/lazy loading issues (#2580)

Co-authored-by: Leandro <leandro.boscariol@gnosis.io>

* Remove isClaiming flag (#2586)

* Remove lint warn (#2587)

* Update index.tsx

* Fix code style issues with Prettier

* Split and refactor FAQ

* Update FAQ to reflect Token Launch (#2540)

* Update FAQ to reflect Token Launch

This PR pushes a new content section from lines 559 - 627 that addresses questions related to the token launch. I have tried format everything as good as I possibly know/can, apologies if it needs to be reworked. 

Questions are: 
562 - Does CowSwap Have a token? 
580 - Is there a proposal to make vCOW transferrable?
585 - what is the purspose of the COW tokens?
605 - what was the criteria for the token airdrop?

* Fix code style issues with Prettier

* Fix building error

* Add space

* Update index.tsx

* Fix code style issues with Prettier

* Add punctuantions

Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Co-authored-by: Anxo Rodriguez <anxolin@gmail.com>

* Fix links

* Add footer

* Add footer to index too

* Update metrics

* Fix internal links

* Hide back in home

* Link to FAQ

* Correct year

* Fix typo. Fixes #2067

* Re-style FAQ menu.

* Title styles.

* Cleanup console.log

* Fix merge and style back to top + misc.

* Text updates.

* Fix code style issues with Prettier

* Remove click events.

* Fix lint issue.

* Remove is claim enabled

Co-authored-by: Alex V <80408352+avsavsavs@users.noreply.github.com>
Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Co-authored-by: Anxo Rodriguez <anxolin@gmail.com>
Co-authored-by: Leandro Boscariol <alfetopito@users.noreply.github.com>
Co-authored-by: Leandro <leandro.boscariol@gnosis.io>
Co-authored-by: w3stside <david.sato64@gmail.com>
nenadV91 added a commit that referenced this pull request Mar 30, 2022
* Split refactor faq (#2592)

* Update FAQ to reflect Token Launch

This PR pushes a new content section from lines 559 - 627 that addresses questions related to the token launch. I have tried format everything as good as I possibly know/can, apologies if it needs to be reworked.

Questions are:
562 - Does CowSwap Have a token?
580 - Is there a proposal to make vCOW transferrable?
585 - what is the purspose of the COW tokens?
605 - what was the criteria for the token airdrop?

* Fix code style issues with Prettier

* Fix building error

* Add space

* Update index.tsx

* Fix code style issues with Prettier

* Split and refactor FAQ

* Fix links

* Add footer

* Add footer to index too

* Update metrics

* Fix internal links

* Hide back in home

* Link to FAQ

* Correct year

* Fix typo. Fixes #2067

Co-authored-by: Alex V <80408352+avsavsavs@users.noreply.github.com>
Co-authored-by: Lint Action <lint-action@samuelmeuli.com>

* Re-style FAQ menu. (#2611)

* Update FAQ to reflect Token Launch

This PR pushes a new content section from lines 559 - 627 that addresses questions related to the token launch. I have tried format everything as good as I possibly know/can, apologies if it needs to be reworked. 

Questions are: 
562 - Does CowSwap Have a token? 
580 - Is there a proposal to make vCOW transferrable?
585 - what is the purspose of the COW tokens?
605 - what was the criteria for the token airdrop?

* Fix code style issues with Prettier

* Fix building error

* Add space

* Profile page: Add vCOW swap feature cards (#2564)

* Style vCOW swap cards.

* Style vCOW swap cards. (#2565)

* Add governance card banner. (#2566)

* Add governance card banner.

* Governance banner update styles.

* Make question icon use dynamic dark/light mode colors.

* Center content properly cross devices.

* Dummy values.

* Disable unused vars

* Cleanup obsolete color

* Moving custom components to mod file to avoid circular dependencies/lazy loading issues (#2580)

Co-authored-by: Leandro <leandro.boscariol@gnosis.io>

* Remove isClaiming flag (#2586)

* Remove lint warn (#2587)

* Update index.tsx

* Fix code style issues with Prettier

* Split and refactor FAQ

* Update FAQ to reflect Token Launch (#2540)

* Update FAQ to reflect Token Launch

This PR pushes a new content section from lines 559 - 627 that addresses questions related to the token launch. I have tried format everything as good as I possibly know/can, apologies if it needs to be reworked. 

Questions are: 
562 - Does CowSwap Have a token? 
580 - Is there a proposal to make vCOW transferrable?
585 - what is the purspose of the COW tokens?
605 - what was the criteria for the token airdrop?

* Fix code style issues with Prettier

* Fix building error

* Add space

* Update index.tsx

* Fix code style issues with Prettier

* Add punctuantions

Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Co-authored-by: Anxo Rodriguez <anxolin@gmail.com>

* Fix links

* Add footer

* Add footer to index too

* Update metrics

* Fix internal links

* Hide back in home

* Link to FAQ

* Correct year

* Fix typo. Fixes #2067

* Re-style FAQ menu.

* Title styles.

* Cleanup console.log

* Fix merge and style back to top + misc.

* Text updates.

* Fix code style issues with Prettier

* Remove click events.

* Fix lint issue.

* Remove is claim enabled

Co-authored-by: Alex V <80408352+avsavsavs@users.noreply.github.com>
Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Co-authored-by: Anxo Rodriguez <anxolin@gmail.com>
Co-authored-by: Leandro Boscariol <alfetopito@users.noreply.github.com>
Co-authored-by: Leandro <leandro.boscariol@gnosis.io>
Co-authored-by: w3stside <david.sato64@gmail.com>

Co-authored-by: Anxo Rodriguez <anxolin@gmail.com>
Co-authored-by: Alex V <80408352+avsavsavs@users.noreply.github.com>
Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Co-authored-by: fairlight <31534717+fairlighteth@users.noreply.github.com>
Co-authored-by: Leandro Boscariol <alfetopito@users.noreply.github.com>
Co-authored-by: Leandro <leandro.boscariol@gnosis.io>
Co-authored-by: w3stside <david.sato64@gmail.com>
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.

4 participants