mgziminsky / FacebookTrackingRemoval

Browser extension that removes ads and the user interaction tracking from content on Facebook
GNU General Public License v3.0
121 stars 19 forks source link

Hiding Friends Like With Hidden Ads #45

Closed aeallord closed 4 years ago

aeallord commented 4 years ago

Describe the bug The problem I'm encountering, a person on my Friends List is being hidden behind the arrow/title to view content mask.

To Reproduce There are no "steps to reproduce" because it is a person on my friends list. I am including screenshots of what happens to the list of friends online when he appears in the list and a screenshot from when I go visit his page. There is no current post of his appearing in my timeline so I can't show what is happening there, but the results are the same with the arrow/title and then opening by clicking on the arrow.

Expected behavior Friends posts shouldn't be hidden

Screenshots Screen Shot 2020-10-30 at 6 23 59 PM Screen Shot 2020-10-30 at 6 24 10 PM Screen Shot 2020-10-30 at 6 29 13 PM Screen Shot 2020-10-30 at 6 33 04 PM

Please provide the following information:

Additional context Add any other context about the problem here.

mgziminsky commented 4 years ago

Thanks for reporting.

To find the source of the issue, enable logging under advanced options and disable the external and internal link cleaning options. then enable only the sponsored option and suggestions option individually to see which one is causing it. With just the problem option enabled, send me the relevant console logs.

This has happened occasionally in the past as FB starts re-using old css classes and such that were previously used exclusively for ads. I've moved on to a more robust method for detecting ads and suggestions, but I was waiting to remove the old code until the redesign had been fully rolled out.

I'm assuming you are a technical user based on your site linked in your github profile, but if that's not the case I can give more detailed instructions on how to obtain the information I'm looking for.

aeallord commented 4 years ago

Thanks for getting back to me…

Here is what I have logged from Sponsored Articles option only being selected

Screen Shot 2020-11-07 at 2 34 37 PM

Here is from the Hide Inline Suggestions option only being selected

Screen Shot 2020-11-07 at 2 35 51 PM

Looks like the Hide Sponsored Posts is causing the issue.

Please let me know if you would like additional debugging on my end. Your assumption is correct, my day job is as a software engineer… Glad to help with any diagnosis with this issue.

-Andy

On Nov 7, 2020, at 2:23 PM, Michael Ziminsky (Z) notifications@github.com wrote:

Thanks for reporting.

To find the source of the issue, enable logging under advanced options and disable the external and internal link cleaning options. then enable only the sponsored option and suggestions option individually to see which one is causing it. With just the problem option enabled, send me the relevant console logs.

This has happened occasionally in the past as FB starts re-using old css classes and such that were previously used exclusively for ads. I've moved on to a more robust method for detecting ads and suggestions, but I was waiting to remove the old code until the redesign had been fully rolled out.

I'm assuming you are a technical user based on your site linked in your github profile, but if that's not the case I can give more detailed instructions on how to obtain the information I'm looking for.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mgziminsky/FacebookTrackingRemoval/issues/45#issuecomment-723502594, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADEYJ2RWWLSTAOAOLJCQKO3SOXCF5ANCNFSM4TFS7SHQ.

mgziminsky commented 4 years ago

Well crap, the problem is your friends name lol

My simple and naive way of detecting ads is to just take the unique characters of the labels I want to hide and use that as a simple key. Your friends name is made up of the same letters that spell "Sponsored." I'm apparently not taking spaces into consideration currently, so that may be an easy fix, but I figured something like this would happen eventually.

I'll play around with it when I have time this weekend and should have a fix sometime soon.