X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Foutput_xhtml.cpp;h=3a157c6f4d68c781da5b7fe65c35a39351c27c71;hb=5f1bfdde1e1eb61987f178f643a08017dc3803f7;hp=1232e79a841451345207b27726b6ff8dc0ea93f7;hpb=6967440797893231c18a740a233a912292879f49;p=lyx.git diff --git a/src/output_xhtml.cpp b/src/output_xhtml.cpp index 1232e79a84..3a157c6f4d 100644 --- a/src/output_xhtml.cpp +++ b/src/output_xhtml.cpp @@ -18,6 +18,7 @@ #include "buffer_funcs.h" #include "BufferParams.h" #include "Counters.h" +#include "Font.h" #include "Layout.h" #include "OutputParams.h" #include "Paragraph.h" @@ -27,9 +28,11 @@ #include "Text.h" #include "TextClass.h" -#include "support/lassert.h" +#include "support/convert.h" #include "support/debug.h" +#include "support/lassert.h" #include "support/lstrings.h" +#include "support/textutils.h" #include @@ -40,53 +43,118 @@ namespace lyx { namespace html { -docstring escapeChar(char_type c) +docstring escapeChar(char_type c, XHTMLStream::EscapeSettings e) { docstring str; - switch (c) { - case ' ': - str += " "; - break; - case '&': - str += "&"; - break; - case '<': - str += "<"; - break; - case '>': - str += ">"; - break; - default: + 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 -docstring htmlize(docstring const & str) { +docstring htmlize(docstring const & str, XHTMLStream::EscapeSettings e) { odocstringstream d; docstring::const_iterator it = str.begin(); docstring::const_iterator en = str.end(); for (; it != en; ++it) - d << escapeChar(*it); + d << escapeChar(*it, e); + return d.str(); +} + + +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) +{ + string newname; + string::const_iterator it = str.begin(); + string::const_iterator en = str.end(); + for (; it != en; ++it) + newname += isalnum(*it) ? *it : '_'; + return newname; +} + + +docstring cleanAttr(docstring const & str) +{ + docstring newname; + docstring::const_iterator it = str.begin(); + docstring::const_iterator en = str.end(); + for (; it != en; ++it) { + char_type const c = *it; + newname += isAlnumASCII(c) ? c : char_type('_'); + } + return newname; +} + + bool isFontTag(string const & s) { - return s == "em" || s == "strong"; // others? + // others? + return s == "em" || s == "strong" || s == "i" || s == "b"; } -} // namespace html docstring StartTag::asTag() const { string output = "<" + tag_; if (!attr_.empty()) - output += " " + attr_; + output += " " + html::htmlize(attr_, XHTMLStream::ESCAPE_NONE); output += ">"; return from_utf8(output); } @@ -110,11 +178,14 @@ docstring CompTag::asTag() const { string output = "<" + tag_; if (!attr_.empty()) - output += " " + attr_; + output += " " + html::htmlize(attr_, XHTMLStream::ESCAPE_NONE); output += " />"; return from_utf8(output); } +} // namespace html + + //////////////////////////////////////////////////////////////// /// @@ -123,21 +194,29 @@ docstring CompTag::asTag() const //////////////////////////////////////////////////////////////// XHTMLStream::XHTMLStream(odocstream & os) - :os_(os) + : os_(os), escape_(ESCAPE_ALL) {} -void XHTMLStream::cr() +void XHTMLStream::writeError(std::string const & s) { - // tabs? - os_ << from_ascii("\n"); + LYXERR0(s); + os_ << from_utf8(""); +} + + +namespace { + // an illegal tag for internal use + static string const parsep_tag = "&LyX_parsep_tag&"; } bool XHTMLStream::closeFontTags() { + if (tag_stack_.empty()) + return true; // first, we close any open font tags we can close - StartTag curtag = tag_stack_.back(); + html::StartTag curtag = tag_stack_.back(); while (html::isFontTag(curtag.tag_)) { os_ << curtag.asEndTag(); tag_stack_.pop_back(); @@ -148,27 +227,85 @@ bool XHTMLStream::closeFontTags() 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; + + if (curtag.tag_ == 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:"); + TagStack::const_reverse_iterator it = tag_stack_.rbegin(); + TagStack::const_reverse_iterator const en = tag_stack_.rend(); for (; it != en; ++it) { - if (html::isFontTag(it->tag_)) { - LYXERR0("Font tag `" << it->tag_ << "' still open in closeFontTags()."); - noFontTags = false; + string const tagname = it->tag_; + if (tagname == parsep_tag) + break; + writeError(it->tag_); + } + return false; +} + + +void XHTMLStream::startParagraph() +{ + pending_tags_.push_back(html::StartTag(parsep_tag)); +} + + +void XHTMLStream::endParagraph() +{ + if (!isTagOpen(parsep_tag)) { + // is it pending? + TagStack::const_iterator dit = pending_tags_.begin(); + TagStack::const_iterator const den = pending_tags_.end(); + bool found = false; + for (; dit != den; ++dit) { + if (dit->tag_ == parsep_tag) { + found = true; + break; + } + } + + if (!found) { + writeError("No paragraph separation tag found in endParagraph()."); + return; } + + // this case is normal. + 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 hasnt' been used. + html::StartTag const cur_tag = pending_tags_.back(); + string const & tag = cur_tag.tag_; + tag_stack_.pop_back(); + if (tag == parsep_tag) + break; + } + return; + } + + // 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()) { + html::StartTag const cur_tag = tag_stack_.back(); + string const & tag = cur_tag.tag_; + tag_stack_.pop_back(); + if (tag == parsep_tag) + break; + writeError("Tag `" + tag + "' still open at end of paragraph. Closing."); + os_ << cur_tag.asEndTag(); } - return noFontTags; } void XHTMLStream::clearTagDeque() { while (!pending_tags_.empty()) { - StartTag const & tag = pending_tags_.front(); - // tabs? - os_ << tag.asTag(); + html::StartTag const & tag = pending_tags_.front(); + if (tag.tag_ != parsep_tag) + // tabs? + os_ << tag.asTag(); tag_stack_.push_back(tag); pending_tags_.pop_front(); } @@ -178,7 +315,8 @@ void XHTMLStream::clearTagDeque() XHTMLStream & XHTMLStream::operator<<(docstring const & d) { clearTagDeque(); - os_ << html::htmlize(d); + os_ << html::htmlize(d, escape_); + escape_ = ESCAPE_ALL; return *this; } @@ -186,7 +324,9 @@ XHTMLStream & XHTMLStream::operator<<(docstring const & d) XHTMLStream & XHTMLStream::operator<<(const char * s) { clearTagDeque(); - os_ << html::htmlize(from_ascii(s)); + docstring const d = from_ascii(s); + os_ << html::htmlize(d, escape_); + escape_ = ESCAPE_ALL; return *this; } @@ -194,12 +334,38 @@ XHTMLStream & XHTMLStream::operator<<(const char * s) XHTMLStream & XHTMLStream::operator<<(char_type c) { clearTagDeque(); - os_ << html::escapeChar(c); + os_ << html::escapeChar(c, escape_); + escape_ = ESCAPE_ALL; + return *this; +} + + +XHTMLStream & XHTMLStream::operator<<(char c) +{ + clearTagDeque(); + string const d = html::escapeChar(c, escape_); + escape_ = ESCAPE_ALL; return *this; } -XHTMLStream & XHTMLStream::operator<<(StartTag const & tag) +XHTMLStream & XHTMLStream::operator<<(int i) +{ + clearTagDeque(); + os_ << i; + escape_ = ESCAPE_ALL; + return *this; +} + + +XHTMLStream & XHTMLStream::operator<<(EscapeSettings e) +{ + escape_ = e; + return *this; +} + + +XHTMLStream & XHTMLStream::operator<<(html::StartTag const & tag) { if (tag.tag_.empty()) return *this; @@ -210,23 +376,31 @@ XHTMLStream & XHTMLStream::operator<<(StartTag const & tag) } -XHTMLStream & XHTMLStream::operator<<(CompTag const & tag) +XHTMLStream & XHTMLStream::operator<<(html::CompTag const & tag) { if (tag.tag_.empty()) return *this; clearTagDeque(); // tabs? os_ << tag.asTag(); + *this << html::CR(); + return *this; +} + + +XHTMLStream & XHTMLStream::operator<<(html::CR const &) +{ + // tabs? + os_ << from_ascii("\n"); return *this; } -bool XHTMLStream::isTagOpen(string const & stag) +bool XHTMLStream::isTagOpen(string const & stag) { TagStack::const_iterator sit = tag_stack_.begin(); TagStack::const_iterator const sen = tag_stack_.end(); for (; sit != sen; ++sit) - // we could check for the if (sit->tag_ == stag) return true; return false; @@ -238,13 +412,21 @@ bool XHTMLStream::isTagOpen(string const & stag) // 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. -XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) +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 (!pending_tags_.empty()) { - StartTag const & stag = pending_tags_.back(); + html::StartTag const & stag = pending_tags_.back(); if (etag.tag_ == stag.tag_) { // we have , so we discard it and remove it // from the pending_tags_. @@ -255,30 +437,32 @@ XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) // to close. // is this tag itself pending? // non-const iterators because we may call erase(). - TagDeque::iterator dit = pending_tags_.begin(); - TagDeque::iterator const den = pending_tags_.end(); + TagStack::iterator dit = pending_tags_.begin(); + TagStack::iterator const den = pending_tags_.end(); for (; dit != den; ++dit) { if (dit->tag_ == etag.tag_) { // it was pending, so we just erase it - LYXERR0("Tried to close pending tag `" << etag.tag_ - << "' when other tags were pending. Tag discarded."); + writeError("Tried to close pending tag `" + etag.tag_ + + "' when other tags were pending. Last pending tag is `" + + pending_tags_.back().tag_ + "'. Tag discarded."); pending_tags_.erase(dit); return *this; } } // so etag isn't itself pending. is it even open? if (!isTagOpen(etag.tag_)) { - LYXERR0("Tried to close `" << etag.tag_ - << "' when tag was not open. Tag discarded."); + 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 // do to close this tag. - LYXERR0("Closing tag `" << etag.tag_ - << "' when other tags are pending. Discarded pending tags:"); + string estr = "Closing tag `" + etag.tag_ + + "' when other tags are pending. Discarded pending tags:\n"; for (dit = pending_tags_.begin(); dit != den; ++dit) - LYXERR0(dit->tag_); + estr += dit->tag_ + "\n"; + writeError(estr); // clear the pending tags... pending_tags_.clear(); // ...and then just fall through. @@ -296,8 +480,8 @@ XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) // 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_)) { - LYXERR0("Tried to close `" << etag.tag_ - << "' when tag was not open. Tag discarded."); + writeError("Tried to close `" + etag.tag_ + + "' when tag was not open. Tag discarded."); return *this; } @@ -314,8 +498,8 @@ XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) break; if (!html::isFontTag(rit->tag_)) { // we'll just leave it and, presumably, have to close it later. - LYXERR0("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; } } @@ -325,7 +509,7 @@ XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) // and are being asked to closed em. we want: // this is bold // first, we close the intervening tags... - StartTag curtag = tag_stack_.back(); + html::StartTag curtag = tag_stack_.back(); // ...remembering them in a stack. TagStack fontstack; while (curtag.tag_ != etag.tag_) { @@ -350,12 +534,13 @@ XHTMLStream & XHTMLStream::operator<<(EndTag const & etag) // 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. - LYXERR0("Closing tag `" << etag.tag_ - << "' when other tags are open, namely:"); - StartTag curtag = tag_stack_.back(); + writeError("Closing tag `" + etag.tag_ + + "' when other tags are open, namely:"); + html::StartTag curtag = tag_stack_.back(); while (curtag.tag_ != etag.tag_) { - LYXERR0(curtag.tag_); - os_ << curtag.asEndTag(); + writeError(curtag.tag_); + if (curtag.tag_ != parsep_tag) + os_ << curtag.asEndTag(); tag_stack_.pop_back(); curtag = tag_stack_.back(); } @@ -374,37 +559,65 @@ namespace { inline void openTag(XHTMLStream & xs, Layout const & lay) { - xs << StartTag(lay.htmltag(), lay.htmlattr()); + xs << html::StartTag(lay.htmltag(), lay.htmlattr()); +} + + +void openTag(XHTMLStream & xs, Layout const & lay, + ParagraphParameters const & params) +{ + // FIXME Are there other things we should handle here? + string const align = alignmentToCSS(params.align()); + if (align.empty()) { + openTag(xs, lay); + return; + } + string attrs = lay.htmlattr() + " style='text-align: " + align + ";'"; + xs << html::StartTag(lay.htmltag(), attrs); } inline void closeTag(XHTMLStream & xs, Layout const & lay) { - xs << EndTag(lay.htmltag()); + xs << html::EndTag(lay.htmltag()); } inline void openLabelTag(XHTMLStream & xs, Layout const & lay) { - xs << StartTag(lay.htmllabeltag(), lay.htmllabelattr()); + xs << html::StartTag(lay.htmllabeltag(), lay.htmllabelattr()); } inline void closeLabelTag(XHTMLStream & xs, Layout const & lay) { - xs << EndTag(lay.htmllabeltag()); + xs << html::EndTag(lay.htmllabeltag()); } inline void openItemTag(XHTMLStream & xs, Layout const & lay) { - xs << StartTag(lay.htmlitemtag(), lay.htmlitemattr(), true); + xs << html::StartTag(lay.htmlitemtag(), lay.htmlitemattr(), true); +} + + +void openItemTag(XHTMLStream & xs, Layout const & lay, + ParagraphParameters const & params) +{ + // FIXME Are there other things we should handle here? + string const align = alignmentToCSS(params.align()); + if (align.empty()) { + openItemTag(xs, lay); + return; + } + string attrs = lay.htmlattr() + " style='text-align: " + align + ";'"; + xs << html::StartTag(lay.htmlitemtag(), attrs); } inline void closeItemTag(XHTMLStream & xs, Layout const & lay) { - xs << EndTag(lay.htmlitemtag()); + xs << html::EndTag(lay.htmlitemtag()); } // end of convenience functions @@ -461,20 +674,20 @@ 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); + buf.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.cr(); + xs << html::CR(); - // FIXME Should we really allow anything other than 'p' here? - // If we are already in a paragraph, and this is the first one, then we // do not want to open the paragraph tag. - bool const opened = - (par == pbegin && runparams.html_in_par) ? false : true; + // 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); + openTag(xs, lay, par->params()); docstring const deferred = par->simpleLyXHTMLOnePar(buf, xs, runparams, text.outerFont(distance(begin, par))); @@ -490,11 +703,10 @@ ParagraphList::const_iterator makeParagraphs(Buffer const & buf, || (!opened && runparams.html_in_par && par == pbegin && nextpar != pend); if (needclose) { closeTag(xs, lay); - xs.cr(); + xs << html::CR(); } if (!deferred.empty()) { - xs << deferred; - xs.cr(); + xs << XHTMLStream::ESCAPE_NONE << deferred << html::CR(); } } return pend; @@ -508,21 +720,24 @@ ParagraphList::const_iterator makeBibliography(Buffer const & buf, ParagraphList::const_iterator const & pbegin, ParagraphList::const_iterator const & pend) { - xs << StartTag("h2", "class='bibliography'"); - xs << pbegin->layout().labelstring(false); - xs << EndTag("h2"); - xs.cr(); - xs << StartTag("div", "class='bibliography'"); - xs.cr(); + // FIXME XHTML + // Use TextClass::htmlTOCLayout() to figure out how we should look. + xs << html::StartTag("h2", "class='bibliography'") + << pbegin->layout().labelstring(false) + << html::EndTag("h2") + << html::CR() + << html::StartTag("div", "class='bibliography'") + << html::CR(); makeParagraphs(buf, xs, runparams, text, pbegin, pend); - xs << EndTag("div"); + xs << html::EndTag("div"); return pend; } bool isNormalEnv(Layout const & lay) { - return lay.latextype == LATEX_ENVIRONMENT; + return lay.latextype == LATEX_ENVIRONMENT + || lay.latextype == LATEX_BIB_ENVIRONMENT; } @@ -540,7 +755,7 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, // open tag for this environment openTag(xs, bstyle); - xs.cr(); + xs << html::CR(); // we will on occasion need to remember a layout from before. Layout const * lastlay = 0; @@ -549,8 +764,17 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, Layout const & style = par->layout(); // the counter only gets stepped if we're in some kind of list, // or if it's the first time through. - if (!style.counter.empty() && (par == pbegin || !isNormalEnv(style))) - buf.params().documentClass().counters().step(style.counter); + // note that enum, etc, are handled automatically. + // 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(); + docstring const & cntr = style.counter; + 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). @@ -569,53 +793,55 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, closeItemTag(xs, *lastlay); lastlay = 0; } + bool const labelfirst = style.htmllabelfirst(); - if (isNormalEnv(style)) { - // in this case, we print the label only for the first - // paragraph (as in a theorem). - openItemTag(xs, style); - if (par == pbegin && style.htmllabeltag() != "NONE") { - docstring const lbl = - pbegin->expandLabel(style, buf.params(), false); - if (!lbl.empty()) { + if (!labelfirst) + openItemTag(xs, style, par->params()); + + // label output + if (style.labeltype != LABEL_NO_LABEL && + style.htmllabeltag() != "NONE") { + if (isNormalEnv(style)) { + // in this case, we print the label only for the first + // paragraph (as in a theorem). + if (par == pbegin) { + docstring const lbl = + pbegin->params().labelString(); + if (!lbl.empty()) { + openLabelTag(xs, style); + xs << lbl; + closeLabelTag(xs, style); + } + xs << html::CR(); + } + } else { // some kind of list + if (style.labeltype == LABEL_MANUAL) { openLabelTag(xs, style); - xs << lbl; + sep = par->firstWordLyXHTML(xs, runparams); closeLabelTag(xs, style); + xs << html::CR(); + } + else { + openLabelTag(xs, style); + xs << par->params().labelString(); + closeLabelTag(xs, style); + xs << html::CR(); } - xs.cr(); - } - } else { // some kind of list - if (!labelfirst) - openItemTag(xs, style); - if (style.labeltype == LABEL_MANUAL - && style.htmllabeltag() != "NONE") { - openLabelTag(xs, style); -// sep = par->firstWordLyXHTML(xs, runparams); - closeLabelTag(xs, style); - xs.cr(); - } - else if (style.labeltype != LABEL_NO_LABEL - && style.htmllabeltag() != "NONE") { - openLabelTag(xs, style); - xs << par->expandLabel(style, buf.params(), false); - closeLabelTag(xs, style); - xs.cr(); } - if (labelfirst) - openItemTag(xs, style); - else - xs << StartTag("span", "class='" + to_utf8(style.name()) + " inneritem'"); - } + } // end label output + + if (labelfirst) + openItemTag(xs, style, par->params()); + par->simpleLyXHTMLOnePar(buf, xs, runparams, text.outerFont(distance(begin, par)), sep); - if (!isNormalEnv(style) && !labelfirst) - xs << EndTag("span"); ++par; - // We may not want to close the tag yet, in particular, - // if we're not at the end... + + // We may not want to close the tag yet, in particular: + // If we're not at the end... if (par != pend // and are doing items... - && style.latextype == LATEX_ITEM_ENVIRONMENT + && !isNormalEnv(style) // and if the depth has changed... && par->params().depth() != origdepth) { // then we'll save this layout for later, and close it when @@ -623,7 +849,7 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, lastlay = &style; } else closeItemTag(xs, style); - xs.cr(); + xs << html::CR(); } // The other possibility is that the depth has increased, in which // case we need to recurse. @@ -653,7 +879,7 @@ ParagraphList::const_iterator makeEnvironmentHtml(Buffer const & buf, if (lastlay != 0) closeItemTag(xs, *lastlay); closeTag(xs, bstyle); - xs.cr(); + xs << html::CR(); return pend; } @@ -666,15 +892,15 @@ void makeCommand(Buffer const & buf, { Layout const & style = pbegin->layout(); if (!style.counter.empty()) - buf.params().documentClass().counters().step(style.counter); + buf.params().documentClass().counters().step(style.counter, OutputUpdate); - openTag(xs, style); + openTag(xs, style, pbegin->params()); // Label around sectioning number: // FIXME Probably need to account for LABEL_MANUAL if (style.labeltype != LABEL_NO_LABEL) { openLabelTag(xs, style); - xs << pbegin->expandLabel(style, buf.params(), false); + xs << pbegin->params().labelString(); closeLabelTag(xs, style); // Otherwise the label might run together with the text xs << from_ascii(" "); @@ -684,7 +910,7 @@ void makeCommand(Buffer const & buf, pbegin->simpleLyXHTMLOnePar(buf, xs, runparams, text.outerFont(distance(begin, pbegin))); closeTag(xs, style); - xs.cr(); + xs << html::CR(); } } // end anonymous namespace @@ -701,6 +927,13 @@ void xhtmlParagraphs(Text const & text, OutputParams ourparams = runparams; while (par != pend) { + 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); + } Layout const & style = par->layout(); ParagraphList::const_iterator lastpar = par; ParagraphList::const_iterator send; @@ -747,4 +980,20 @@ void xhtmlParagraphs(Text const & text, } +string alignmentToCSS(LyXAlignment align) { + switch (align) { + case LYX_ALIGN_BLOCK: + // we are NOT going to use text-align: justify!! + case LYX_ALIGN_LEFT: + return "left"; + case LYX_ALIGN_RIGHT: + return "right"; + case LYX_ALIGN_CENTER: + return "center"; + default: + break; + } + return ""; +} + } // namespace lyx