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

Style update + hide approve button for ETH. #2171

Merged
merged 2 commits into from
Jan 17, 2022
Merged

Style update + hide approve button for ETH. #2171

merged 2 commits into from
Jan 17, 2022

Conversation

fairlighteth
Copy link
Contributor

@fairlighteth fairlighteth commented Jan 17, 2022 •
edited
Loading

Summary

  • Minor style updates
  • Hide APPROVE ETH button (even in disabled state) for the ETH approval.

Screen Shot 2022-01-17 at 15 54 43

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

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@fairlighteth fairlighteth merged commit 4b5667b into claim-styleImprove-7 Jan 17, 2022
fairlighteth added a commit that referenced this pull request Jan 17, 2022
@fairlighteth
Wording change and CSS cleanup. (#2170)
2f2f1cb 
* Wording change and CSS cleanup.

* Style update + hide approve button for ETH. (#2171)

* Style update + hide approve button for ETH.

* Style updates + organise invest options (#2172)
@elena-zh
Copy link

LGTM!

@elena-zh
Copy link

@biocom , just a tiny question: are these grey bg colors final?
image

From my perspective, it would be better a bit to make them less gray and a bit more transparent. WDYT?

@fairlighteth
Copy link
Contributor Author

@elena-zh Not final correct.

@alfetopito alfetopito deleted the claim-styleImprove-8 branch January 18, 2022 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@W3stside W3stside W3stside 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 @W3stside