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

Fund Brave wallet field should be read only #14825

Closed
srirambv opened this issue Jul 24, 2018 · 0 comments
Closed

Fund Brave wallet field should be read only #14825

srirambv opened this issue Jul 24, 2018 · 0 comments
Assignees
@Slava @cezaraugusto
Labels
bug feature/ETH-Wallet priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). QA/test-plan-specified release/not-blocking release-notes/exclude reverted
Milestone
ETH Wallet

Comments

@srirambv
Copy link
Collaborator

Description

Fund Brave wallet field should be read only

Steps to Reproduce

  1. Build from Eth-wallet rebased #14734
  2. Enable brave payments and create Brave wallet
  3. Enable Eth-wallet
  4. Click Fund Brave wallet
  5. To filed which is auto-populated with Brave wallet address should be read only. Even though clicking outside reverts back to the correct address the field value is editable

Actual result:
bravewallet-eth

Expected result:
Field should be read only

Reproduces how often:
100%

Brave Version

ac4c649

Reproducible on current live release:
N/A

Additional Information
@srirambv srirambv added bug feature/extensions priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). labels Jul 24, 2018
@cezaraugusto cezaraugusto self-assigned this Jul 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Slava Slava

@cezaraugusto cezaraugusto

Labels
bug feature/ETH-Wallet priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). QA/test-plan-specified release/not-blocking release-notes/exclude reverted
Projects
None yet
Milestone
ETH Wallet
Development

No branches or pull requests
7 participants
@tildelowengrimm @alexwykoff @kjozwiak @Slava @cezaraugusto @bsclifton @srirambv