Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add confirmation to exit if closing an incognito tab closes the browser with other incognito tabs open #1606

Open
4 tasks done
BugReportI opened this issue Oct 23, 2024 · 7 comments
Labels
need triage I will look into it, I promise!

Comments

@BugReportI
Copy link

BugReportI commented Oct 23, 2024

Preliminary checklist

  • I have read the README
  • I have searched existing issues for my feature request. This is a new issue (NOT a duplicate) and is not related to another issue.
  • I have searched wont fix issues and this request is not among them
  • This is a feature request for the Cromite browser; not the website nor F-Droid nor anything else.

Is your feature request related to privacy?

No

Is there a patch available for this feature somewhere?

Not sure.

Describe the solution you would like

Closing a tab by back button/gesture should not close all tabs in incognito.

Also make this option work if the above is done through normal tabs so if user does not want the incognito tabs to be closed they will not be closed unless user explicitly closes them but does want to close a specific tab.

Describe alternatives you have considered

I do not think there is one.

@uazo uazo changed the title Add comfirm to close tabs in incoginto option Add comfirm to close tabs in incognito option Oct 24, 2024
@uazo
Copy link
Owner

uazo commented Oct 24, 2024

Also make this option work if the above is done through normal tabs so if user does not want the incognito tabs to be closed they will not be closed unless user explicitly closes them but does want to close a specific tab.

No, closing all incognito windows also deletes the profile and thus the deletion of all data.
Allows the user to reopen a new, clean incognito session.

@uazo uazo closed this as not planned Won't fix, can't repro, duplicate, stale Oct 24, 2024
@BugReportI
Copy link
Author

Also make this option work if the above is done through normal tabs so if user does not want the incognito tabs to be closed they will not be closed unless user explicitly closes them but does want to close a specific tab.

No, closing all incognito windows also deletes the profile and thus the deletion of all data. Allows the user to reopen a new, clean incognito session.

I know closing all tabs should do that.

I mean when you try to close one tab clicking back button or gestures then it should not close all the other incognito tabs.

This only happens if you cannot navigate any further back on the tab that you are navigating from and this closes Cromite or just the Cromite window if you have multiple windows of Cromite open.

I do not think anyone wants all the other incognito tabs to be closed just because they use back button/gestures(not swipe to close tab) to close just one tab.

You can try this by opening multiple incognito tabs and closing one of them by pressing the back button or gestures(not swipe to close tab) and it will close all incognito tabs not just that one incognito tab you want to close. The same thing happens when trying to close normal tab it should not close the all incognito tabs if the user only wants to close one tab like this.

I cannot record a video of this because it is in incognito and changing the flag to allow does not allow to record so I cannot provide video showing this.

@BugReportI
Copy link
Author

I cannot record a video of this because it is in incognito and changing the flag to allow does not allow to record so I cannot provide video showing this.

If this is as I think related to Android and you have Root+Magisk/XPosed/LSposed, then there are Modules which can bypass Flag_Secure and allow you to make a screenshot / record a video no matter what (in theory, I don't have any of the above). I hope this is helpful to the OP.

Is there a specific one?

I tried one and it does allow to take screenshots and record video but it does not close any of the incognito tabs by doing the actions mentioned in my previous post.

So I do not know how to proceed if the only way to record is to disable secure flag but doing so also does not close the incognito tabs.

@uazo uazo changed the title Add comfirm to close tabs in incognito option Add confirmation to exit if closing an incognito tab closes the browser with other incognito tabs open Oct 25, 2024
@uazo uazo reopened this Oct 25, 2024
@uazo uazo added the need triage I will look into it, I promise! label Oct 25, 2024
@BugReportI
Copy link
Author

I tried some other Magisk modules and they seem to behave the same way as the initial one I tried.

I think it would be good to treat the other incognito tabs as the same way normal tabs are treated when you are just trying to only close one tab regardless of if it is incognito tab or not. If adding a confirm is no good.

@BugReportI
Copy link
Author

BugReportI commented Oct 28, 2024

I tried out Bromite in incognito and it does not seem to close all incognito tabs if I press back button to close one tab and it closes Bromite.

The Bromite version is 108.0.5359.156.

Edited to add: Could the way Bromite handles incognito tabs in these situations be used in Cromite?

@BugReportI
Copy link
Author

It seems I misunderstood how Bromite handles the closing tabs in incognito, it seems to close the browser but not the tab if you press back button.

So it would not help if user just wants to close one tab this way in Cromite.

@BugReportI
Copy link
Author

BugReportI commented Nov 8, 2024

It seems to behaves differently in Cromite 130.0.6723.117.

Now it does not close the other incognito tabs or does not close the single incognito/normal tab through back button/gesture if the flags Redact sensitive content during screen sharing, screen recording, and similar actions is disabled and Incognito Screenshot is enabled.

In both situations the incognito tabs are kept.

Edited to add: Correct flags.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need triage I will look into it, I promise!
Projects
None yet
Development

No branches or pull requests

2 participants