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

Regional Adblock doesn't work on Linux, follow up for #41 #1238

Closed
srirambv opened this issue Sep 21, 2018 · 7 comments
Closed

Regional Adblock doesn't work on Linux, follow up for #41 #1238

srirambv opened this issue Sep 21, 2018 · 7 comments

Comments

@srirambv
Copy link
Contributor

Description

Regional Adblock doesn't work on Linux, follow up for #41

Steps to Reproduce

  1. Clean install 0.55.5 on Linux
  2. Set browser language to non English
  3. Open chrome://adblock, regional adblock shows as disabled

Actual result:

On Linux having multiple languages, doesn't show option to Display Brave in this language
image

https://youtu.be/uXC0m_n3pOQ

Expected result:

Should set non English language as default and regional adblock should be enabled

Reproduces how often:

Easy

Brave version (chrome://version info)

Brave 0.55.5 Chromium: 70.0.3538.16 (Official Build) dev (64-bit)
Revision 16ed95b41bb05e565b11fb66ac33c660b721f778-refs/branch-heads/3538@{#306}
OS Linux

Reproducible on current release:

No

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Chrome?

Additional Information

Found while verifying #41
cc: @emerick @bbondy

@emerick
Copy link
Contributor

emerick commented Sep 21, 2018

@srirambv It seems that on Linux, this is controlled via the LANGUAGE environment variable (see https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/D6Eoeh0G9Y4 for the gory details).

When launching on Linux like this, I saw the expected settings in chrome://adblock:

LANGUAGE=fr_FR ./brave

@bbondy bbondy modified the milestones: Releasable builds 0.55.x, 1.x Backlog Sep 22, 2018
@rebron rebron modified the milestones: 1.x Backlog, Releasable builds 0.55.x Sep 28, 2018
@LaurenWags LaurenWags changed the title Regional Adblock doesn't work on Linux, follow up for #41 Regional Adblock doesn't work on Linux or macOS, follow up for #41 Oct 1, 2018
@LaurenWags
Copy link
Member

Verified steps from Description reproduce on macOS. However, if I set my system language/region to French/France:
screen shot 2018-10-01 at 17 24 21

and close/relaunch Brave, I see that Regional Adblock is enabled:
screen shot 2018-10-01 at 17 25 16

@kjozwiak kjozwiak removed their assignment Oct 4, 2018
@kjozwiak
Copy link
Member

kjozwiak commented Oct 4, 2018

@LaurenWags I think that's expected as @emerick mentioned that macOS and Linux relies on system level settings in #41 (comment) which I assume would require the browser to be restarted for the settings to take affect.

@bbondy does that sound correct? Seems like it's only affecting Linux in any case as @LaurenWags's screenshot shows that Regional Ad Block: Enabled after a browser restart.

@bbondy
Copy link
Member

bbondy commented Oct 5, 2018

I think that's correct, were you expecting something different to happen @LaurenWags ?

@LaurenWags
Copy link
Member

LaurenWags commented Oct 5, 2018

@bbondy nope, I'm good - @emerick responded with #41 (comment). Wanted to be sure I documented what I saw for historical purposes.

@kjozwiak kjozwiak changed the title Regional Adblock doesn't work on Linux or macOS, follow up for #41 Regional Adblock doesn't work on Linux, follow up for #41 Oct 5, 2018
@srirambv
Copy link
Contributor Author

srirambv commented Oct 5, 2018

I think this issue is easily solvable on Linux as well. Here's what i did

  1. Add language packs from system settings
  2. Change Language from settings and restart
  3. Launch Brave and load adblock page, shows regional ad block enabled
    regional adblock

@srirambv srirambv closed this as completed Oct 5, 2018
@srirambv srirambv reopened this Oct 5, 2018
@srirambv srirambv closed this as completed Oct 5, 2018
@bbondy bbondy modified the milestones: 1.x Backlog, FROZEN: 0.55.x - Release Oct 17, 2018
@srirambv
Copy link
Contributor Author

Verification Passed on

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

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

No branches or pull requests

6 participants