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

Additions to easylist_general_hide.txt #254

Merged
merged 1 commit into from Jan 24, 2017
Merged

Additions to easylist_general_hide.txt #254

merged 1 commit into from Jan 24, 2017

Conversation

diaux
Copy link
Contributor

@diaux diaux commented Jan 23, 2017

Found several div IDs and classes that, imho, should be included into EasyList.

@Khrin
Copy link
Member

Khrin commented Jan 24, 2017

I can't find these elements on the page you've referenced...

@diaux
Copy link
Contributor Author

diaux commented Jan 24, 2017

Jackpot! They changed the code of the website in between my commit and your checkout, it was there when I checked. See this link for the website source code 4 days ago (January 20th). Your call: (1) include the filters into EasyList if you like or (2) close this Pull Request without merging anything. Either way, I will have to go through the website again to adjust/add any filters for EasyList Spanish. Thanks much Khrin!

@diaux
Copy link
Contributor Author

diaux commented Jan 24, 2017

Markdown I hate you! Here's the link, sorry: http://web.archive.org/web/20170120090505/http://deportes.televisa.com/

@Khrin Khrin merged commit b2e3a38 into easylist:master Jan 24, 2017
@Khrin
Copy link
Member

Khrin commented Jan 24, 2017

We still have the references, still valid for me.

@diaux diaux deleted the patch-4 branch January 24, 2017 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants