Index not working for entries within

1v4n0 · 10-27-2017, 07:49 AM

I hope this is the right thread. I pretty sure, as per the title, the sigil index creator does not indicize the entries that are within EM tags. It does include the words within tag though, so if I search and replace all instances of with it works.

Can anyone confirm? Is there another way out?

Also, is there a way to make it case insensitive?

Thanks.

DiapDealer · 10-27-2017, 08:44 AM

Sigil's auto-index generation routines do not pay the least bit of attention to how a word may be formatted. It only gathers the words themselves.

DiapDealer · 10-27-2017, 09:13 AM

Sorry. I misunderstood the issue being described. Someone has confirmed that words inside em tags get ignored during index generation. We'll look into it.

KevinH · 10-27-2017, 12:42 PM

I think we need to look at XhtmlDoc.cpp and look at how ID_TAGS are defined and possibly add "em" to this list. I have not walke the code as I am away from my dev box, so that is just a wag.

DiapDealer · 10-27-2017, 02:42 PM

Thanks, Kevin. I've not looked at that code either.

JSWolf · 10-27-2017, 06:49 PM

Would you also have to add where you have ?

KevinH · 10-28-2017, 08:56 AM

Both strong and b tags are already part of ID_TAGS. That function that uses ID_TAGS is invoked from the indexer code to get the text of a node but I really need to sit down to understand how it works first before knowing if this is the culprit or not.

KevinH · 10-29-2017, 03:44 PM

Just pushed to master the fix for this. Index terms inside em tags are now properly marked and used during Index generation.

Turtle91 · 10-29-2017, 04:51 PM

You guys are awesome!

Do you have a best guestimate when the next official release will be posted?

KevinH · 10-29-2017, 06:24 PM

I am hoping in about 2 or 3 weeks once my teaching ends and I have more free time.

Quote:

Originally Posted by Turtle91

You guys are awesome!

Do you have a best guestimate when the next official release will be posted?

1v4n0 · 10-30-2017, 09:53 AM

Thank you. That was fast

10-27-2017, 07:49 AM	#1
1v4n0 Groupie Posts: 171 Karma: 40000 Join Date: Oct 2013 Device: kindle	Index not working for entries within <em> I hope this is the right thread. I pretty sure, as per the title, the sigil index creator does not indicize the entries that are within EM tags. It does include the words within < i > tag though, so if I search and replace all instances of < em > with < i > it works. Can anyone confirm? Is there another way out? Also, is there a way to make it case insensitive? Thanks. Last edited by 1v4n0; 10-27-2017 at 08:06 AM.

10-27-2017, 06:49 PM	#6
JSWolf Resident Curmudgeon Posts: 73,835 Karma: 128597114 Join Date: Nov 2006 Location: Roslindale, Massachusetts Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3	Would you also have to add <strong> where you have <b>?

10-28-2017, 08:56 AM	#7
KevinH Sigil Developer Posts: 7,602 Karma: 5433388 Join Date: Nov 2009 Device: many	Both strong and b tags are already part of ID_TAGS. That function that uses ID_TAGS is invoked from the indexer code to get the text of a node but I really need to sit down to understand how it works first before knowing if this is the culprit or not. Last edited by KevinH; 10-28-2017 at 09:44 AM.

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
Troubleshooting Ghost entries	procyon	Amazon Kindle	2	08-24-2016 12:36 PM
Error: Index has too many entries...	Katsunami	Conversion	6	10-12-2013 09:06 AM
Error: Too many index entries	pimol	Conversion	2	10-16-2012 08:18 AM
Index: Making a linked index in epub	virtual_ink	ePub	21	10-19-2011 11:23 PM

10-27-2017, 08:44 AM	#2
DiapDealer Grand Sorcerer Posts: 27,532 Karma: 193191846 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Sigil's auto-index generation routines do not pay the least bit of attention to how a word may be formatted. It only gathers the words themselves.

10-27-2017, 09:13 AM	#3
DiapDealer Grand Sorcerer Posts: 27,532 Karma: 193191846 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Sorry. I misunderstood the issue being described. Someone has confirmed that words inside em tags get ignored during index generation. We'll look into it.

10-27-2017, 12:42 PM	#4
KevinH Sigil Developer Posts: 7,602 Karma: 5433388 Join Date: Nov 2009 Device: many	I think we need to look at XhtmlDoc.cpp and look at how ID_TAGS are defined and possibly add "em" to this list. I have not walke the code as I am away from my dev box, so that is just a wag.

10-27-2017, 02:42 PM	#5
DiapDealer Grand Sorcerer Posts: 27,532 Karma: 193191846 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Thanks, Kevin. I've not looked at that code either.

10-29-2017, 03:44 PM	#8
KevinH Sigil Developer Posts: 7,602 Karma: 5433388 Join Date: Nov 2009 Device: many	Just pushed to master the fix for this. Index terms inside em tags are now properly marked and used during Index generation.

10-29-2017, 04:51 PM	#9
Turtle91 A Hairy Wizard Posts: 3,093 Karma: 18727053 Join Date: Dec 2012 Location: Charleston, SC today Device: iPhone 11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire	You guys are awesome! Do you have a best guestimate when the next official release will be posted?

10-30-2017, 09:53 AM	#11
1v4n0 Groupie Posts: 171 Karma: 40000 Join Date: Oct 2013 Device: kindle	Thank you. That was fast