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

Blank page for few seconds on initial load #23154

Closed
wooque opened this issue May 31, 2022 · 5 comments · Fixed by brave/adblock-lists#968
Closed

Blank page for few seconds on initial load #23154

wooque opened this issue May 31, 2022 · 5 comments · Fixed by brave/adblock-lists#968

Comments

@wooque
Copy link

wooque commented May 31, 2022

Description

Some websites have delay in showing full page, they show as blank white page at first and after few seconds they reveal in full. Inspecting it I noticed that html tag has async-hide class that is removed after few seconds and that lead me to past issue that seems to be back again #4402

Steps to Reproduce

  1. Open https://www.kupujemprodajem.com/index.php

Actual result:

It shows blank white page, after few seconds it shows fully

Expected result:

No artificial blank white page delay

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 1.39.111 Chromium: 102.0.5005.61 (Official Build) (64-bit)
Revision 0e59bcc00cc4985ce39ad31c150065f159d95ad3-refs/branch-heads/5005@{#819}
OS Linux
JavaScript V8 10.2.154.4
User Agent Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/102.0.5005.61 Safari/537.36
Command Line /opt/brave.com/brave/brave --enable-features=OverlayScrollbar,VaapiVideoDecoder --enable-gpu-rasterization --enable-zero-copy --disable-features=UseChromeOSDirectVideoDecoder --disable-smooth-scrolling --enable-crashpad --disable-client-side-phishing-detection --disable-domain-reliability --enable-dom-distiller --no-pings --component-updater=url-source=https://go-updater.brave.com/extensions --origin-trial-public-key=bYUKPJoPnCxeNvu72j4EmPuK7tr1PAC7SHh8ld9Mw3E=,fMS4mpO6buLQ/QMd+zJmxzty/VQ6B1EUZqoCU04zoRU= --sync-url=https://sync-v2.brave.com/v2 --lso-url=https://no-thanks.invalid --variations-server-url=https://variations.brave.com/seed --variations-insecure-server-url=https://variations.brave.com/seed --enable-features=VaapiVideoDecoder,OverlayScrollbar --disable-features=UseChromeOSDirectVideoDecoder --flag-switches-begin --flag-switches-end
Executable Path /opt/brave.com/brave/brave
Profile Path /home/vuk/.config/BraveSoftware/Brave-Browser/Default
Variations AdRewardsStudy:NextPaymentDayAdServingStudy:DefaultAdNotificationsPerHour=10/MaximumAdNotificationsPerDay=100/MaximumInlineContentAdsPerHour=12/MaximumInlineContentAdsPerDay=60BraveAds.UserActivityStudy:Triggers=EMPTY/Threshold=0.0/IdleTimeThreshold=5BraveDebounceStudy:EnabledConversionsStudy:NewDefaultPatternDefault1pBlockingStudy:DisabledEphemeralStorageStudy:EnabledExtendedVariationsSafeMode5:SignalAndWriteViaFileUtil5FirstPartyEphemeralDomainBlockStudy:EnabledForceDisableBlinkFeatures:EnabledNativeCosmeticFilteringStudy:EnabledNavigationThreadingOptimizationsCompat:DisabledPartitionBlinkMemoryCacheStudy:EnabledPartitionConnectionsByNetworkIsolationKeyStudy:EnabledPermissionLifetimeReleaseStudy:EnabledRequestAdsEnabledApiStudy:EnabledSpeedreaderReleaseStudy:Enabled

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

@wooque wooque changed the title Page blank for few seconds Blank page for few seconds on initial load May 31, 2022
@rebron rebron added feature/shields/adblock Blocking ads & trackers with Shields regression labels Jun 1, 2022
@wooque
Copy link
Author

wooque commented Jun 20, 2022

I had similar issue on accuweather.com and I solved it with customer filters

accuweather.com##+js(remove-attr, class, .ads-not-loaded, stay)
kupujemprodajem.com##+js(remove-attr, class, .async-hide, stay)
polovniautomobili.com##+js(remove-attr, class, .async-hide, stay)

@bsclifton
Copy link
Member

@wooque is this still happening?
cc: @ryanbr

@ryanbr
Copy link

ryanbr commented Jun 21, 2022

Sorry I missed the post, has these delays always been here? just tested FF on https://www.polovniautomobili.com/auto-oglasi/19998599/bmw-520?attp=p8_pv0_pc1_pl9_plv0 Noticed a delay here also @wooque

||googletagmanager.com/gtm.js$script,redirect-rule=googletagmanager_gtm.js:5 from uBO prevents the delays, we are implementing uBO redirect-rule soon-ish which should help

In Brave Nightly, add these to brave://adblock

||securepubads.g.doubleclick.net/tag/js/gpt.js$script,redirect-rule=googletagservices_gpt.js
||googletagmanager.com/gtm.js$script,redirect-rule=googletagmanager_gtm.js
||google-analytics.com/analytics.js$script,redirect-rule=google-analytics.com/analytics.js

Should fix the delays

@wooque
Copy link
Author

wooque commented Jun 21, 2022

@bsclifton yes, it's still happening
@ryanbr for accuweather.com and kupujemprodajem.com it started relatively recently, didn't saw delays before. I don't know if they changed something on their sites or it's Brave Shields regression.
Note: Accuweather is not blank page but it shows artificial loading animation for few seconds before it shows full content https://www.accuweather.com/en/rs/belgrade/298198/weather-forecast/298198

Luckily found workaround with ##+js(remove-attr, class, .class, stay) until those redirect rules are added

@wooque
Copy link
Author

wooque commented Sep 8, 2022

@ryanbr Fix has a bug, only accuweather.com is fixed by removing .ads-not-loaded, kupujemprodajem.com i polovniautomobili.com are fixed by removing .async-hide

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.

4 participants