Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Payment History popover probably needs a little more attention #3485

Closed
mrose17 opened this issue Aug 27, 2016 · 2 comments · Fixed by #6047
Closed

Payment History popover probably needs a little more attention #3485

mrose17 opened this issue Aug 27, 2016 · 2 comments · Fixed by #6047
Assignees
@luixxiul @bradleyrichter
Labels
feature/rewards polish Nice to have — usually related to front-end/visual tasks. QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.13.0

Comments

@mrose17
Copy link
Member

mrose17 commented Aug 27, 2016

it seems sort of "drab" in comparison to the rest of the graphic treatments...

3
@mrose17 mrose17 added polish Nice to have — usually related to front-end/visual tasks. feature/rewards labels Aug 27, 2016
@bbondy
Copy link
Member

bbondy commented Aug 28, 2016

@bradleyrichter it looks like there's missing padding on the bottom of that popup

@mrose17 mrose17 added this to the 1.0.0 milestone Sep 7, 2016
This was referenced Dec 10, 2016
Closed
Merged
@luixxiul luixxiul self-assigned this Dec 15, 2016
@luixxiul luixxiul modified the milestones: 0.13.0, 1.0.0 Dec 15, 2016
This was referenced Dec 20, 2016
Closed
Closed
Closed
Closed
@alexwykoff alexwykoff mentioned this issue Dec 20, 2016
Closed
@luixxiul
Copy link
Contributor

Test Plan:

  1. Disable the ledger
  2. run npm run add-simulated-payment-history some times
  3. Enable the ledger
  4. Click "View Payment History..."
  5. Make sure the dialog like this appears:

screenshot 2016-12-15 3 36 37

This was referenced Jan 16, 2017
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@luixxiul luixxiul

@bradleyrichter bradleyrichter

Labels
feature/rewards polish Nice to have — usually related to front-end/visual tasks. QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.13.0
Development

Successfully merging a pull request may close this issue.

Refactor about:preferences#payments followups
5 participants
@mrose17 @bbondy @luixxiul @bradleyrichter @srirambv