brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.94k stars 2.35k forks source link

no option to open bookmark in Tor Window #1699

Open LaurenWags opened 6 years ago

LaurenWags commented 6 years ago

Description

Found while testing https://github.com/brave/brave-browser/issues/1387

Per description in 1387 it seems as though when on Bookmarks Manager, if you right click on a bookmark or click on the 3 dots menu for a bookmark, there should be an option to 'Open Private Window with Tor' (or similar wording). This option does not exist.

Steps to Reproduce

  1. If you don't have bookmarks, add at least one.
  2. Navigate to Bookmark Manager.
  3. Click on 3 dots menu for bookmark or right click on bookmark.

Actual result:

No option to open in Tor window from Bookmarks manager or from Bookmarks Bar: screen shot 2018-10-18 at 10 05 45 am

screen shot 2018-10-18 at 10 07 10 am

Expected result:

Option to open in Tor window.

Reproduces how often:

easy

Brave version (brave://version info)

Brave 0.55.17 Chromium: 70.0.3538.67 (Official Build) (64-bit)
Revision 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS Mac OS X

Reproducible on current release:

Website problems only:

Additional Information

Reproduced by @btlechowski and @srirambv on Win/Linux

LaurenWags commented 6 years ago

cc @tomlowenthal - the way I read https://github.com/brave/brave-browser/issues/1387 this should have been included (3 from description) but if this is unnecessary please advise.

tildelowengrimm commented 6 years ago

Good catch, you're right. Everywhere Chromium has open in new tab / window / incognito window we should add Tor windows.

riastradh-brave commented 6 years ago

Grah, I thought I'd dealt with that one. Apparently I didn't.

simonhong commented 6 years ago

I think the reason is tor window is based on guest profile. @darkdh

NumDeP commented 6 years ago

I'd like it if the labelling bookmarks feature in bookmarks of differentiating all the window types the bookmarks were saved under were developed alongside this and this #910 so users don't have to recall which window or tab type to open it under and so Brave can either prevent or at least warn a user if a tab is being opened under an unusual window type.

riastradh-brave commented 5 years ago

This is blocked on me figuring out which of the multiple bookmark manager UI mechanisms need to be updated; as I recall the draft I started a while back (https://github.com/brave/brave-core/tree/riastradh-1699-openbookmarktor) was in the wrong place.

DrEVILish commented 3 years ago

Currently still an issue one the Lastest version:

I'd like to also add a request for there to be an option on a bookmark to always open in TOR or to have a TOR specific url.