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

Commit

Permalink
Open all bookmark items under a bookmark folder in foreground tabs
Browse files Browse the repository at this point in the history
If SWITCH_TO_NEW_TABS is truthy

Auditors: Unsure what to fill in here

Test Plan:
  - Simulate middle clicking a bookmark folder
  - Check that as many new foreground tabs are opened with the same URLs of the
  bookmarks in order (or just check that all such events are fired in order if
  URL checks are not feasible)

Fix #3068.
  • Loading branch information
GreenRecycleBin committed Nov 24, 2016
1 parent 3658897 commit 85e51a9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/actions/bookmarkActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ const bookmarkActions = {
// Only load the first 25 tabs as loaded
bookmarks
.forEach((bookmark, i) =>
windowActions.newFrame(Object.assign(siteUtil.toFrameOpts(bookmark), {unloaded: i > 25}), false))
windowActions.newFrame(Object.assign(siteUtil.toFrameOpts(bookmark), {unloaded: i > 25}), getSetting(SWITCH_TO_NEW_TABS)))
},

/**
Expand Down

0 comments on commit 85e51a9

Please sign in to comment.