NY Times video captions

jfhutson · 01-28-2017, 08: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, 11: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, 11: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, 02: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, 01:49 PM

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

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

nevermind someone beat me to it

01-28-2017, 08: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.

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

01-28-2017, 11:17 PM	#2
kovidgoyal creator of calibre Posts: 45,617 Karma: 28549044 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, 11: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, 02:20 AM	#4
kovidgoyal creator of calibre Posts: 45,617 Karma: 28549044 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, 01: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, 01:50 PM	#6
bobbysteel Big Poppa Posts: 110 Karma: 10 Join Date: Jul 2010 Device: Nook	nevermind someone beat me to it

Advert

Advert