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

Commit

Permalink
Make button styles consistent (part 3)
Browse files Browse the repository at this point in the history
- Paddings of buttons on about:preferences were increased
- Removed unnecessary class

Auditors:

Test Plan:
  • Loading branch information
Suguru Hirahara committed Dec 2, 2016
1 parent e8955ce commit b52eb32
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 6 deletions.
2 changes: 1 addition & 1 deletion js/about/preferences.js
Original file line number Diff line number Diff line change
Expand Up @@ -927,7 +927,7 @@ class PaymentsTab extends ImmutableComponent {
const onButtonClick = this.props.ledgerData.get('created')
? this.props.showOverlay.bind(this, 'addFunds')
: (this.props.ledgerData.get('creating') ? () => {} : this.createWallet)
return <Button l10nId={buttonText} className='primaryButton addFunds' onClick={onButtonClick.bind(this)} disabled={this.props.ledgerData.get('creating')} />
return <Button l10nId={buttonText} className='primaryButton wideButton' onClick={onButtonClick.bind(this)} disabled={this.props.ledgerData.get('creating')} />
}

get paymentHistoryButton () {
Expand Down
4 changes: 1 addition & 3 deletions less/about/preferences.less
Original file line number Diff line number Diff line change
Expand Up @@ -655,9 +655,7 @@ div.nextPaymentSubmission {
width: auto;
min-width: 235px;
}
span.browserButton.primaryButton.addFunds {
margin-right: 0;
}

.fundsAmount {
display: inline;
margin: 0;
Expand Down
12 changes: 10 additions & 2 deletions less/button.less
Original file line number Diff line number Diff line change
Expand Up @@ -158,9 +158,10 @@ span.buttonSeparator {
// for about:preferences
.prefBody {
.settingsList > .settingItem + button,
.settingItem > span + button {
.settingItem > span + button,
#paymentsContainer button:not(.close) {
font-size: 0.9em;
padding: 5px 20px;
padding: 8px 20px;
}

.settingsList > .settingItem + button {
Expand All @@ -170,4 +171,11 @@ span.buttonSeparator {
.settingItem > span + button {
margin-top: 5px;
}

#paymentsContainer {
.browserButton + .browserButton {
margin-left: 8px;
}
}

}

0 comments on commit b52eb32

Please sign in to comment.