Skip to content

Commit

Permalink
Disable autofill management button when autofill is disabled
Browse files Browse the repository at this point in the history
fix brave#5146

Auditors: @bsclifton

Test Plan:
1. Go to about:preferences#security
2. Toggle "Enable Autofill" off
3. Manage Autofill Data button should be disabled
  • Loading branch information
darkdh committed Oct 31, 2016
1 parent 170a14e commit e10a698
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/about/preferences.js
Original file line number Diff line number Diff line change
Expand Up @@ -1528,7 +1528,7 @@ class SecurityTab extends ImmutableComponent {
<Button l10nId='manageAutofillData' className='primaryButton manageAutofillDataButton'
onClick={aboutActions.newFrame.bind(null, {
location: 'about:autofill'
}, true)} />
}, true)} disabled={!getSetting(settings.AUTOFILL_ENABLED, this.props.settings)} />
</SettingsList>
<div className='sectionTitle' data-l10n-id='doNotTrackTitle' />
<SettingsList>
Expand Down

1 comment on commit e10a698

@bsclifton
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++ 😄

Please sign in to comment.