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

Misc styling #2228

Merged
merged 4 commits into from
Jan 20, 2022
Merged

Misc styling #2228

merged 4 commits into from
Jan 20, 2022

Conversation

fairlighteth
Copy link
Contributor

@fairlighteth fairlighteth commented Jan 20, 2022 •
edited
Loading

Summary

  • Change check icon SVG colors
  • Hide token symbol component for Airdrops: Only show the vCOW token.
  • Replace 'eligible' banner check icon, with the other check icons we're using.
  • Range step, remove % elements
  • Misc styling

@fairlighteth fairlighteth requested review from a team January 20, 2022 09:14
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@elena-zh
Copy link

Hey @biocom , great changes!
Some issues I faced:

  1. Percentage exceeds the form width
    image
  2. When an investment amount is too low, the bar seems to be filled outside its area
    image
  3. I've got a 'broken' icon image on 'eligible' banner when I went back from the investment screen
    image
  4. Maybe make these icons a bit bigger? The image on the vCOW icon is hardly visible:
    image

@fairlighteth
Copy link
Contributor Author

@elena-zh Thanks for reviewing. Will address these points now in a separate PR.

elena-zh
elena-zh approved these changes Jan 20, 2022
View reviewed changes
Copy link

@elena-zh elena-zh left a comment

Choose a reason for hiding this comment

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

Approved then!

@fairlighteth fairlighteth merged commit 5aab634 into claim Jan 20, 2022
maria-vslvn pushed a commit that referenced this pull request Jan 25, 2022
@circle-hotaru
fix: older browser suppport (#2228)
5dfe44d 
* fix: Object.fromEntries polyfill

* fix: update polyfills

* fix: update polyfills
@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

@anxolin anxolin anxolin approved these changes

@elena-zh elena-zh elena-zh 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.
3 participants
@fairlighteth @elena-zh @anxolin