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

Fix default engine display problem for fresh profile #3275

Merged
merged 2 commits into from
Aug 20, 2016
Merged

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Aug 20, 2016

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Ran git rebase -i to squash commits if needed.

Fix #3254

auditor: @bbondy

@@ -365,11 +365,12 @@ class GeneralTab extends ImmutableComponent {

class SearchSelectEntry extends ImmutableComponent {
shouldComponentUpdate (nextProps, nextState) {
return this.props.settings.get(settings.DEFAULT_SEARCH_ENGINE) !== nextProps.settings.get(settings.DEFAULT_SEARCH_ENGINE)
return getSetting(settings.DEFAULT_SEARCH_ENGINE, this.props.settings) !==
getSetting(settings.DEFAULT_SEARCH_ENGINE, nextProps.settings)
Copy link
Member

Choose a reason for hiding this comment

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

I commented on the commit on your branch by mistake instead of here:
darkdh@f82a848

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for the feedback. Fixed in 1477399

@bbondy bbondy merged commit 90f0676 into brave:master Aug 20, 2016
@luixxiul luixxiul added this to the 0.11.6dev milestone Aug 20, 2016
@darkdh darkdh deleted the 3254 branch August 21, 2016 08:59
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.

4 participants