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

brave://flags/#copy-link-to-text should be disabled by default #17994

Closed
Tonev opened this issue Sep 10, 2021 · 10 comments · Fixed by brave/brave-core#10085
Closed

brave://flags/#copy-link-to-text should be disabled by default #17994

Tonev opened this issue Sep 10, 2021 · 10 comments · Fixed by brave/brave-core#10085

Comments

@Tonev
Copy link
Contributor

Tonev commented Sep 10, 2021

Description

There is no reason brave://flags/#copy-link-to-text to remain enabled by default since the Copy link to highlight feature isn't available in Brave due to concerns shared by @pes10k over here - WICG/scroll-to-text-fragment#76.

Steps to Reproduce

  1. Select a random text.
  2. Right-click on it and select Copy link to highlight.
  3. Enter the copied link in the address bar to visit that specific section of the page.

Actual result:

Nothing, the feature isn't available in Brave.

Expected result:

The created link should send users to the specific highlighted section of the page.

Reproduces how often:

Easily reproduced.

Brave version (brave://version info)

Brave 1.29.79 Chromium: 93.0.4577.63 (Official Build) (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Windows 7 Service Pack 1 (Build 7601)
Brave 1.30.68 Chromium: 93.0.4577.63 (Official Build) beta (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Windows 7 Service Pack 1 (Build 7601)
Brave 1.31.30 Chromium: 93.0.4577.63 (Official Build) nightly (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Windows 7 Service Pack 1 (Build 7601)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? No

Miscellaneous Information:

@rebron
Copy link
Collaborator

rebron commented Sep 13, 2021

cc: @pes10k @mkarolin

@rebron rebron added QA/Yes release-notes/include feature/context-menu priority/P3 The next thing for us to work on. It'll ride the trains. labels Sep 13, 2021
@pes10k
Copy link
Contributor

pes10k commented Sep 13, 2021

no argument from me. If all thats needed here is to remove the UI-facing parts of the feature, that sounds 👍 to me.

Just to make sure there is no misunderstanding though, this is only about making the UI match the behavior, and not changing the behavior

@Tonev
Copy link
Contributor Author

Tonev commented Sep 14, 2021

no argument from me. If all thats needed here is to remove the UI-facing parts of the feature, that sounds to me.

Just to make sure there is no misunderstanding though, this is only about making the UI match the behavior, and not changing the behavior

Correct. I only suggest the removal of the UI part of the feature as long as the feature isn't available in Brave.

Right now, the default value of brave://flags/#copy-link-to-text is enabled, but at the same time the feature itself is missing in Brave, which makes the default enabled state meaningless.

@mkarolin
Copy link
Contributor

QA STR:

  1. Start Brave
  2. Navigate to any site
  3. Select some text on the page (that's not a link)
  4. Right-click to get the context menu

Observe:
Menu has an item "Copy link to highlight"

Expected result:
Menu does not have an item "Copy link to highlight"

@GeetaSarvadnya
Copy link

Verification passed on


Brave | 1.31.50 Chromium: 94.0.4606.50 (Official Build) nightly (64-bit)
-- | --
Revision | 0c1ac2c4842a4746c27c937c1a0453f98da1a972-refs/branch-heads/4606@{#1049}
OS | Windows 10 Version 21H1 (Build 19043.1165)

Reproduced the issue in 1.29.x
image

Confirmed menu doesn't have an item "Copy link to highlight" when text is selected on the website
image

@vegerot
Copy link

vegerot commented Apr 26, 2022

Why not keep this feature behind a flag?

@pirate
Copy link

pirate commented Aug 28, 2024

Bumping this as it's an extremely useful feature that I used daily in Chrome, and miss in Brave. I'd love to be able to enable it behind a flag as the minor security risk is easily tolerable for me.

@ronnie
Copy link

ronnie commented Sep 10, 2024

Please re-enable with a flag, I want this too and use it daily.

@mary-ext
Copy link

mary-ext commented Nov 19, 2024

Would like this to be brought back, part of what I do on a daily basis is referencing parts of documentations, where linking to a specific heading isn't enough. The copy highlighted link functionality allows me to highlight the relevant sentence of the things I talk about.

@pirate
Copy link

pirate commented Nov 20, 2024

I've opened a new issue to formally request they re-add this flag: #42393

Please comment/vote over there as comments on closed PRs and issues likely wont get seen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants