Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

"Open all..." option (Synced tabs, Bookmark Folder, etc) #11404

Closed
yoasif opened this issue Jun 9, 2020 · 5 comments · Fixed by #21212, #27138, fork-house/fenix#13 or nathanmkaya/fenix#108
Closed
Assignees
Labels
eng:qa:verified QA Verified Feature:Bookmarks feature request 🌟 New functionality and improvements Feature:SyncTabs Sync tabs
Milestone

Comments

@yoasif
Copy link
Contributor

yoasif commented Jun 9, 2020

What is the user problem or growth opportunity you want to see solved?

Users should be able to open all of their synced tabs in Fenix in order to quickly restore a session from another device.

How do you know that this problem exists today? Why is this important?

My Pixel is getting replaced due to a hardware issue. I want to maintain my user data from my previous device, and #2080 is not yet a feature. Google restores also don't restore my session from the cloud.

The closest feature to restore my tabs from my old device is sync, but it is extremely cumbersome to tap 3 times for every tab I want to migrate. I have over 100 tabs in my old device, so that at a minimum, 300 taps on my screen along with the sheer inefficiency of the process.

Who will benefit from it?

Anyone who wants to migrate their existing session to a new device.

┆Issue is synchronized with this Jira Task

@github-actions github-actions bot added the needs:triage Issue needs triage label Jun 9, 2020
@liuche liuche changed the title Open all synced tabs from a synced device "Open all..." option (Synced tabs, Bookmark Folder, etc) Jun 10, 2020
@liuche
Copy link
Contributor

liuche commented Jun 10, 2020

Maybe something in this context menu:
Screenshot_20200610-072810

@liuche liuche added feature request 🌟 New functionality and improvements Feature:Bookmarks Feature:SyncTabs Sync tabs and removed needs:triage Issue needs triage labels Jun 10, 2020
@data-sync-user data-sync-user changed the title "Open all..." option (Synced tabs, Bookmark Folder, etc) FNX3-14611 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) Aug 10, 2020
@data-sync-user data-sync-user changed the title FNX3-14611 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) FNX-12336 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-12336 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) FNX2-13450 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) Aug 11, 2020
@kbrosnan kbrosnan changed the title FNX2-13450 ⁃ "Open all..." option (Synced tabs, Bookmark Folder, etc) "Open all..." option (Synced tabs, Bookmark Folder, etc) Aug 29, 2020
@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Sep 19, 2022
rvandermeulen pushed a commit that referenced this issue Sep 19, 2022
- Create element to be displayed

- Update the interface and bind in the view holder
Set the filter to limit this action to FOLDER.

- Create core function
Main logic is done on the controller (has it should be done). The whole
process is done in one coroutine to be non-blocking as many
(sub)folders & links can be present. If folder is empty, a toast is
displayed. Else iterate on items. When item is:
- a FOLDER -> restart process (without toast) on the folder
- a ITEM -> open it
- a SEPARATOR -> do nothing
Once finished, show the tabs tray.

Toast message is defined in fragment to have access to context.

- Create androidTest for openAllInTabs
- Create tests for handleBookmarkFolderOpening
- Display 'open all' options only if folder has at least on child:
A coroutine and suspended functions have to be used, since `getTree`
is async.
rvandermeulen pushed a commit that referenced this issue Sep 19, 2022
Expiration date is set to the same of others bookmark metrics.
rvandermeulen pushed a commit that referenced this issue Sep 19, 2022
#21212 (comment)

- Add option is tree dot menu
- Edit folder opening to be aware of browsing mode
- Add metrics
- Add unit tests for 'Open all in private tabs'
- Add Android tests for open all in private.
rvandermeulen pushed a commit that referenced this issue Sep 19, 2022
rvandermeulen pushed a commit that referenced this issue Sep 19, 2022
Specify Firefox will only slow down while pages are loading.
@github-actions github-actions bot reopened this Sep 19, 2022
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Sep 19, 2022
@LaurentiuApahideanSV
Copy link

I tested the issue on Firefox Nightly 107.0a1 (2022-09-21) and the option to open all tabs from a bookmarks folder is present and functioning correctly. Additionally a message is displayed when trying to open 15 or more tabs at once.
1663771807590
1663771807467

Devices used:

  • Google Pixel 6 (Android 13)
  • Samsung Galaxy S22 Ultra (Android 12)

rocketsroger added a commit to rocketsroger/fenix that referenced this issue Oct 11, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Oct 11, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Oct 14, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Oct 17, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
rocketsroger added a commit to rocketsroger/fenix that referenced this issue Oct 25, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
@mergify mergify bot closed this as completed in #27138 Oct 26, 2022
mergify bot pushed a commit that referenced this issue Oct 26, 2022
* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
@github-actions github-actions bot added this to the 108 milestone Oct 26, 2022
@github-actions github-actions bot reopened this Oct 26, 2022
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Oct 26, 2022
@AdinaPetridean
Copy link

Verified as fixed on Nightly 108.0a1 from 10/27 with OnePlus 9 Pro (Android 12). "Open all in new tabs" and "Open all in private tabs" options are displayed in Bookmark folders menu.

@SoftVision-LorandJanos
Copy link

Verified as fixed on the latest Beta 108.0b1.
Device used: Google Pixel 4 (Android 13).

JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
- Create element to be displayed

- Update the interface and bind in the view holder
Set the filter to limit this action to FOLDER.

- Create core function
Main logic is done on the controller (has it should be done). The whole
process is done in one coroutine to be non-blocking as many
(sub)folders & links can be present. If folder is empty, a toast is
displayed. Else iterate on items. When item is:
- a FOLDER -> restart process (without toast) on the folder
- a ITEM -> open it
- a SEPARATOR -> do nothing
Once finished, show the tabs tray.

Toast message is defined in fragment to have access to context.

- Create androidTest for openAllInTabs
- Create tests for handleBookmarkFolderOpening
- Display 'open all' options only if folder has at least on child:
A coroutine and suspended functions have to be used, since `getTree`
is async.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…abs metric in Glean

Expiration date is set to the same of others bookmark metrics.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…s' option as requested.

mozilla-mobile/fenix#21212 (comment)

- Add option is tree dot menu
- Edit folder opening to be aware of browsing mode
- Add metrics
- Add unit tests for 'Open all in private tabs'
- Add Android tests for open all in private.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…lot of tabs are to be opened.

Current threshold is 15.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
… message

Specify Firefox will only slow down while pages are loading.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…kmarks tests pending compose refactoring"

This reverts commit f89aa29.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
… when a lot of tabs are to be opened."

This reverts commit 8517fea.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…vate tabs' option as requested."

This reverts commit 83ed1ab.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…in_new_tabs metric in Glean"

This reverts commit d4a38b3.
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…okmarks

* Add "Open all in new tabs" options in bookmarks
* Add "Open all in private tabs" options in bookmarks
* Add metrics tracking if the usage of "Open all in..." in bookmarks

Co-authored-by: Pg <pg.developper.fr@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.