I will wait for Chris as the other major user of the script to say what he thinks, then I commit. But I agree. I changed my own script and it works better.
I am still a bit annoyed with the position of the section head - it appears to be always in the verse prior. That is weird to me. Peter Daniel Owens wrote: > From my perspective (based on the USFM 2.1 reference document), all \s# > tags should be handled in such a way that a new <div> element is > created, with higher numbers being more deeply nested. If you feel > comfortable changing the code, please do so. It seems to me it's a bug > in usfm2osis.pl. > > Daniel > > Peter von Kaehne wrote: >> Peter von Kaehne wrote: >> >>> Currently usfm2osis.pl appears to mishandle minor section headings: >>> >>> _______________________________________ >>> >> Looking at the source code it appears there is a specific difference in >> the handling of \s2 vs \s1 and \s3, \s4 etc.. See here: >> >> >> # \s \s1 section (From Chapters and Verses) >> if ($line =~ /^\\s1?\b\s*(.+)/) { >> push (@outdata, closeTag("<\/p>")); >> push (@outdata, closeTag("<\/div type=\"section\">")); >> push (@outdata, "<div type=\"section\">\n"); >> openTag("<\/div type=\"section\">"); >> $line =~ s/\\s1?\b\s*(.+)/<title>$1<\/title>/; >> if ($line =~ /HEBREW TITLE/) { >> $line =~ s/<title>/<title type=\"psalm\">/; >> } >> } >> >> # \ss \s2 subSection (From Chapters and Verses) >> if ($line =~ /^\\s[s2]\b\s*(.+)/) { >> $line =~ s/\\s[s2]\b\s*(.+)/<title>$1<\/title>/; >> } >> >> # \sss \s3 x-subsubSection (From Chapters and Verses) >> if ($line =~ /^\\s(ss|3)\b\s*(.+)/) { >> push (@outdata, closeTag("<\/p>")); >> push (@outdata, closeTag("<\/div >> >> type=\"x=subSubSection\">")); >> push (@outdata, "<div type=\"x-subSubSection\">\n"); >> openTag("<\/div type=\"x-subSubSection\">"); >> $line =~ s/\\s(ss|3)\b\s*(.+)/<title>$2<\/title>/; >> >> >> \s2 is not creating a <div>. Is there a specific reason? >> >> Peter >> >> _______________________________________________ >> sword-devel mailing list: sword-devel@crosswire.org >> http://www.crosswire.org/mailman/listinfo/sword-devel >> Instructions to unsubscribe/change your settings at above page >> >> > > _______________________________________________ > sword-devel mailing list: sword-devel@crosswire.org > http://www.crosswire.org/mailman/listinfo/sword-devel > Instructions to unsubscribe/change your settings at above page _______________________________________________ sword-devel mailing list: sword-devel@crosswire.org http://www.crosswire.org/mailman/listinfo/sword-devel Instructions to unsubscribe/change your settings at above page