X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Foutput_xhtml.cpp;h=080369f104e935535a2be7e6a548416584717b29;hb=c7d29be153debac82e3d2e8865fcc849f0a5f40d;hp=adf468dba9bb0546613be5baa58c1deb0addbc49;hpb=ff782f993a38f5fb8b2bb563464a0d59a9b66d79;p=lyx.git diff --git a/src/output_xhtml.cpp b/src/output_xhtml.cpp index adf468dba9..080369f104 100644 --- a/src/output_xhtml.cpp +++ b/src/output_xhtml.cpp @@ -4,7 +4,7 @@ * Licence details can be found in the file COPYING. * * \author Richard Heck - * + * * This code is based upon output_docbook.cpp * * Full author contact details are available in file CREDITS. @@ -36,6 +36,9 @@ #include +// Uncomment to activate debugging code. +// #define XHTML_DEBUG + using namespace std; using namespace lyx::support; @@ -71,7 +74,8 @@ docstring escapeChar(char_type c, XHTMLStream::EscapeSettings e) // escape what needs escaping -docstring htmlize(docstring const & str, XHTMLStream::EscapeSettings e) { +docstring htmlize(docstring const & str, XHTMLStream::EscapeSettings e) +{ odocstringstream d; docstring::const_iterator it = str.begin(); docstring::const_iterator en = str.end(); @@ -81,52 +85,10 @@ docstring htmlize(docstring const & str, XHTMLStream::EscapeSettings e) { } -string escapeChar(char c, XHTMLStream::EscapeSettings e) -{ - string str; - switch (e) { - case XHTMLStream::ESCAPE_NONE: - str += c; - break; - case XHTMLStream::ESCAPE_ALL: - if (c == '<') { - str += "<"; - break; - } else if (c == '>') { - str += ">"; - break; - } - // fall through - case XHTMLStream::ESCAPE_AND: - if (c == '&') - str += "&"; - else - str +=c ; - break; - } - return str; -} - - -// escape what needs escaping -string htmlize(string const & str, XHTMLStream::EscapeSettings e) { - ostringstream d; - string::const_iterator it = str.begin(); - string::const_iterator en = str.end(); - for (; it != en; ++it) - d << escapeChar(*it, e); - return d.str(); -} - - -string cleanAttr(string const & str) +docstring escapeChar(char c, XHTMLStream::EscapeSettings e) { - string newname; - string::const_iterator it = str.begin(); - string::const_iterator en = str.end(); - for (; it != en; ++it) - newname += isalnum(*it) ? *it : '_'; - return newname; + LATTEST(static_cast(c) < 0x80); + return escapeChar(static_cast(c), e); } @@ -139,49 +101,182 @@ docstring cleanAttr(docstring const & str) char_type const c = *it; newname += isAlnumASCII(c) ? c : char_type('_'); } - return newname; + return newname; } -bool isFontTag(string const & s) +docstring StartTag::writeTag() const { - return s == "em" || s == "strong"; // others? + docstring output = '<' + from_utf8(tag_); + if (!attr_.empty()) + output += ' ' + html::htmlize(from_utf8(attr_), XHTMLStream::ESCAPE_NONE); + output += ">"; + return output; } -docstring StartTag::asTag() const +docstring StartTag::writeEndTag() const { - string output = "<" + tag_; - if (!attr_.empty()) - output += " " + html::htmlize(attr_, XHTMLStream::ESCAPE_NONE); - output += ">"; + string output = ""; return from_utf8(output); } -docstring StartTag::asEndTag() const +bool StartTag::operator==(FontTag const & rhs) const { - string output = ""; - return from_utf8(output); + return rhs == *this; } -docstring EndTag::asEndTag() const +docstring EndTag::writeEndTag() const { string output = ""; return from_utf8(output); } -docstring CompTag::asTag() const +ParTag::ParTag(std::string const & tag, std::string attr, + std::string const & parid) + : StartTag(tag) +{ + if (!parid.empty()) + attr += " id='" + parid + "'"; + attr_ = attr; +} + + +docstring CompTag::writeTag() const { - string output = "<" + tag_; + docstring output = '<' + from_utf8(tag_); if (!attr_.empty()) - output += " " + html::htmlize(attr_, XHTMLStream::ESCAPE_NONE); + output += ' ' + html::htmlize(from_utf8(attr_), XHTMLStream::ESCAPE_NONE); output += " />"; - return from_utf8(output); + return output; } + + +namespace { + +string fontToTag(html::FontTypes type) +{ + switch(type) { + case FT_EMPH: + return "em"; + case FT_BOLD: + return "b"; + case FT_NOUN: + return "dfn"; + case FT_UBAR: + case FT_WAVE: + case FT_DBAR: + return "u"; + case FT_SOUT: + case FT_XOUT: + return "del"; + case FT_ITALIC: + return "i"; + case FT_UPRIGHT: + case FT_SLANTED: + case FT_SMALLCAPS: + case FT_ROMAN: + case FT_SANS: + case FT_TYPE: + case FT_SIZE_TINY: + case FT_SIZE_SCRIPT: + case FT_SIZE_FOOTNOTE: + case FT_SIZE_SMALL: + case FT_SIZE_NORMAL: + case FT_SIZE_LARGE: + case FT_SIZE_LARGER: + case FT_SIZE_LARGEST: + case FT_SIZE_HUGE: + case FT_SIZE_HUGER: + case FT_SIZE_INCREASE: + case FT_SIZE_DECREASE: + return "span"; + } + // kill warning + return ""; +} + +string fontToAttribute(html::FontTypes type) +{ + switch(type) { + case FT_EMPH: + case FT_BOLD: + return ""; + case FT_NOUN: + return "class='lyxnoun'"; + case FT_UBAR: + return ""; + case FT_DBAR: + return "class='dline'"; + case FT_XOUT: + case FT_SOUT: + return "class='strikeout'"; + case FT_WAVE: + return "class='wline'"; + case FT_ITALIC: + return ""; + case FT_UPRIGHT: + return "style='font-style:normal;'"; + case FT_SLANTED: + return "style='font-style:oblique;'"; + case FT_SMALLCAPS: + return "style='font-variant:small-caps;'"; + case FT_ROMAN: + return "style='font-family:serif;'"; + case FT_SANS: + return "style='font-family:sans-serif;'"; + case FT_TYPE: + return "style='font-family:monospace;'"; + case FT_SIZE_TINY: + case FT_SIZE_SCRIPT: + case FT_SIZE_FOOTNOTE: + return "style='font-size:x-small;'"; + case FT_SIZE_SMALL: + return "style='font-size:small;'"; + case FT_SIZE_NORMAL: + return "style='font-size:normal;'"; + case FT_SIZE_LARGE: + return "style='font-size:large;'"; + case FT_SIZE_LARGER: + case FT_SIZE_LARGEST: + return "style='font-size:x-large;'"; + case FT_SIZE_HUGE: + case FT_SIZE_HUGER: + return "style='font-size:xx-large;'"; + case FT_SIZE_INCREASE: + return "style='font-size:larger;'"; + case FT_SIZE_DECREASE: + return "style='font-size:smaller;'"; + } + // kill warning + return ""; +} + +} // end anonymous namespace + + +FontTag::FontTag(FontTypes type) + : StartTag(fontToTag(type), fontToAttribute(type)), font_type_(type) +{} + + +bool FontTag::operator==(StartTag const & tag) const +{ + FontTag const * const ftag = tag.asFontTag(); + if (!ftag) + return false; + return (font_type_ == ftag->font_type_); +} + + +EndFontTag::EndFontTag(FontTypes type) + : EndTag(fontToTag(type)), font_type_(type) +{} + } // namespace html @@ -192,56 +287,159 @@ docstring CompTag::asTag() const /// //////////////////////////////////////////////////////////////// -XHTMLStream::XHTMLStream(odocstream & os) - : os_(os), escape_(ESCAPE_ALL) +XHTMLStream::XHTMLStream(odocstream & os) + : os_(os), escape_(ESCAPE_ALL) {} -void XHTMLStream::writeError(std::string const & s) +#ifdef XHTML_DEBUG +void XHTMLStream::dumpTagStack(string const & msg) +{ + *this << html::CR(); + writeError(msg); + *this << html::CR(); + writeError("Tag Stack"); + TagDeque::const_reverse_iterator it = tag_stack_.rbegin(); + TagDeque::const_reverse_iterator en = tag_stack_.rend(); + for (; it != en; ++it) { + writeError(it->get()->tag_); + } + writeError("End Tag Stack"); + *this << html::CR(); + writeError("Pending Tags"); + it = pending_tags_.rbegin(); + en = pending_tags_.rend(); + for (; it != en; ++it) { + writeError(it->get()->tag_); + } + writeError("End Pending Tags"); + *this << html::CR(); +} +#endif + + +void XHTMLStream::writeError(std::string const & s) const { LYXERR0(s); - os_ << from_utf8(""); + os_ << from_utf8("\n"); } +namespace { + // an illegal tag for internal use + static html::StartTag const parsep_tag("&LyX_parsep_tag&"); +} // namespace + + bool XHTMLStream::closeFontTags() { + if (isTagPending(parsep_tag)) + // we haven't had any content + return true; + +#ifdef XHTML_DEBUG + dumpTagStack("Beging Close Font Tags"); +#endif + + // this may be a useless check, since we ought at least to have + // the parsep_tag. but it can't hurt too much to be careful. if (tag_stack_.empty()) return true; + // first, we close any open font tags we can close - html::StartTag curtag = tag_stack_.back(); - while (html::isFontTag(curtag.tag_)) { - os_ << curtag.asEndTag(); + TagPtr curtag = tag_stack_.back(); + while (curtag->asFontTag()) { + os_ << curtag->writeEndTag(); tag_stack_.pop_back(); - if (tag_stack_.empty()) - // this probably shouldn't happen, since then the - // font tags weren't in any other tag. but that - // problem will likely be caught elsewhere. - return true; + // this shouldn't happen, since then the font tags + // weren't in any other tag. + LASSERT(!tag_stack_.empty(), return true); curtag = tag_stack_.back(); } - // so we've hit a non-font tag. let's see if any of the - // remaining tags are font tags. - TagStack::const_iterator it = tag_stack_.begin(); - TagStack::const_iterator en = tag_stack_.end(); - bool noFontTags = true; + +#ifdef XHTML_DEBUG + dumpTagStack("End Close Font Tags"); +#endif + + if (*curtag == parsep_tag) + return true; + + // so we've hit a non-font tag. + writeError("Tags still open in closeFontTags(). Probably not a problem,\n" + "but you might want to check these tags:"); + TagDeque::const_reverse_iterator it = tag_stack_.rbegin(); + TagDeque::const_reverse_iterator const en = tag_stack_.rend(); for (; it != en; ++it) { - if (html::isFontTag(it->tag_)) { - writeError("Font tag `" + it->tag_ + "' still open in closeFontTags().\n" - "This is likely not a problem, but you might want to check."); - noFontTags = false; + if (**it == parsep_tag) + break; + writeError((*it)->tag_); + } + return false; +} + + +void XHTMLStream::startDivision(bool keep_empty) +{ + pending_tags_.push_back(makeTagPtr(html::StartTag(parsep_tag))); + if (keep_empty) + clearTagDeque(); +#ifdef XHTML_DEBUG + dumpTagStack("StartDivision"); +#endif +} + + +void XHTMLStream::endDivision() +{ + if (isTagPending(parsep_tag)) { + // this case is normal. it just means we didn't have content, + // so the parsep_tag never got moved onto the tag stack. + while (!pending_tags_.empty()) { + // clear all pending tags up to and including the parsep tag. + // note that we work from the back, because we want to get rid + // of everything that hasn't been used. + TagPtr const cur_tag = pending_tags_.back(); + pending_tags_.pop_back(); + if (*cur_tag == parsep_tag) + break; } + +#ifdef XHTML_DEBUG + dumpTagStack("EndDivision"); +#endif + + return; + } + + if (!isTagOpen(parsep_tag)) { + writeError("No division separation tag found in endDivision()."); + return; } - return noFontTags; + + // this case is also normal, if the parsep tag is the last one + // on the stack. otherwise, it's an error. + while (!tag_stack_.empty()) { + TagPtr const cur_tag = tag_stack_.back(); + tag_stack_.pop_back(); + if (*cur_tag == parsep_tag) + break; + writeError("Tag `" + cur_tag->tag_ + "' still open at end of paragraph. Closing."); + os_ << cur_tag->writeEndTag(); + } + +#ifdef XHTML_DEBUG + dumpTagStack("EndDivision"); +#endif } void XHTMLStream::clearTagDeque() { while (!pending_tags_.empty()) { - html::StartTag const & tag = pending_tags_.front(); - // tabs? - os_ << tag.asTag(); + TagPtr const tag = pending_tags_.front(); + if (*tag != parsep_tag) + // tabs? + os_ << tag->writeTag(); tag_stack_.push_back(tag); pending_tags_.pop_front(); } @@ -279,7 +477,7 @@ XHTMLStream & XHTMLStream::operator<<(char_type c) XHTMLStream & XHTMLStream::operator<<(char c) { clearTagDeque(); - string const d = html::escapeChar(c, escape_); + os_ << html::escapeChar(c, escape_); escape_ = ESCAPE_ALL; return *this; } @@ -295,35 +493,52 @@ XHTMLStream & XHTMLStream::operator<<(int i) XHTMLStream & XHTMLStream::operator<<(EscapeSettings e) -{ +{ escape_ = e; return *this; } -XHTMLStream & XHTMLStream::operator<<(html::StartTag const & tag) +XHTMLStream & XHTMLStream::operator<<(html::StartTag const & tag) { if (tag.tag_.empty()) return *this; - pending_tags_.push_back(tag); + pending_tags_.push_back(makeTagPtr(tag)); if (tag.keepempty_) clearTagDeque(); return *this; } -XHTMLStream & XHTMLStream::operator<<(html::CompTag const & tag) +XHTMLStream & XHTMLStream::operator<<(html::ParTag const & tag) +{ + if (tag.tag_.empty()) + return *this; + pending_tags_.push_back(makeTagPtr(tag)); + return *this; +} + + +XHTMLStream & XHTMLStream::operator<<(html::CompTag const & tag) { if (tag.tag_.empty()) return *this; clearTagDeque(); - // tabs? - os_ << tag.asTag(); + os_ << tag.writeTag(); *this << html::CR(); return *this; } +XHTMLStream & XHTMLStream::operator<<(html::FontTag const & tag) +{ + if (tag.tag_.empty()) + return *this; + pending_tags_.push_back(makeTagPtr(tag)); + return *this; +} + + XHTMLStream & XHTMLStream::operator<<(html::CR const &) { // tabs? @@ -332,19 +547,41 @@ XHTMLStream & XHTMLStream::operator<<(html::CR const &) } -bool XHTMLStream::isTagOpen(string const & stag) +bool XHTMLStream::isTagOpen(html::StartTag const & stag) const +{ + TagDeque::const_iterator sit = tag_stack_.begin(); + TagDeque::const_iterator const sen = tag_stack_.end(); + for (; sit != sen; ++sit) + if (**sit == stag) + return true; + return false; +} + + +bool XHTMLStream::isTagOpen(html::EndTag const & etag) const { - TagStack::const_iterator sit = tag_stack_.begin(); - TagStack::const_iterator const sen = tag_stack_.end(); + TagDeque::const_iterator sit = tag_stack_.begin(); + TagDeque::const_iterator const sen = tag_stack_.end(); for (; sit != sen; ++sit) - if (sit->tag_ == stag) + if (etag == **sit) + return true; + return false; +} + + +bool XHTMLStream::isTagPending(html::StartTag const & stag) const +{ + TagDeque::const_iterator sit = pending_tags_.begin(); + TagDeque::const_iterator const sen = pending_tags_.end(); + for (; sit != sen; ++sit) + if (**sit == stag) return true; return false; } // this is complicated, because we want to make sure that -// everything is properly nested. the code ought to make +// everything is properly nested. the code ought to make // sure of that, but we won't assert (yet) if we run into // a problem. we'll just output error messages and try our // best to make things work. @@ -353,109 +590,111 @@ XHTMLStream & XHTMLStream::operator<<(html::EndTag const & etag) if (etag.tag_.empty()) return *this; - // make sure there are tags to be closed - if (tag_stack_.empty()) { - writeError("Tried to close `" + etag.tag_ - + "' when no tags were open!"); - return *this; - } - - // first make sure we're not closing an empty tag + // if this tag is pending, we can simply discard it. if (!pending_tags_.empty()) { - html::StartTag const & stag = pending_tags_.back(); - if (etag.tag_ == stag.tag_) { - // we have , so we discard it and remove it + + if (etag == *pending_tags_.back()) { + // we have , so we discard it and remove it // from the pending_tags_. pending_tags_.pop_back(); return *this; } + // there is a pending tag that isn't the one we are trying - // to close. + // to close. + // is this tag itself pending? // non-const iterators because we may call erase(). - TagStack::iterator dit = pending_tags_.begin(); - TagStack::iterator const den = pending_tags_.end(); + TagDeque::iterator dit = pending_tags_.begin(); + TagDeque::iterator const den = pending_tags_.end(); for (; dit != den; ++dit) { - if (dit->tag_ == etag.tag_) { + if (etag == **dit) { // it was pending, so we just erase it - writeError("Tried to close pending tag `" + etag.tag_ + writeError("Tried to close pending tag `" + etag.tag_ + "' when other tags were pending. Last pending tag is `" - + pending_tags_.back().tag_ + "'. Tag discarded."); + + to_utf8(pending_tags_.back()->writeTag()) + + "'. Tag discarded."); pending_tags_.erase(dit); return *this; } } // so etag isn't itself pending. is it even open? - if (!isTagOpen(etag.tag_)) { - writeError("Tried to close `" + etag.tag_ + if (!isTagOpen(etag)) { + writeError("Tried to close `" + etag.tag_ + "' when tag was not open. Tag discarded."); return *this; } // ok, so etag is open. - // our strategy will be as below: we will do what we need to + // our strategy will be as below: we will do what we need to // do to close this tag. - string estr = "Closing tag `" + etag.tag_ + string estr = "Closing tag `" + etag.tag_ + "' when other tags are pending. Discarded pending tags:\n"; for (dit = pending_tags_.begin(); dit != den; ++dit) - estr += dit->tag_ + "\n"; + estr += to_utf8(html::htmlize((*dit)->writeTag(), XHTMLStream::ESCAPE_ALL)) + "\n"; writeError(estr); // clear the pending tags... pending_tags_.clear(); // ...and then just fall through. } + // make sure there are tags to be closed + if (tag_stack_.empty()) { + writeError("Tried to close `" + etag.tag_ + + "' when no tags were open!"); + return *this; + } + // is the tag we are closing the last one we opened? - if (etag.tag_ == tag_stack_.back().tag_) { + if (etag == *tag_stack_.back()) { // output it... - os_ << etag.asEndTag(); + os_ << etag.writeEndTag(); // ...and forget about it tag_stack_.pop_back(); return *this; - } - - // we are trying to close a tag other than the one last opened. + } + + // we are trying to close a tag other than the one last opened. // let's first see if this particular tag is still open somehow. - if (!isTagOpen(etag.tag_)) { - writeError("Tried to close `" + etag.tag_ + if (!isTagOpen(etag)) { + writeError("Tried to close `" + etag.tag_ + "' when tag was not open. Tag discarded."); return *this; } - + // so the tag was opened, but other tags have been opened since // and not yet closed. // if it's a font tag, though... - if (html::isFontTag(etag.tag_)) { + if (etag.asFontTag()) { // it won't be a problem if the other tags open since this one // are also font tags. - TagStack::const_reverse_iterator rit = tag_stack_.rbegin(); - TagStack::const_reverse_iterator ren = tag_stack_.rend(); + TagDeque::const_reverse_iterator rit = tag_stack_.rbegin(); + TagDeque::const_reverse_iterator ren = tag_stack_.rend(); for (; rit != ren; ++rit) { - if (rit->tag_ == etag.tag_) + if (etag == **rit) break; - if (!html::isFontTag(rit->tag_)) { + if (!(*rit)->asFontTag()) { // we'll just leave it and, presumably, have to close it later. - writeError("Unable to close font tag `" + etag.tag_ - + "' due to open non-font tag `" + rit->tag_ + "'."); + writeError("Unable to close font tag `" + etag.tag_ + + "' due to open non-font tag `" + (*rit)->tag_ + "'."); return *this; } } - + // so we have e.g.: // this is bold // and are being asked to closed em. we want: // this is bold // first, we close the intervening tags... - html::StartTag curtag = tag_stack_.back(); + TagPtr curtag = tag_stack_.back(); // ...remembering them in a stack. - TagStack fontstack; - while (curtag.tag_ != etag.tag_) { - os_ << curtag.asEndTag(); + TagDeque fontstack; + while (etag != *curtag) { + os_ << curtag->writeEndTag(); fontstack.push_back(curtag); tag_stack_.pop_back(); curtag = tag_stack_.back(); } - // now close our tag... - os_ << etag.asEndTag(); + os_ << etag.writeEndTag(); tag_stack_.pop_back(); // ...and restore the other tags. @@ -465,50 +704,53 @@ XHTMLStream & XHTMLStream::operator<<(html::EndTag const & etag) pending_tags_.push_back(*rit); return *this; } - + // it wasn't a font tag. - // so other tags were opened before this one and not properly closed. - // so we'll close them, too. that may cause other issues later, but it + // so other tags were opened before this one and not properly closed. + // so we'll close them, too. that may cause other issues later, but it // at least guarantees proper nesting. - writeError("Closing tag `" + etag.tag_ + writeError("Closing tag `" + etag.tag_ + "' when other tags are open, namely:"); - html::StartTag curtag = tag_stack_.back(); - while (curtag.tag_ != etag.tag_) { - writeError(curtag.tag_); - os_ << curtag.asEndTag(); + TagPtr curtag = tag_stack_.back(); + while (etag != *curtag) { + writeError(curtag->tag_); + if (*curtag != parsep_tag) + os_ << curtag->writeEndTag(); tag_stack_.pop_back(); curtag = tag_stack_.back(); } // curtag is now the one we actually want. - os_ << curtag.asEndTag(); + os_ << curtag->writeEndTag(); tag_stack_.pop_back(); - + return *this; } // End code for XHTMLStream namespace { - + // convenience functions -inline void openTag(XHTMLStream & xs, Layout const & lay) +inline void openParTag(XHTMLStream & xs, Layout const & lay, + std::string parlabel) { - xs << html::StartTag(lay.htmltag(), lay.htmlattr()); + xs << html::ParTag(lay.htmltag(), lay.htmlattr(), parlabel); } -void openTag(XHTMLStream & xs, Layout const & lay, - ParagraphParameters const & params) +void openParTag(XHTMLStream & xs, Layout const & lay, + ParagraphParameters const & params, + std::string parlabel) { // FIXME Are there other things we should handle here? string const align = alignmentToCSS(params.align()); if (align.empty()) { - openTag(xs, lay); + openParTag(xs, lay, parlabel); return; } string attrs = lay.htmlattr() + " style='text-align: " + align + ";'"; - xs << html::StartTag(lay.htmltag(), attrs); + xs << html::ParTag(lay.htmltag(), attrs, parlabel); } @@ -536,7 +778,7 @@ inline void openItemTag(XHTMLStream & xs, Layout const & lay) } -void openItemTag(XHTMLStream & xs, Layout const & lay, +void openItemTag(XHTMLStream & xs, Layout const & lay, ParagraphParameters const & params) { // FIXME Are there other things we should handle here? @@ -557,7 +799,7 @@ inline void closeItemTag(XHTMLStream & xs, Layout const & lay) // end of convenience functions -ParagraphList::const_iterator searchParagraphHtml( +ParagraphList::const_iterator findLastParagraph( ParagraphList::const_iterator p, ParagraphList::const_iterator const & pend) { @@ -568,8 +810,8 @@ ParagraphList::const_iterator searchParagraphHtml( } -ParagraphList::const_iterator searchEnvironmentHtml( - ParagraphList::const_iterator const pstart, +ParagraphList::const_iterator findEndOfEnvironment( + ParagraphList::const_iterator const & pstart, ParagraphList::const_iterator const & pend) { ParagraphList::const_iterator p = pstart; @@ -582,15 +824,19 @@ ParagraphList::const_iterator searchEnvironmentHtml( // it can happen. We pretend that it's just at lowest depth. if (style.latextype == LATEX_COMMAND) return p; + // If depth is down, we're done if (p->params().depth() < depth) return p; + // If depth is up, we're not done if (p->params().depth() > depth) continue; - // Now we know we are at the same depth - if (style.latextype == LATEX_PARAGRAPH - || style.latexname() != bstyle.latexname()) + + // FIXME I am not sure about the first check. + // Surely we *could* have different layouts that count as + // LATEX_PARAGRAPH, right? + if (style.latextype == LATEX_PARAGRAPH || style != bstyle) return p; } return pend; @@ -609,37 +855,72 @@ ParagraphList::const_iterator makeParagraphs(Buffer const & buf, for (; par != pend; ++par) { Layout const & lay = par->layout(); if (!lay.counter.empty()) - buf.params().documentClass().counters().step(lay.counter, OutputUpdate); + buf.masterBuffer()->params(). + documentClass().counters().step(lay.counter, OutputUpdate); + // FIXME We should see if there's a label to be output and // do something with it. if (par != pbegin) xs << html::CR(); - // If we are already in a paragraph, and this is the first one, then we - // do not want to open the paragraph tag. - // we also do not want to open it if the current layout does not permit - // multiple paragraphs. - bool const opened = runparams.html_make_pars && - (par != pbegin || !runparams.html_in_par); - if (opened) - openTag(xs, lay, par->params()); - docstring const deferred = - par->simpleLyXHTMLOnePar(buf, xs, runparams, text.outerFont(distance(begin, par))); + // We want to open the paragraph tag if: + // (i) the current layout permits multiple paragraphs + // (ii) we are either not already inside a paragraph (HTMLIsBlock) OR + // we are, but this is not the first paragraph + // + // But there is also a special case, and we first see whether we are in it. + // We do not want to open the paragraph tag if this paragraph contains + // only one item, and that item is "inline", i.e., not HTMLIsBlock (such + // as a branch). On the other hand, if that single item has a font change + // applied to it, then we still do need to open the paragraph. + // + // Obviously, this is very fragile. The main reason we need to do this is + // because of branches, e.g., a branch that contains an entire new section. + // We do not really want to wrap that whole thing in a
...
. + bool special_case = false; + Inset const * specinset = par->size() == 1 ? par->getInset(0) : 0; + if (specinset && !specinset->getLayout().htmlisblock()) { + Layout const & style = par->layout(); + FontInfo const first_font = style.labeltype == LABEL_MANUAL ? + style.labelfont : style.font; + FontInfo const our_font = + par->getFont(buf.masterBuffer()->params(), 0, + text.outerFont(distance(begin, par))).fontInfo(); + if (first_font == our_font) + special_case = true; + } + + bool const open_par = runparams.html_make_pars + && (!runparams.html_in_par || par != pbegin) + && !special_case; // We want to issue the closing tag if either: // (i) We opened it, and either html_in_par is false, // or we're not in the last paragraph, anyway. - // (ii) We didn't open it and html_in_par is true, + // (ii) We didn't open it and html_in_par is true, // but we are in the first par, and there is a next par. ParagraphList::const_iterator nextpar = par; - nextpar++; - bool const needclose = - (opened && (!runparams.html_in_par || nextpar != pend)) - || (!opened && runparams.html_in_par && par == pbegin && nextpar != pend); - if (needclose) { + ++nextpar; + bool const close_par = + (open_par && (!runparams.html_in_par || nextpar != pend)) + || (!open_par && runparams.html_in_par && par == pbegin && nextpar != pend); + + if (open_par) { + // We do not issue the paragraph id if we are doing + // this for the TOC (or some similar purpose) + openParTag(xs, lay, par->params(), + runparams.for_toc ? "" : par->magicLabel()); + } + + docstring const deferred = par->simpleLyXHTMLOnePar(buf, xs, + runparams, text.outerFont(distance(begin, par)), + open_par, close_par); + + if (close_par) { closeTag(xs, lay); xs << html::CR(); } + if (!deferred.empty()) { xs << XHTMLStream::ESCAPE_NONE << deferred << html::CR(); } @@ -653,7 +934,7 @@ ParagraphList::const_iterator makeBibliography(Buffer const & buf, OutputParams const & runparams, Text const & text, ParagraphList::const_iterator const & pbegin, - ParagraphList::const_iterator const & pend) + ParagraphList::const_iterator const & pend) { // FIXME XHTML // Use TextClass::htmlTOCLayout() to figure out how we should look. @@ -675,13 +956,13 @@ bool isNormalEnv(Layout const & lay) || lay.latextype == LATEX_BIB_ENVIRONMENT; } - -ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, + +ParagraphList::const_iterator makeEnvironment(Buffer const & buf, XHTMLStream & xs, OutputParams const & runparams, Text const & text, ParagraphList::const_iterator const & pbegin, - ParagraphList::const_iterator const & pend) + ParagraphList::const_iterator const & pend) { ParagraphList::const_iterator const begin = text.paragraphs().begin(); ParagraphList::const_iterator par = pbegin; @@ -689,7 +970,7 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, depth_type const origdepth = pbegin->params().depth(); // open tag for this environment - openTag(xs, bstyle); + openParTag(xs, bstyle, pbegin->magicLabel()); xs << html::CR(); // we will on occasion need to remember a layout from before. @@ -703,44 +984,44 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, // FIXME There may be a bug here about user defined enumeration // types. If so, then we'll need to take the counter and add "i", // "ii", etc, as with enum. - Counters & cnts = buf.params().documentClass().counters(); + Counters & cnts = buf.masterBuffer()->params().documentClass().counters(); docstring const & cntr = style.counter; - if (!style.counter.empty() - && (par == pbegin || !isNormalEnv(style)) + if (!style.counter.empty() + && (par == pbegin || !isNormalEnv(style)) && cnts.hasCounter(cntr) ) cnts.step(cntr, OutputUpdate); ParagraphList::const_iterator send; - // this will be positive, if we want to skip the initial word - // (if it's been taken for the label). - pos_type sep = 0; switch (style.latextype) { case LATEX_ENVIRONMENT: case LATEX_LIST_ENVIRONMENT: case LATEX_ITEM_ENVIRONMENT: { - // There are two possiblities in this case. - // One is that we are still in the environment in which we + // There are two possiblities in this case. + // One is that we are still in the environment in which we // started---which we will be if the depth is the same. if (par->params().depth() == origdepth) { - LASSERT(bstyle == style, /* */); + LATTEST(bstyle == style); if (lastlay != 0) { closeItemTag(xs, *lastlay); lastlay = 0; } - + + // this will be positive, if we want to skip the + // initial word (if it's been taken for the label). + pos_type sep = 0; bool const labelfirst = style.htmllabelfirst(); if (!labelfirst) openItemTag(xs, style, par->params()); - + // label output - if (style.labeltype != LABEL_NO_LABEL && + if (style.labeltype != LABEL_NO_LABEL && style.htmllabeltag() != "NONE") { if (isNormalEnv(style)) { - // in this case, we print the label only for the first + // in this case, we print the label only for the first // paragraph (as in a theorem). if (par == pbegin) { - docstring const lbl = + docstring const lbl = pbegin->params().labelString(); if (!lbl.empty()) { openLabelTag(xs, style); @@ -749,7 +1030,7 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, } xs << html::CR(); } - } else { // some kind of list + } else { // some kind of list if (style.labeltype == LABEL_MANUAL) { openLabelTag(xs, style); sep = par->firstWordLyXHTML(xs, runparams); @@ -768,13 +1049,14 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, if (labelfirst) openItemTag(xs, style, par->params()); - par->simpleLyXHTMLOnePar(buf, xs, runparams, - text.outerFont(distance(begin, par)), sep); + docstring deferred = par->simpleLyXHTMLOnePar(buf, xs, runparams, + text.outerFont(distance(begin, par)), true, true, sep); + xs << XHTMLStream::ESCAPE_NONE << deferred; ++par; // We may not want to close the tag yet, in particular: // If we're not at the end... - if (par != pend + if (par != pend // and are doing items... && !isNormalEnv(style) // and if the depth has changed... @@ -789,13 +1071,13 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, // The other possibility is that the depth has increased, in which // case we need to recurse. else { - send = searchEnvironmentHtml(par, pend); - par = makeEnvironmentHtml(buf, xs, runparams, text, par, send); + send = findEndOfEnvironment(par, pend); + par = makeEnvironment(buf, xs, runparams, text, par, send); } break; } case LATEX_PARAGRAPH: - send = searchParagraphHtml(par, pend); + send = findLastParagraph(par, pend); par = makeParagraphs(buf, xs, runparams, text, par, send); break; // Shouldn't happen @@ -820,19 +1102,24 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, void makeCommand(Buffer const & buf, - XHTMLStream & xs, - OutputParams const & runparams, - Text const & text, - ParagraphList::const_iterator const & pbegin) + XHTMLStream & xs, + OutputParams const & runparams, + Text const & text, + ParagraphList::const_iterator const & pbegin) { Layout const & style = pbegin->layout(); if (!style.counter.empty()) - buf.params().documentClass().counters().step(style.counter, OutputUpdate); + buf.masterBuffer()->params(). + documentClass().counters().step(style.counter, OutputUpdate); - openTag(xs, style, pbegin->params()); + bool const make_parid = !runparams.for_toc && runparams.html_make_pars; + + openParTag(xs, style, pbegin->params(), + make_parid ? pbegin->magicLabel() : ""); // Label around sectioning number: // FIXME Probably need to account for LABEL_MANUAL + // FIXME Probably also need now to account for labels ABOVE and CENTERED. if (style.labeltype != LABEL_NO_LABEL) { openLabelTag(xs, style); xs << pbegin->params().labelString(); @@ -857,27 +1144,41 @@ void xhtmlParagraphs(Text const & text, OutputParams const & runparams) { ParagraphList const & paragraphs = text.paragraphs(); - ParagraphList::const_iterator par = paragraphs.begin(); - ParagraphList::const_iterator pend = paragraphs.end(); + if (runparams.par_begin == runparams.par_end) { + runparams.par_begin = 0; + runparams.par_end = paragraphs.size(); + } + pit_type bpit = runparams.par_begin; + pit_type const epit = runparams.par_end; + LASSERT(bpit < epit, + { xs << XHTMLStream::ESCAPE_NONE << "\n"; return; }); OutputParams ourparams = runparams; - while (par != pend) { + ParagraphList::const_iterator const pend = + (epit == (int) paragraphs.size()) ? + paragraphs.end() : paragraphs.constIterator(epit); + while (bpit < epit) { + ParagraphList::const_iterator par = paragraphs.constIterator(bpit); if (par->params().startOfAppendix()) { - // FIXME: only the counter corresponding to toplevel - // sectioning should be reset - Counters & cnts = buf.masterBuffer()->params().documentClass().counters(); - cnts.reset(); - cnts.appendix(true); + // We want to reset the counter corresponding to toplevel sectioning + Layout const & lay = + buf.masterBuffer()->params().documentClass().getTOCLayout(); + docstring const cnt = lay.counter; + if (!cnt.empty()) { + Counters & cnts = + buf.masterBuffer()->params().documentClass().counters(); + cnts.reset(cnt); + } } Layout const & style = par->layout(); - ParagraphList::const_iterator lastpar = par; + ParagraphList::const_iterator const lastpar = par; ParagraphList::const_iterator send; switch (style.latextype) { case LATEX_COMMAND: { // The files with which we are working never have more than // one paragraph in a command structure. - // FIXME + // FIXME // if (ourparams.html_in_par) // fix it so we don't get sections inside standard, e.g. // note that we may then need to make runparams not const, so we @@ -892,30 +1193,28 @@ void xhtmlParagraphs(Text const & text, case LATEX_LIST_ENVIRONMENT: case LATEX_ITEM_ENVIRONMENT: { // FIXME Same fix here. - send = searchEnvironmentHtml(par, pend); - par = makeEnvironmentHtml(buf, xs, ourparams, text, par, send); + send = findEndOfEnvironment(par, pend); + par = makeEnvironment(buf, xs, ourparams, text, par, send); break; } case LATEX_BIB_ENVIRONMENT: { // FIXME Same fix here. - send = searchEnvironmentHtml(par, pend); + send = findEndOfEnvironment(par, pend); par = makeBibliography(buf, xs, ourparams, text, par, send); break; } case LATEX_PARAGRAPH: - send = searchParagraphHtml(par, pend); + send = findLastParagraph(par, pend); par = makeParagraphs(buf, xs, ourparams, text, par, send); break; } - // FIXME?? - // makeEnvironment may process more than one paragraphs and bypass pend - if (distance(lastpar, par) >= distance(lastpar, pend)) - break; + bpit += distance(lastpar, par); } } -string alignmentToCSS(LyXAlignment align) { +string alignmentToCSS(LyXAlignment align) +{ switch (align) { case LYX_ALIGN_BLOCK: // we are NOT going to use text-align: justify!!