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

Fixed style of some ledger elements #7380

Closed
cezaraugusto opened this issue Feb 25, 2017 · 4 comments
Closed

Fixed style of some ledger elements #7380

cezaraugusto opened this issue Feb 25, 2017 · 4 comments
Assignees
@NejcZdovc
Labels
feature/rewards QA/no-qa-needed refactoring/aphrodite release-notes/include settings
Milestone
0.14.0

Comments

@cezaraugusto
Copy link
Contributor

Addresses:

  • Dry and make about:preferences code easier to maintain
  • Refactor component styles to Aphrodite
  • Make integration tests easier to write
@cezaraugusto cezaraugusto self-assigned this Feb 25, 2017
@cezaraugusto cezaraugusto mentioned this issue Mar 1, 2017
Merged
@mrose17
Copy link
Member

mrose17 commented Mar 7, 2017

@cezaraugusto - what is the status of this issue? should it be closed? thanks!

@cezaraugusto
Copy link
Contributor Author

will be addressed with #7481 along with other Payments-related components

@cezaraugusto cezaraugusto removed their assignment Mar 7, 2017
@cezaraugusto
Copy link
Contributor Author

/cc @NejcZdovc

@NejcZdovc NejcZdovc added this to the 0.13.6 milestone Mar 8, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 8, 2017
@NejcZdovc
Refactor of paymentTab
031c353 
Resolves brave#7501 brave#7380

Auditors: @bsclifton @cezaraugusto

Test plan:
- everything should work the same as was before the chage

Refactor of disabled content
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 8, 2017
@NejcZdovc
Refactor of paymentTab
cd1d842 
Resolves brave#7501 brave#7380 brave#6364

Auditors: @bsclifton @cezaraugusto

Test plan:
- everything should work the same as was before the chage

Refactor of disabled content
@cezaraugusto cezaraugusto mentioned this issue Mar 11, 2017
Merged
4 tasks
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 11, 2017
@NejcZdovc
Refactor of paymentTab
f375451 
Resolves brave#7501 brave#7380 brave#6364

Auditors: @bsclifton @cezaraugusto

Test plan:
- everything should work the same as was before the chage

Refactor of disabled content
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 12, 2017
@NejcZdovc
Refactor of paymentTab
405efe5 
Resolves brave#7501 brave#7380 brave#6364

Auditors: @bsclifton @cezaraugusto

Test plan:
- everything should work the same as was before the chage
cezaraugusto pushed a commit that referenced this issue Mar 12, 2017
@NejcZdovc @cezaraugusto
Refactor of paymentTab (#7481)
0bfceeb 
Resolves #7501 #7380 #6364

Auditors: @bsclifton @cezaraugusto

Test plan:
- everything should work the same as was before the chage
@NejcZdovc
Copy link
Contributor

Closed with #7481

This was referenced Mar 14, 2017
Closed
Closed
@alexwykoff alexwykoff changed the title Split and refactor BitcoinDashboard Fixed style of some ledger elements Mar 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
feature/rewards QA/no-qa-needed refactoring/aphrodite release-notes/include settings
Projects
None yet
Milestone
0.14.0
Development

No branches or pull requests
4 participants
@mrose17 @luixxiul @cezaraugusto @NejcZdovc