Browser-laptop: Vertical alignment is off on Payments "Add Funds" button

Created on 24 Jul 2017  路  8Comments  路  Source: brave/browser-laptop

  • 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):
    https://github.com/brave/browser-laptop/commit/73465bb6af2289ede944b9bd6349cdef65859efa

  • 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

Qchecked-Linux Qchecked-Win32 Qchecked-Win64 Qchecked-macOS Qtest-plan-specified misbutton regression release-noteexclude

All 8 comments

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

weird, the issue was fixed on 0.17.19. https://github.com/brave/browser-laptop/pull/9487 CC @cezaraugusto

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

@bsclifton thanks for digging in :smile:

Thanks, @cezaraugusto! :smile:

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

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

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jkup picture jkup  路  3Comments

lukemulks picture lukemulks  路  3Comments

shortstuffsushi picture shortstuffsushi  路  3Comments

mykkymk picture mykkymk  路  3Comments

stevespringett picture stevespringett  路  3Comments