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

Payments page is missing spacing between buttons #12776

Closed
LaurenWags opened this issue Jan 22, 2018 · 1 comment
Closed

Payments page is missing spacing between buttons #12776

LaurenWags opened this issue Jan 22, 2018 · 1 comment
Assignees
@cezaraugusto
Labels
0.20.x issue first seen in 0.20.x bug QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified regression release-notes/exclude
Milestone
0.20.x (Release C...

Comments

@LaurenWags
Copy link
Member

Description

'Claim my free tokens' button for UGP promotion and 'Add Funds...' button should have space between them.

Steps to Reproduce

  1. Have active UGP promo (prod) or use staging to create a wallet
  2. View Payments page

Actual result:
Button spacing is incorrect:

screen shot 2018-01-22 at 2 37 30 pm

Expected result:
Button spacing should be as it is in 0.19.139:

screen shot 2018-01-22 at 2 38 08 pm

Reproduces how often:
Easily

Brave Version

about:brave info:
Brave | 0.20.17
V8 | 6.4.388.29
rev | 5fa67ff
Muon | 4.7.3
OS Release | 16.7.0
Update Channel | Beta
OS Architecture | x64
OS Platform | macOS
Node.js | 7.9.0
Brave Sync | v1.4.2
libchromiumcontent | 64.0.3282.99

Reproducible on current live release:
no

Additional Information
@LaurenWags LaurenWags added bug 0.20.x issue first seen in 0.20.x labels Jan 22, 2018
@LaurenWags LaurenWags added this to the 0.20.x (Beta Channel) milestone Jan 22, 2018
@cezaraugusto cezaraugusto self-assigned this Jan 22, 2018
cezaraugusto added a commit that referenced this issue Jan 22, 2018
@cezaraugusto
fix glued buttons on ledger panel
bb179c9 
-
this should be only for 0.21.x and 0.20.x
-
this separated claim button from add funds button a bit
fix  #12776
@cezaraugusto cezaraugusto mentioned this issue Jan 22, 2018
Merged
@bsclifton
Copy link
Member

Fixed with #12779

@srirambv srirambv mentioned this issue Jan 24, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cezaraugusto cezaraugusto

Labels
0.20.x issue first seen in 0.20.x bug QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified regression release-notes/exclude
Projects
None yet
Milestone
0.20.x (Release Channel)
Development

No branches or pull requests
5 participants
@kjozwiak @cezaraugusto @bsclifton @srirambv @LaurenWags