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

Show bookmarks toolbar automatically if it was disabled after importing bookmarks successfully #4587

Merged
merged 1 commit into from
Oct 6, 2016

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Oct 6, 2016

  • 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).

fix #4507

Auditors: @bbondy

Test Plan:

  1. Make sure bookmakrs toolbar doesn't show
  2. Import bookmarks
  3. bookmarks toolbar will show after successful import

…ng bookmarks successfully

fix brave#4507

Auditors: @bbondy

Test Plan:
1. Make sure bookmakrs toolbar doesn't show
2. Import bookmarks from browser
3. bookmarks toolbar will show after successful import

1. Make sure bookmakrs toolbar doesn't show
2. Import bookmarks from html
3. bookmarks toolbar will show after successful import

1. Make sure bookmakrs toolbar doesn't show
2. Make sure there already exist some bookmarks
3. Import bookmarks from browser
4. bookmarks toolbar should not show after successful import

1. Make sure bookmakrs toolbar doesn't show
2. Make sure there already exist some bookmarks
3. Import bookmarks from html
4. bookmarks toolbar should not show after successful import
@bbondy
Copy link
Member

bbondy commented Oct 6, 2016

++

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.

Show bookmarks toolbar after data import
4 participants