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

Disable sharing-qr-code-generator feature. #15008

Closed
mkarolin opened this issue Mar 29, 2021 · 19 comments · Fixed by brave/brave-core#8385
Closed

Disable sharing-qr-code-generator feature. #15008

mkarolin opened this issue Mar 29, 2021 · 19 comments · Fixed by brave/brave-core#8385

Comments

@mkarolin
Copy link
Contributor

Per @rebron, this feature should be disabled by default in Brave.

Steps to reproduce:

  1. Start Brave browser
  2. Navigate to any website (e.g. brave.com)
  3. Click inside the location bar
  4. An action icon shows up in the location bar. When clicked a bubble with QR code shows up.
    image

Expected behavior:

  • No action icon shows up in the step 4 above.
@mkarolin mkarolin self-assigned this Mar 29, 2021
mkarolin added a commit to brave/brave-core that referenced this issue Mar 30, 2021
Fixes brave/brave-browser#15008

Chromium change:

https://source.chromium.org/chromium/chromium/src/+/60f0b23dc9b6f1f6ba5ddd8d763a4f60f2f501b5

commit 60f0b23dc9b6f1f6ba5ddd8d763a4f60f2f501b5
Author: Travis Skare <[email protected]>
Date:   Tue Feb 2 03:04:13 2021 +0000

    Mark QRCode generator enabled by default

    BUG: 982914
@rodrigoswz
Copy link

Why? This is a great chromium feature

@dentistformyeye
Copy link

@rodrigoswz I don't know about specific privacy/security problems, but the feature is unnecessary. If you use DuckDuckGo, search for qr <paste URL here>: example.

@stephendonner
Copy link

Verified PASSED using

Brave 1.23.66 Chromium: 89.0.4389.114 (Official Build) beta (x86_64)
Revision 1ea76e193b4fadb723bfea2a19a66c93a1bc0ca6-refs/branch-heads/4389@{#1616}
OS macOS Version 11.2.3 (Build 20D91)

Verified using the inline testplan here that there is no longer a QR-code generator icon in the URL bar, upon focus/click.

@siltentassasin
Copy link

Why removing this feature, I see no disadvantages ... I found this feature realy comfortable

@NathanaelA
Copy link

NathanaelA commented Apr 15, 2021

@rebron @mkarolin - Can you give a good or security reason to actually remove this feature?
This also isn't listed in the features you disable list so there doesn't seem to be any reason anywhere why this feature was actually removed.

@rodrigoswz
Copy link

I would also like to know, does this use Google servers or anything similar to generate qrcodes?

I used it a lot, now I will have to look for some extension (or use the DuckDuckGo flag mentioned above).

Which doesn't make sense for me since it was something native to Chromium...

@rebron
Copy link
Collaborator

rebron commented Apr 20, 2021

If you'd like to add this function into the url bar, you can do it here: brave://flags/#sharing-qr-code-generator
This type of feature is why we built extensions into the browser, to allow whatever feature user's want while doing our best to maintain some UI/UE discipline to appeal to the majority of our users.

With that said, we can evaluate whether to surface/highlight this feature makes sense down the road and evaluate its current placement/prominence. For the time being, the flag or an extension are your workarounds.

@rodrigoswz FLoC is also native to Chromium and we removed that as a recent example items we remove/neutralize.

@NathanaelA
Copy link

@rebron - Thanks, I've had the QR flag active since it was a early field test, and it still wasn't showing up -- which is why I started digging into what happened it. Do you know if a recent update recently fixed the flag to work again, because it is showing in my URL bar now (Yay!) and it wasn't 6 days ago...

@Taomyn
Copy link

Taomyn commented May 10, 2021

The option to enable it via flags no longer works - no icon or right-click menu appears.

Version 1.24.82 Chromium: 90.0.4430.93 (Official Build) (64-bit)

@bsclifton
Copy link
Member

bsclifton commented May 14, 2021

Opened up a security/privacy review so we can evaluate this - thanks for your patience and feedback all 😄 If that looks clean, we can re-enable. There is still a branding issue - but @jonathansampson has a potential fix we can do for the dino that shows up
#11606

@bsclifton
Copy link
Member

OK folks - our next Nightly will have this feature re-enabled 😄 (merged with brave/brave-core#8825)

Next step would be to "Bravify" it if we get the chance 😛 The dino isn't all that bad though

@soodmax
Copy link

soodmax commented Jun 16, 2021

I still do not see this feature in brave. Is this now enabled? It was really useful. How do I get it in brave now?

@stephendonner
Copy link

I still do not see this feature in brave. Is this now enabled? It was really useful. How do I get it in brave now?

@soodmax it can be found in the nightly build, available here: https://brave.com/download-nightly/

@Amariithynar
Copy link

How in the everloving crap do I re-disable this annoying function?

@rodrigoswz
Copy link

How in the everloving crap do I re-disable this annoying function?

Seriously? What's the problem with just not using it?

If you don't click on this little icon, you'll never know it exists.

@Amariithynar
Copy link

Amariithynar commented Jun 28, 2021

How in the everloving crap do I re-disable this annoying function?

Seriously? What's the problem with just not using it?

If you don't click on this little icon, you'll never know it exists.

I don't care about the icon in the URL bar that much; however, having it shoved in my face in the right-click context menu every single time, moreover with it having its own separate section to even further bloat it? THAT I have serious issue with.

Right-Click Context Menu
Shown here with no extensions; with extensions, especially those used frequently like Form History Control or UBlock element blocker or such, it gets much worse. Furthermore, there's a thick bar in the menu as shown below that is completely unnecessary that was added with the addition of the QR code menu option.
Right-Click Context Menu Extensions Enabled

It may not seem like much to you, but it's a fair amount of space that I now have to accomodate with every right-click context menu usage, and even moreso it's extra space that forces the menu to open upwards further away from the bottom of the screen than before.

@NathanaelA
Copy link

@Amariithynar - Not sure why it has a right mouse context, can't say I've ever noticed it. But I can understand your annoyance with it. I don't use the context menu as often.

I would open a new bug report for either removing the option in the context menu, or adding a new setting to enable/disable it in whole as the flags option seems to be gone now... If you know programming, then adding a PR to remove the context option would probably be fastest method to get back to your "normal".

@Amariithynar
Copy link

@NathanaelA I'm not a programmer myself, unfortunately, and know nothing about fiddling with the context menu at all, either. I'll try opening a new issue referencing this one.

@Neurotoxin001
Copy link

Can you please give option to disable it?

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