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

Vertical alignment is off on Payments "Add Funds" button #10098

Closed
bsclifton opened this issue Jul 24, 2017 · 8 comments
Closed

Vertical alignment is off on Payments "Add Funds" button #10098

bsclifton opened this issue Jul 24, 2017 · 8 comments

Comments

@bsclifton
Copy link
Member

  • Did you search for similar issues before submitting this one?
    yes

  • Describe the issue you encountered:
    0.18.10 beta

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    macOS

  • Brave Version (revision SHA):
    73465bb

  • Steps to reproduce:

    1. Go to Preferences > Payment
    2. Enable payments
    3. Notice button text
  • Actual result:

  • Expected result:

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?

  • Is this an issue in the currently released version?

  • Can this issue be consistently reproduced?

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    screen shot 2017-07-23 at 7 45 34 pm

  • Any related issues:

cc: @luixxiul

@bsclifton bsclifton added this to the 0.18.x (Beta Channel) milestone Jul 24, 2017
@bsclifton
Copy link
Member Author

Does this button properly match the style examples shown on about:styles? I didn't notice any issues with buttons there

@luixxiul
Copy link
Contributor

weird, the issue was fixed on 0.17.19. #9487 CC @cezaraugusto

@bsclifton
Copy link
Member Author

bsclifton commented Jul 24, 2017

good catch, @luixxiul- it was missed in 0.18.x

@cezaraugusto can you please merge into 0.18.x? I get a conflict when cherry-picking. It's already in master and 0.19.x 😄

git cherry-pick -m 1 ddc30fda6fd4596d48f63d154f4dee10014f55d0

@luixxiul
Copy link
Contributor

@bsclifton thanks for digging in 😄

@cezaraugusto
Copy link
Contributor

thanks team, cherry-picked to 0.18.x 67b911b. Closing

@bsclifton
Copy link
Member Author

Thanks, @cezaraugusto! 😄

@NejcZdovc
Copy link
Contributor

@cezaraugusto should we change milestone of #9487 from 0.19 to 0.18?

@bsclifton
Copy link
Member Author

@NejcZdovc good catch! I just moved those now, so that they're in 0.18.x 😄

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