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

Removed min-width for buttons on the footer of payment history dialog #6686

Merged
merged 1 commit into from
Jan 28, 2017
Merged

Removed min-width for buttons on the footer of payment history dialog #6686

merged 1 commit into from
Jan 28, 2017

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Jan 16, 2017

The min-width for the buttons were specified with #6384 globally, so it is no longer necessary

Closes #6685

Auditors:

Test Plan:
#6685 (comment)

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Test Plan:

The min-width for the buttons were specified with #6384 globally, so it is no longer necessary

Closes #6685

Auditors:

Test Plan:
1. Disable the ledger
2. run `npm run add-simulated-payment-history` some times
3. Enable the ledger
4. Click "View Payment History..."
5. Make sure the min-width of "OK" button is set to 78px
@luixxiul luixxiul added this to the 0.13.1 milestone Jan 16, 2017
@luixxiul
Copy link
Contributor Author

I added this to 0.13.1 as this is a minor fix

@bsclifton bsclifton changed the title Revoved min-width for buttons on the footer of payment history dialog Removed min-width for buttons on the footer of payment history dialog Jan 16, 2017
@luixxiul luixxiul changed the base branch from master to 0.13.1-branch January 19, 2017 03:52
@bsclifton bsclifton force-pushed the 0.13.1-branch branch 6 times, most recently from 67b2326 to 5eb09be Compare January 26, 2017 20:09
@bsclifton bsclifton closed this Jan 26, 2017
@bsclifton bsclifton changed the base branch from 0.13.1-branch to master January 26, 2017 21:17
@bsclifton bsclifton reopened this Jan 26, 2017
@bbondy
Copy link
Member

bbondy commented Jan 28, 2017

++

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

Successfully merging this pull request may close these issues.

3 participants