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

Bookmarks should open in existing tab when navigating to bookmarks UI from a page #13336

Closed
yoasif opened this issue Aug 6, 2020 · 66 comments · Fixed by #23946
Closed

Bookmarks should open in existing tab when navigating to bookmarks UI from a page #13336

yoasif opened this issue Aug 6, 2020 · 66 comments · Fixed by #23946
Assignees
Milestone

Comments

@yoasif
Copy link
Contributor

yoasif commented Aug 6, 2020

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

Fennec migration, ease of use, user expectation, competitiveness.

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

Existing behavior in Fennec. People asking for it: https://www.reddit.com/r/firefox/comments/i4kmyz/firefox_android_opening_sites_from_bookmarks/

Previously requested: #10837 #10645

"Don't make me think" and a time saver.

Existing behavior in Chrome (where most new users will be switching from) and Firefox desktop.

Who will benefit from it?

People who are directly navigating from existing pages to bookmarks no longer have to clean up their old tabs; anyone who is used to the behavior in other Android browsers.

┆Issue is synchronized with this Jira Task

@github-actions github-actions bot added the needs:triage Issue needs triage label Aug 6, 2020
@yoasif
Copy link
Contributor Author

yoasif commented Aug 6, 2020

@cadeyrn the best I could find is #3551 but that doesn't really capture the use case of opening a bookmarks when starting navigating from an existing tab, which is what my issue is about. I think that this is worth discussing.

@yoasif
Copy link
Contributor Author

yoasif commented Aug 6, 2020

I think this merits another look, especially since that use case is not called out in any design documentation or stories that I have access to. The best I can find with that continues to be #3551, and there has not been any discussion since then.

@person808 person808 added Feature:Bookmarks needs:triage Issue needs triage and removed needs:triage Issue needs triage labels Aug 6, 2020
@person808
Copy link
Contributor

This appears to be the reason for the current behavior: #1012 (comment)

@yoasif
Copy link
Contributor Author

yoasif commented Aug 6, 2020

@person808 Thanks for looking into that. Today, Library no longer exists, so I think this is still worth another look. 😄

@binocry
Copy link

binocry commented Aug 7, 2020

so strange when on bookmark page 3 dot menu you can select "open to new tab" so what the point open bookmark to another tab as default?

@liuche liuche added feature request 🌟 New functionality and improvements and removed needs:triage Issue needs triage labels Aug 10, 2020
@data-sync-user data-sync-user changed the title Bookmarks should open in existing tab when navigating to bookmarks UI from a page FNX2-18763 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page Aug 10, 2020
@data-sync-user data-sync-user changed the title FNX2-18763 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page FNX3-23036 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-23036 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page FNX-14517 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page Aug 11, 2020
@haltbarmilch
Copy link

The current behavior is not good for me. It would be great if i click on a bookmark it opens in the same tab, like other browsers.

@hotrodwinston
Copy link

The current behavior is not good for me. It would be great if i click on a bookmark it opens in the same tab, like other browsers.

Exactly this. Opening in a new tab by default without a way to change the behavior is going to drive people to other browsers - most of them chromium based making the web less open. At least give us an option.

@sonora
Copy link

sonora commented Sep 9, 2020

It also leads to secondary actions, more code, and complications: We now talk about automatically closing tabs in the background after a certain time, something which carries the risk of introducing even more in omprehensible behavior. And it was triggered by users working a lot with bookmarks and thus uncontrollably and mostly involuntarily opening large numbers of tabs...

@gtg947h
Copy link

gtg947h commented Sep 10, 2020

Every other browser opens bookmarks in the same/existing tab. Why does this need to be different? 90% of my browsing is from bookmarks and between this problem and having to choose close from the menu I spend way too much time clearing out old tabs.

@gtg947h
Copy link

gtg947h commented Sep 18, 2020

Every other browser opens bookmarks in the same/existing tab. Why does this need to be different? 90% of my browsing is from bookmarks and between this problem and having to choose close from the menu I spend way too much time clearing out old tabs.

This behavior is annoying me more and more... A 10 minute browsing session and I have 9 different tabs open; none of them were deliberately opened as new tabs. What gives?

Between this and #5865, I find myself purging tabs every few minutes and it's really driving me nuts. Yes, my desktop browser is set to do this, but that's almost an entirely different animal.

@haltbarmilch
Copy link

Every other browser opens bookmarks in the same/existing tab. Why does this need to be different? 90% of my browsing is from bookmarks and between this problem and having to choose close from the menu I spend way too much time clearing out old tabs.

This behavior is annoying me more and more... A 10 minute browsing session and I have 9 different tabs open; none of them were deliberately opened as new tabs. What gives?

Between this and #5865, I find myself purging tabs every few minutes and it's really driving me nuts. Yes, my desktop browser is set to do this, but that's almost an entirely different animal.

You speack from my heart. ;-)

@Diapolo
Copy link

Diapolo commented Sep 23, 2020

Every other browser opens bookmarks in the same/existing tab. Why does this need to be different? 90% of my browsing is from bookmarks and between this problem and having to choose close from the menu I spend way too much time clearing out old tabs.

This behavior is annoying me more and more... A 10 minute browsing session and I have 9 different tabs open; none of them were deliberately opened as new tabs. What gives?

Between this and #5865, I find myself purging tabs every few minutes and it's really driving me nuts. Yes, my desktop browser is set to do this, but that's almost an entirely different animal.

You are getting the point here, it feels EXTREMELY weird using the "new" Firefox for Android, that I consinder not using it anymore. Why did the devs chose such an ugly and nonsene behaviour at all? It's also weird that clicking into the addressbar doesn't allow to anyhow select my favorites? WHO did "invent" this? Yes that is an inproductive rant, but it's also a pitty that Mozilla destroys it's user base by intention (as it seems).

This needs to be fixed, changed, looked at.

mergify bot pushed a commit that referenced this issue Feb 23, 2022
mergify bot pushed a commit that referenced this issue Feb 23, 2022
This reverts commit e73deb2.

(cherry picked from commit d1c0e9b)
rocketsroger added a commit that referenced this issue Feb 23, 2022
This reverts commit e73deb2.

(cherry picked from commit d1c0e9b)

Co-authored-by: Roger Yang <royang@mozilla.com>
@czlucius
Copy link
Contributor

czlucius commented Feb 23, 2022

Oh no, we finally made it and now this :( Can't the behavior be fixed instead of directly falling back to reverting this change?

It seems like opening in the current tab is more problematic than opening in a new tab, given Fenix's current state. I think reverting the change will be better for Firefox Fenix, so users would not be confused, and there'll be more time to investigate the issue before rolling out to end users.

To the Fenix devs (rocketsroger & jonalmeida), sorry for troubling you in this fix.

@maverick74
Copy link

@rocketsroger can you please also work on the pinned bookmarks on the home tab

Improve home screen logic, always associate Home screen with an actual tab(#20012 (comment))

They apparently share a lot...

@Cheap-Skate
Copy link

@czlucius thanks for all your hard work. I think it will bring benefits because it highlights how broken Fenix is at the moment. All other browsers give a simple choice to open in a new or existing tab. But Fenix does not, it is hardwired to open everything in a new tab.

Hopefully devs will take notice (I expect they already are). But I think fixing Fenix properly will be quite complicated.

@czlucius
Copy link
Contributor

because it highlights how broken Fenix is at the moment. 

Just to clarify, I don't view Fenix as broken.

rocketsroger added a commit to rocketsroger/fenix that referenced this issue Feb 23, 2022
…mode is not private or previous fragment is not home
@gabrielluong gabrielluong added this to the 99 milestone Feb 23, 2022
@mergify mergify bot closed this as completed in #23946 Feb 24, 2022
mergify bot pushed a commit that referenced this issue Feb 24, 2022
@gabrielluong gabrielluong reopened this Feb 24, 2022
@gabrielluong gabrielluong added the eng:qa:needed QA Needed label Feb 24, 2022
@rocketsroger
Copy link
Contributor

Thanks for reopening and tagging for me 👍

@SoftVision-LorandJanos
Copy link

Verified as fixed on the latest Nightly build 99.0a1 (2022-02-24).
Device used: Oppo Reno 6 (Android 11).
Closing the ticket as fixed.

@SoftVision-LorandJanos SoftVision-LorandJanos added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Feb 24, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 10, 2022
…le#23169)

* For mozilla-mobile#13336: Open bookmarks in current tab

* For mozilla-mobile#13336: Fix tests to verify bookmark opening in current tab

* Change test name for handleBookmarkTapped
pedroldk pushed a commit to pedroldk/fenix that referenced this issue Mar 22, 2022
pedroldk pushed a commit to pedroldk/fenix that referenced this issue Mar 22, 2022
…mode is not private or previous fragment is not home
@data-sync-user data-sync-user changed the title FNX-14517 ⁃ Bookmarks should open in existing tab when navigating to bookmarks UI from a page Bookmarks should open in existing tab when navigating to bookmarks UI from a page May 19, 2022
@r6squeegee
Copy link

r6squeegee commented May 24, 2022

I don't think this is fixed, 23946 only fixes this if you manually click on "settings" icon then "bookmarks" then click on a bookmark.

The scenario of bookmarks showing on the home page, doesn't work. This is my most common flow :

  1. I'm on a web page, and done with it
  2. I click on the url field to bring up the home screen so I can see my book marks (1 click on a big bar, adverse to 2 clicks for going to settings (small icon) then bookmarks (on a popup menu))
  3. I click on the next bookmark

That use to open the new page in the existing tab, it doesn't anymore. If I go through that routine with 8 bookmarks I end up with 8 tabs.

@rocketsroger
Copy link
Contributor

I don't think this is fixed, 23946 only fixes this if you manually click on "settings" icon then "bookmarks" then click on a bookmark.

Created an issue to track this specific workflow. #25362. Thanks

@Obscerno
Copy link

Thanks for your work on this @rocketsroger! If you can make this fix, we'll be one step closer to untangling the homepage.

And if you want to go down this rabbit hole, this comment lists some of the other strange behaviour on the homepage: #20012 (comment)

@rocketsroger
Copy link
Contributor

And if you want to go down this rabbit hole, this comment lists some of the other strange behaviour on the homepage: #20012 (comment)

Looks like an issue was opened by the contributor. That said, it sounds very similar to the issue you have here with bookmarks on homepage.

@plwt
Copy link

plwt commented Aug 24, 2022

User feedback - https://support.mozilla.org/en-US/questions/1387083

@maverick74
Copy link

Vote about how home opens bookmarks (new tab vs current tab) ongoing at:

https://connect.mozilla.org/t5/ideas/on-firefox-mobile-make-home-page-links-open-in-current-tab/idi-p/15672

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.