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

Ledger is shown as enabled for a split second #11387

Closed
NejcZdovc opened this issue Oct 9, 2017 · 0 comments
Closed

Ledger is shown as enabled for a split second #11387

NejcZdovc opened this issue Oct 9, 2017 · 0 comments

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Oct 9, 2017

Description

When you are enabling brave payment and you don't have existing wallet, we display add founds button orange for a split second, before transforming it into disabled state until wallet is creating.

Steps to Reproduce

  1. clean profile
  2. enable brave payments

Actual result:

button is orange for a split second

Expected result:
button should be greyed out all the time until the wallet is created

Reproduces how often: [What percentage of the time does it reproduce?]

Brave Version

about:brave info:

Reproducible on current live release:

Additional Information

@NejcZdovc NejcZdovc self-assigned this Oct 9, 2017
@NejcZdovc NejcZdovc added this to the 0.19.x (Beta Channel) milestone Oct 9, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 9, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 9, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Oct 9, 2017
Resolves brave#11387
Resolves brave#4066

Auditors:

Test Plan:
cezaraugusto added a commit that referenced this issue Oct 9, 2017
Fixes color flash for add funds button
cezaraugusto added a commit that referenced this issue Oct 9, 2017
Fixes color flash for add funds button
cezaraugusto added a commit that referenced this issue Oct 9, 2017
Fixes color flash for add funds button
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Resolves brave#11387
Resolves brave#4066

Auditors:

Test Plan:
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.