Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve header 6 #602

Merged
merged 3 commits into from
Jun 2, 2022
Merged

Improve header 6 #602

merged 3 commits into from
Jun 2, 2022

Conversation

fairlighteth
Copy link
Contributor

Summary

@fairlighteth fairlighteth requested review from a team May 30, 2022 15:51
@github-actions
Copy link
Contributor

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@@ -101,7 +101,7 @@ export const RAW_CODE_LINK = 'https://raw.githubusercontent.com/' + GITHUB_REPOS
export const DOCS_LINK = 'https://docs.cow.fi'
export const CONTRACTS_CODE_LINK = 'https://github.com/cowprotocol/contracts'
export const DISCORD_LINK = 'https://discord.com/invite/cowprotocol'
export const DUNE_DASHBOARD_LINK = 'https://duneanalytics.com/gnosis.protocol/Gnosis-Protocol-V2'
export const DUNE_DASHBOARD_LINK = 'https://dune.com/gnosis.protocol/Gnosis-Protocol-V2'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we are updating the link, better use the one that points to COW Protocol rather than gp?

I got the general https://dune.com/gnosis.protocol/CowSwap, but even this is still under gnosis.protocol.

Better update it another time then

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will review this URL to make sure it's the right one.

src/custom/components/MenuDropdown/index.tsx Outdated Show resolved Hide resolved
* Further flyout menu styling

* Uniform menu items styling

* Move styled components to styled

* Add hamburger icon with animation component.

* Improve header 8 (#600)

* Add mediaQuery hook

* Improve header 9 (#599)

* Progress mobile responsive menu

* Progress mobile responsive menu

* Improve header 10 (#598)

* Move to helper function

* Optimise menu code

* Improve header 11 (#597)

* Move URLS to an ENUM

* Fix enum issue.

* Styling and route fixes.

* Styling.

* Styling.

* Header/Menu improvement (waterfall PR FINAL)  (#596)

* Styling.

* Styling footer and bridge banner.

* Improve header 13 (#610)

* Fix network selector position.

* Fix exports/imports for network selector.

* Only open ordersPanel if account is true.
@fairlighteth fairlighteth merged commit 39eeb6e into improve-header-5 Jun 2, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jun 2, 2022
@alfetopito alfetopito deleted the improve-header-6 branch June 3, 2022 07:37
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.

3 participants