NY Times video captions

jfhutson · 01-28-2017, 07:13 PM

I have another minor request for the NYT recipe. The recipe strips out the videos, but not the captions. I think you need the remove the "caption-video" div class.

kovidgoyal · 01-28-2017, 10:17 PM

Not my recipe but I'd guess the recipe author left those in deliberately so you know there is a video there in the original and what the video was about. In any case you can always customize your local copy of th erecipe to remove those if you want.

jfhutson · 01-30-2017, 10:34 PM

I don't see them as helpful (you'll notice in print mode the website leaves them out). I figured out how to remove them, if anyone else is interested. In addition to adding 'caption-video' to 'remove-tags', I needed to add the following 'dict(name='h4', attrs={'class': 'headline'})' to get rid of an extra headline that was showing up. My code is here: https://github.com/jfhutson/calibre-...ter/nyt.recipe (with several other modifications).

kovidgoyal · 01-31-2017, 01:20 AM

Since the original author of the recipe is no longer active, if you wish to maintain this recipe, feel free to send a pull request for it.

bobbysteel · 02-03-2017, 12:49 PM

i think this change makes sense. i'll submit now.

bobbysteel · 02-03-2017, 12:50 PM

nevermind someone beat me to it

01-28-2017, 07:13 PM	#1
jfhutson Junior Member Posts: 7 Karma: 10 Join Date: Jan 2017 Device: Kindle Paperwhite	NY Times video captions I have another minor request for the NYT recipe. The recipe strips out the videos, but not the captions. I think you need the remove the "caption-video" div class.

Thread Tools	Search this Thread
Show Printable Version Email this Page	Search this Thread: Advanced Search

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Photo captions	anleva	Recipes	5	11-06-2016 11:11 AM
images and captions	JSWolf	ePub	13	04-07-2015 11:27 AM
How do you make captions?	GraciousMe	ePub	6	08-05-2012 03:22 PM
Figure captions	amoroso	Sigil	6	07-13-2010 02:29 PM
captions and images together	detede	Apple Devices	11	06-14-2010 09:00 PM

01-28-2017, 10:17 PM	#2
kovidgoyal creator of calibre Posts: 45,364 Karma: 27230406 Join Date: Oct 2006 Location: Mumbai, India Device: Various	Not my recipe but I'd guess the recipe author left those in deliberately so you know there is a video there in the original and what the video was about. In any case you can always customize your local copy of th erecipe to remove those if you want.

01-30-2017, 10:34 PM	#3
jfhutson Junior Member Posts: 7 Karma: 10 Join Date: Jan 2017 Device: Kindle Paperwhite	I don't see them as helpful (you'll notice in print mode the website leaves them out). I figured out how to remove them, if anyone else is interested. In addition to adding 'caption-video' to 'remove-tags', I needed to add the following 'dict(name='h4', attrs={'class': 'headline'})' to get rid of an extra headline that was showing up. My code is here: https://github.com/jfhutson/calibre-...ter/nyt.recipe (with several other modifications).

01-31-2017, 01:20 AM	#4
kovidgoyal creator of calibre Posts: 45,364 Karma: 27230406 Join Date: Oct 2006 Location: Mumbai, India Device: Various	Since the original author of the recipe is no longer active, if you wish to maintain this recipe, feel free to send a pull request for it.

02-03-2017, 12:49 PM	#5
bobbysteel Big Poppa Posts: 110 Karma: 10 Join Date: Jul 2010 Device: Nook	i think this change makes sense. i'll submit now.

02-03-2017, 12:50 PM	#6
bobbysteel Big Poppa Posts: 110 Karma: 10 Join Date: Jul 2010 Device: Nook	nevermind someone beat me to it

Advert

Advert