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

Close tab button #1362

Closed
dimqua opened this issue Sep 22, 2017 · 10 comments
Closed

Close tab button #1362

dimqua opened this issue Sep 22, 2017 · 10 comments

Comments

@dimqua
Copy link

dimqua commented Sep 22, 2017

Please add a possibility to close a tab on a tabbar by using close button.

@codebu5ter
Copy link
Contributor

Currently, there's no way to close an individual tab, but to repeatedly press the back button until it closes. I've implemented a more user-intuitive way, in which the user just has to long press the tab and it will be closed.

Also, the process of closing all tabs requires two clicks at the moment. I've implemented the same functionality to close all tabs, in which the floating action button needs to be long pressed to close all tabs, which is more user-intuitive.

@Benestar
Copy link
Contributor

Benestar commented Nov 1, 2017

Instead of long pressing, I would rather implement some mechanism to swipe the tab away which seems to be more intuitive and consistent with material design guidelines (compare eg. the Gmail App). Also, in Firefox Mobile on Android, you can swipe a tab away to close it.

@bbinto
Copy link
Contributor

bbinto commented Nov 6, 2017

Adding this to the [meta]

@bbinto bbinto added the P5 label Nov 6, 2017
@lime124 lime124 modified the milestone: Reserve Backlog Nov 10, 2017
codebu5ter added a commit to codebu5ter/focus-android that referenced this issue Nov 19, 2017
codebu5ter added a commit to codebu5ter/focus-android that referenced this issue Nov 19, 2017
@pocmo
Copy link
Contributor

pocmo commented Nov 28, 2017

@lime124 @antlam We have a contributor PR that implements "swipe-to-dismiss" for tabs. Is this something we are interested in? Managing single tabs has been debated a lot in the past.

@antlam
Copy link
Contributor

antlam commented Nov 29, 2017

@pocmo I'm going to leave this to @lime124

But since this is a P5, I worry it could be kind of messy if we start pulling at this thread. This interaction on top of the expanded FAB isn't something that is very standard in Android and just feels a bit weird to me (versus say, simply an X button).

@lime124
Copy link
Collaborator

lime124 commented Nov 30, 2017

I agree with @antlam - i also worry about the ripple effects this could potentially have without a lot more investigation to make sure all the i's are dotted so to speak.

@bbinto bbinto modified the milestones: Backlog (P2, P3), Backlog (>P3) Jun 4, 2018
colintheshots pushed a commit to colintheshots/focus-android that referenced this issue Aug 22, 2018
asafh pushed a commit to asafh/focus-android that referenced this issue Jul 9, 2019
@asafh
Copy link

asafh commented Jul 10, 2019

Hi,
I've implemented (#4309) a swipe to dismiss over the new (kotlin) codebase. I believe the interaction feels natural for android.
Should I do anything to get this reviewed or just wait patiently?

@ekager
Copy link
Contributor

ekager commented Jul 15, 2019

Hi @asafh ! Thank you for wanting to contribute but we currently don't have the bandwith on our team to accept feature patches for Focus. Please check out one of our other projects if you want to get involved:
https://github.com/mozilla-mobile/android-components
https://github.com/mozilla-mobile/fenix

@asafh
Copy link

asafh commented Aug 1, 2019

Hi @ekager, that's unfortunate but no worries. I'll look into supporting my focus use case within fenix.

Cheers

@pocmo
Copy link
Contributor

pocmo commented Dec 7, 2021

This is getting implemented in #5741

@pocmo pocmo closed this as completed Dec 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
9 participants