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

TRCL-3281 : Add transfer entry point to profile page pt 2 - UI Tweaks #32

Conversation

mike-dydx
Copy link
Contributor

@mike-dydx mike-dydx commented Nov 30, 2023

Links (dYdX Internal Use Only)

Linear Ticket: TRCL-3281 : Add transfer entry point to profile page pt 2 - UI Tweaks

Figma Design: (https://www.figma.com/file/fMQodZKzn9B5aZTN5G0fLJ/dYdX-%E2%80%BA-Desktop?type=design&node-id=20085%3A7145&mode=design&t=NFDL4j2GWUVuLDWT-1)


Description / Intuition

  1. Make ‘Copy’ and ‘Block explorer’ buttons 1 shade lighter
  2. Make ‘X’ icon smaller to match designs
  3. Increase padding for ‘Manage wallets’ to 32 px
  4. Update buttons from pills to rectangles
  5. Round the sheet indicator
  6. ‘Disable’ the text, in addition to the buttons
  7. Ensure light mode color mappings are correct.

Before/After Screenshots or Videos

Before After
Screenshot 2023-12-01 at 11 25 27 AM Screenshot 2023-12-01 at 11 25 28 AM
Screenshot 2023-12-01 at 11 25 51 AM Screenshot 2023-12-01 at 11 25 53 AM

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring or Technical Debt
  • Documentation update
  • Other (please describe: )

Copy link

linear bot commented Nov 30, 2023

TRCL-3281 Add transfer entry point to profile page pt 2 - UI Tweaks

the original designs have been adjusted slightly

from Design:

created a section with feedback

  1. Make ‘Copy’ and ‘Block explorer’ buttons 1 shade lighter
  2. Make ‘X’ icon smaller to match designs
  3. Increase padding for ‘Manage wallets’ to 32 px
  4. Update buttons from pills to rectangles
  5. Round the sheet indicator
  6. ‘Disable’ the text, in addition to the buttons
  7. Ensure light mode color mappings are correct.

@mike-dydx mike-dydx requested a review from ruixhuang December 1, 2023 16:33
let dragIndicator = Rectangle()
.themeColor(background: .layer1)
.frame(width: 36, height: 4)
.clipShape(.capsule)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

only effective change here was rounding colors per Dan's feedback

@mike-dydx mike-dydx force-pushed the mike/trcl-3281-add-transfer-entry-point-to-profile-page-pt-2-ui-tweaks branch 4 times, most recently from 7c7ee3e to c0440d5 Compare December 1, 2023 16:41
undo podfile lock changes
@mike-dydx mike-dydx force-pushed the mike/trcl-3281-add-transfer-entry-point-to-profile-page-pt-2-ui-tweaks branch from c0440d5 to 798c3c5 Compare December 1, 2023 16:42
@@ -9,7 +9,7 @@
"layer_1": "#EDF0F2",
"layer_2": "#F9FAFB",
"layer_3": "#EDF0F2",
"layer_4": "#E0E6EB",
"layer_4": "#FFFFFF",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this color had been incorrect, see figma

@mike-dydx mike-dydx marked this pull request as ready for review December 1, 2023 18:36
@mike-dydx mike-dydx merged commit f37ef38 into main Dec 1, 2023
2 checks passed
@mike-dydx mike-dydx deleted the mike/trcl-3281-add-transfer-entry-point-to-profile-page-pt-2-ui-tweaks branch December 1, 2023 18:37
@mike-dydx mike-dydx self-assigned this Dec 1, 2023
mike-dydx added a commit that referenced this pull request Aug 20, 2024
…#32)

* apply design feedback

* update more colors to match design

* enable profile buttons always

* clean up

undo podfile lock changes

* fix compile issue
mike-dydx added a commit that referenced this pull request Aug 21, 2024
…#32)

* apply design feedback

* update more colors to match design

* enable profile buttons always

* clean up

undo podfile lock changes

* fix compile issue
mike-dydx added a commit that referenced this pull request Aug 21, 2024
…#32)

* apply design feedback

* update more colors to match design

* enable profile buttons always

* clean up

undo podfile lock changes

* fix compile issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants