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

Claim flow styles #2042

Merged
merged 17 commits into from
Jan 3, 2022
Merged

Claim flow styles #2042

merged 17 commits into from
Jan 3, 2022

Conversation

fairlighteth
Copy link
Contributor

Summary

  • Big PR adding the claim flow / styles

Screen Shot 2021-12-24 at 17 24 51

@fairlighteth fairlighteth requested a review from a team December 24, 2021 16:29
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

alfetopito
alfetopito approved these changes Dec 24, 2021
View reviewed changes
Copy link
Contributor

@alfetopito alfetopito left a comment

Choose a reason for hiding this comment

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

Awesome!
Clean merge, no conflicts.
Can't wait to start connecting all the dots in this.

Some stiles are still missing but I'm sure you are aware, or it's Firefox only:

Screenshot from 2021-12-24 13-57-49
Screenshot from 2021-12-24 13-57-31
Screenshot from 2021-12-24 13-55-58
Screenshot from 2021-12-24 13-55-30

@fairlighteth
Copy link
Contributor Author

@alfetopito Yes, as mentioned, styles are missing and still WIP. The main goal was to get the raw content out. Will aim at getting styles up to date ASAP.

@alfetopito
Copy link
Contributor

@alfetopito Yes, as mentioned, styles are missing and still WIP. The main goal was to get the raw content out. Will aim at getting styles up to date ASAP.

Great Michel, I'll go forward and merge as is then

nenadV91 pushed a commit that referenced this pull request Jan 3, 2022
fix: remove x-axis clamping behavior in liquidity chart (#2042)
f86db00 
* remove x-axis clamping behavior

* remove extra curly braces around errorMessage
nenadV91 pushed a commit that referenced this pull request Jan 3, 2022
fix: revert remove x-axis clamping behavior in liquidity chart (#2045)
d5ef7f9 
* Revert "fix: remove x-axis clamping behavior in liquidity chart (#2042)"

This reverts commit f86db00.

* removed extra curly braces around errorMessage
@alfetopito alfetopito merged commit 956c342 into claim Jan 3, 2022
@alfetopito alfetopito mentioned this pull request Jan 25, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alfetopito alfetopito alfetopito approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fairlighteth @alfetopito