]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_html.c
The .UR and .MT blocks in man(7) are represented by <a> elements
[mandoc.git] / mdoc_html.c
index a3c05f087093a6ec78c7fdc330909082db662a16..ea750cdb3585c5f27e2ee3ebbcd3b152848075c7 100644 (file)
@@ -1,7 +1,7 @@
-/*     $Id: mdoc_html.c,v 1.320 2018/12/25 00:06:25 schwarze Exp $ */
+/*     $Id: mdoc_html.c,v 1.327 2019/01/18 14:36:21 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2011, 2014 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2014,2015,2016,2017,2018 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2014-2019 Ingo Schwarze <schwarze@openbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -268,22 +268,21 @@ synopsis_pre(struct html *h, const struct roff_node *n)
        case MDOC_Fo:
        case MDOC_In:
        case MDOC_Vt:
-               print_paragraph(h);
                break;
        case MDOC_Ft:
-               if (MDOC_Fn != n->tok && MDOC_Fo != n->tok) {
-                       print_paragraph(h);
+               if (n->tok != MDOC_Fn && n->tok != MDOC_Fo)
                        break;
-               }
                /* FALLTHROUGH */
        default:
                print_otag(h, TAG_BR, "");
-               break;
+               return;
        }
+       html_close_paragraph(h);
+       print_otag(h, TAG_P, "c", "Pp");
 }
 
 void
-html_mdoc(void *arg, const struct roff_man *mdoc)
+html_mdoc(void *arg, const struct roff_meta *mdoc)
 {
        struct html             *h;
        struct roff_node        *n;
@@ -295,19 +294,19 @@ html_mdoc(void *arg, const struct roff_man *mdoc)
        if ((h->oflags & HTML_FRAGMENT) == 0) {
                print_gen_decls(h);
                print_otag(h, TAG_HTML, "");
-               if (n->type == ROFFT_COMMENT)
+               if (n != NULL && n->type == ROFFT_COMMENT)
                        print_gen_comment(h, n);
                t = print_otag(h, TAG_HEAD, "");
-               print_mdoc_head(&mdoc->meta, h);
+               print_mdoc_head(mdoc, h);
                print_tagq(h, t);
                print_otag(h, TAG_BODY, "");
        }
 
-       mdoc_root_pre(&mdoc->meta, h);
+       mdoc_root_pre(mdoc, h);
        t = print_otag(h, TAG_DIV, "c", "manual-text");
-       print_mdoc_nodelist(&mdoc->meta, n, h);
+       print_mdoc_nodelist(mdoc, n, h);
        print_tagq(h, t);
-       mdoc_root_post(&mdoc->meta, h);
+       mdoc_root_post(mdoc, h);
        print_tagq(h, NULL);
 }
 
@@ -346,18 +345,21 @@ print_mdoc_nodelist(MDOC_ARGS)
 static void
 print_mdoc_node(MDOC_ARGS)
 {
-       int              child;
        struct tag      *t;
+       int              child;
 
        if (n->type == ROFFT_COMMENT || n->flags & NODE_NOPRT)
                return;
 
+       html_fillmode(h, n->flags & NODE_NOFILL ? ROFF_nf : ROFF_fi);
+
        child = 1;
-       t = h->tag;
        n->flags &= ~NODE_ENDED;
-
        switch (n->type) {
        case ROFFT_TEXT:
+               t = h->tag;
+               t->refcnt++;
+
                /* No tables in this mode... */
                assert(NULL == h->tblt);
 
@@ -366,15 +368,18 @@ print_mdoc_node(MDOC_ARGS)
                 * (i.e., within a <PRE>) don't print the newline.
                 */
                if (*n->string == ' ' && n->flags & NODE_LINE &&
-                   (h->flags & (HTML_LITERAL | HTML_NONEWLINE)) == 0)
+                   (h->flags & HTML_NONEWLINE) == 0 &&
+                   (n->flags & NODE_NOFILL) == 0)
                        print_otag(h, TAG_BR, "");
                if (NODE_DELIMC & n->flags)
                        h->flags |= HTML_NOSPACE;
                print_text(h, n->string);
                if (NODE_DELIMO & n->flags)
                        h->flags |= HTML_NOSPACE;
-               return;
+               break;
        case ROFFT_EQN:
+               t = h->tag;
+               t->refcnt++;
                print_eqn(h, n->eqn);
                break;
        case ROFFT_TBL:
@@ -391,15 +396,16 @@ print_mdoc_node(MDOC_ARGS)
                 * the "meta" table state.  This will be reopened on the
                 * next table element.
                 */
-               if (h->tblt != NULL) {
+               if (h->tblt != NULL)
                        print_tblclose(h);
-                       t = h->tag;
-               }
                assert(h->tblt == NULL);
+               t = h->tag;
+               t->refcnt++;
                if (n->tok < ROFF_MAX) {
                        roff_html_pre(h, n);
-                       child = 0;
-                       break;
+                       t->refcnt--;
+                       print_stagq(h, t);
+                       return;
                }
                assert(n->tok >= MDOC_Dd && n->tok < MDOC_MAX);
                if (mdoc_html_acts[n->tok - MDOC_Dd].pre != NULL &&
@@ -414,17 +420,18 @@ print_mdoc_node(MDOC_ARGS)
                h->flags |= HTML_PREKEEP;
        }
 
-       if (child && n->child)
+       if (child && n->child != NULL)
                print_mdoc_nodelist(meta, n->child, h);
 
+       t->refcnt--;
        print_stagq(h, t);
 
        switch (n->type) {
+       case ROFFT_TEXT:
        case ROFFT_EQN:
                break;
        default:
-               if (n->tok < ROFF_MAX ||
-                   mdoc_html_acts[n->tok - MDOC_Dd].post == NULL ||
+               if (mdoc_html_acts[n->tok - MDOC_Dd].post == NULL ||
                    n->flags & NODE_ENDED)
                        break;
                (*mdoc_html_acts[n->tok - MDOC_Dd].post)(meta, n, h);
@@ -432,6 +439,12 @@ print_mdoc_node(MDOC_ARGS)
                        n->body->flags |= NODE_ENDED;
                break;
        }
+
+       if (n->flags & NODE_NOFILL &&
+           (n->next == NULL || n->next->flags & NODE_LINE)) {
+               h->col++;
+               print_endline(h);
+       }
 }
 
 static void
@@ -515,6 +528,7 @@ mdoc_sh_pre(MDOC_ARGS)
 
        switch (n->type) {
        case ROFFT_BLOCK:
+               html_close_paragraph(h);
                if ((h->oflags & HTML_TOC) == 0 ||
                    h->flags & HTML_TOCDONE ||
                    n->sec <= SEC_SYNOPSIS)
@@ -562,7 +576,7 @@ mdoc_sh_pre(MDOC_ARGS)
                break;
        case ROFFT_HEAD:
                id = html_make_id(n, 1);
-               print_otag(h, TAG_H1, "cTi", "Sh", id);
+               print_otag(h, TAG_H1, "ci", "Sh", id);
                if (id != NULL)
                        print_otag(h, TAG_A, "chR", "permalink", id);
                break;
@@ -581,11 +595,20 @@ mdoc_ss_pre(MDOC_ARGS)
 {
        char    *id;
 
-       if (n->type != ROFFT_HEAD)
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
+               return 1;
+       case ROFFT_HEAD:
+               break;
+       case ROFFT_BODY:
                return 1;
+       default:
+               abort();
+       }
 
        id = html_make_id(n, 1);
-       print_otag(h, TAG_H2, "cTi", "Ss", id);
+       print_otag(h, TAG_H2, "ci", "Ss", id);
        if (id != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
        return 1;
@@ -598,7 +621,7 @@ mdoc_fl_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Fl", id);
+       print_otag(h, TAG_CODE, "ci", "Fl", id);
 
        print_text(h, "\\-");
        if (!(n->child == NULL &&
@@ -617,19 +640,27 @@ mdoc_cm_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Cm", id);
+       print_otag(h, TAG_CODE, "ci", "Cm", id);
        return 1;
 }
 
 static int
 mdoc_nd_pre(MDOC_ARGS)
 {
-       if (n->type != ROFFT_BODY)
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
                return 1;
-
+       case ROFFT_HEAD:
+               return 0;
+       case ROFFT_BODY:
+               break;
+       default:
+               abort();
+       }
        print_text(h, "\\(em");
        /* Cannot use TAG_SPAN because it may contain blocks. */
-       print_otag(h, TAG_DIV, "cT", "Nd");
+       print_otag(h, TAG_DIV, "c", "Nd");
        return 1;
 }
 
@@ -637,18 +668,21 @@ static int
 mdoc_nm_pre(MDOC_ARGS)
 {
        switch (n->type) {
+       case ROFFT_BLOCK:
+               break;
        case ROFFT_HEAD:
                print_otag(h, TAG_TD, "");
                /* FALLTHROUGH */
        case ROFFT_ELEM:
-               print_otag(h, TAG_CODE, "cT", "Nm");
+               print_otag(h, TAG_CODE, "c", "Nm");
                return 1;
        case ROFFT_BODY:
                print_otag(h, TAG_TD, "");
                return 1;
        default:
-               break;
+               abort();
        }
+       html_close_paragraph(h);
        synopsis_pre(h, n);
        print_otag(h, TAG_TABLE, "c", "Nm");
        print_otag(h, TAG_TR, "");
@@ -662,11 +696,11 @@ mdoc_xr_pre(MDOC_ARGS)
                return 0;
 
        if (h->base_man1)
-               print_otag(h, TAG_A, "cThM", "Xr",
+               print_otag(h, TAG_A, "chM", "Xr",
                    n->child->string, n->child->next == NULL ?
                    NULL : n->child->next->string);
        else
-               print_otag(h, TAG_A, "cT", "Xr");
+               print_otag(h, TAG_A, "c", "Xr");
 
        n = n->child;
        print_text(h, n->string);
@@ -695,7 +729,7 @@ mdoc_ns_pre(MDOC_ARGS)
 static int
 mdoc_ar_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_VAR, "cT", "Ar");
+       print_otag(h, TAG_VAR, "c", "Ar");
        return 1;
 }
 
@@ -710,7 +744,6 @@ static int
 mdoc_it_pre(MDOC_ARGS)
 {
        const struct roff_node  *bl;
-       struct tag              *t;
        enum mdoc_list           type;
 
        bl = n->parent;
@@ -752,17 +785,6 @@ mdoc_it_pre(MDOC_ARGS)
        case LIST_tag:
                switch (n->type) {
                case ROFFT_HEAD:
-                       if (h->style != NULL && !bl->norm->Bl.comp &&
-                           (n->parent->prev == NULL ||
-                            n->parent->prev->body == NULL ||
-                            n->parent->prev->body->child != NULL)) {
-                               t = print_otag(h, TAG_DT, "");
-                               print_text(h, "\\ ");
-                               print_tagq(h, t);
-                               t = print_otag(h, TAG_DD, "");
-                               print_text(h, "\\ ");
-                               print_tagq(h, t);
-                       }
                        print_otag(h, TAG_DT, "");
                        break;
                case ROFFT_BODY:
@@ -801,12 +823,15 @@ mdoc_bl_pre(MDOC_ARGS)
        enum htmltag     elemtype;
 
        switch (n->type) {
-       case ROFFT_BODY:
-               return 1;
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
+               break;
        case ROFFT_HEAD:
                return 0;
+       case ROFFT_BODY:
+               return 1;
        default:
-               break;
+               abort();
        }
 
        bl = &n->norm->Bl;
@@ -876,28 +901,34 @@ mdoc_ex_pre(MDOC_ARGS)
 static int
 mdoc_st_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_SPAN, "cT", "St");
+       print_otag(h, TAG_SPAN, "c", "St");
        return 1;
 }
 
 static int
 mdoc_em_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_I, "cT", "Em");
+       print_otag(h, TAG_I, "c", "Em");
        return 1;
 }
 
 static int
 mdoc_d1_pre(MDOC_ARGS)
 {
-       if (n->type != ROFFT_BLOCK)
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
+               break;
+       case ROFFT_HEAD:
+               return 0;
+       case ROFFT_BODY:
                return 1;
-
+       default:
+               abort();
+       }
        print_otag(h, TAG_DIV, "c", "Bd Bd-indent");
-
        if (n->tok == MDOC_Dl)
                print_otag(h, TAG_CODE, "c", "Li");
-
        return 1;
 }
 
@@ -907,7 +938,7 @@ mdoc_sx_pre(MDOC_ARGS)
        char    *id;
 
        id = html_make_id(n, 0);
-       print_otag(h, TAG_A, "cThR", "Sx", id);
+       print_otag(h, TAG_A, "chR", "Sx", id);
        free(id);
        return 1;
 }
@@ -915,85 +946,51 @@ mdoc_sx_pre(MDOC_ARGS)
 static int
 mdoc_bd_pre(MDOC_ARGS)
 {
-       int                      comp, sv;
+       char                     buf[16];
        struct roff_node        *nn;
+       int                      comp;
 
-       if (n->type == ROFFT_HEAD)
-               return 0;
-
-       if (n->type == ROFFT_BLOCK) {
-               comp = n->norm->Bd.comp;
-               for (nn = n; nn && ! comp; nn = nn->parent) {
-                       if (nn->type != ROFFT_BLOCK)
-                               continue;
-                       if (MDOC_Ss == nn->tok || MDOC_Sh == nn->tok)
-                               comp = 1;
-                       if (nn->prev)
-                               break;
-               }
-               if ( ! comp)
-                       print_paragraph(h);
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
                return 1;
+       case ROFFT_HEAD:
+               return 0;
+       case ROFFT_BODY:
+               break;
+       default:
+               abort();
        }
 
-       /* Handle the -offset argument. */
-
-       if (n->norm->Bd.offs == NULL ||
-           ! strcmp(n->norm->Bd.offs, "left"))
-               print_otag(h, TAG_DIV, "c", "Bd");
-       else
-               print_otag(h, TAG_DIV, "c", "Bd Bd-indent");
+       /* Handle preceding whitespace. */
 
-       if (n->norm->Bd.type != DISP_unfilled &&
-           n->norm->Bd.type != DISP_literal)
-               return 1;
-
-       print_otag(h, TAG_PRE, "c", "Li");
-
-       /* This can be recursive: save & set our literal state. */
-
-       sv = h->flags & HTML_LITERAL;
-       h->flags |= HTML_LITERAL;
-
-       for (nn = n->child; nn; nn = nn->next) {
-               print_mdoc_node(meta, nn, h);
-               /*
-                * If the printed node flushes its own line, then we
-                * needn't do it here as well.  This is hacky, but the
-                * notion of selective eoln whitespace is pretty dumb
-                * anyway, so don't sweat it.
-                */
-               switch (nn->tok) {
-               case ROFF_br:
-               case ROFF_sp:
-               case MDOC_Sm:
-               case MDOC_Bl:
-               case MDOC_D1:
-               case MDOC_Dl:
-               case MDOC_Pp:
+       comp = n->norm->Bd.comp;
+       for (nn = n; nn != NULL && comp == 0; nn = nn->parent) {
+               if (nn->type != ROFFT_BLOCK)
                        continue;
-               default:
+               if (nn->tok == MDOC_Sh || nn->tok == MDOC_Ss)
+                       comp = 1;
+               if (nn->prev != NULL)
                        break;
-               }
-               if (h->flags & HTML_NONEWLINE ||
-                   (nn->next && ! (nn->next->flags & NODE_LINE)))
-                       continue;
-               else if (nn->next)
-                       print_text(h, "\n");
-
-               h->flags |= HTML_NOSPACE;
        }
+       (void)strlcpy(buf, "Bd", sizeof(buf));
+       if (comp == 0)
+               (void)strlcat(buf, " Pp", sizeof(buf));
 
-       if (0 == sv)
-               h->flags &= ~HTML_LITERAL;
+       /* Handle the -offset argument. */
 
-       return 0;
+       if (n->norm->Bd.offs != NULL &&
+           strcmp(n->norm->Bd.offs, "left") != 0)
+               (void)strlcat(buf, " Bd-indent", sizeof(buf));
+
+       print_otag(h, TAG_DIV, "c", buf);
+       return 1;
 }
 
 static int
 mdoc_pa_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_SPAN, "cT", "Pa");
+       print_otag(h, TAG_SPAN, "c", "Pa");
        return 1;
 }
 
@@ -1024,7 +1021,7 @@ mdoc_an_pre(MDOC_ARGS)
        if (n->sec == SEC_AUTHORS && ! (h->flags & HTML_NOSPLIT))
                h->flags |= HTML_SPLIT;
 
-       print_otag(h, TAG_SPAN, "cT", "An");
+       print_otag(h, TAG_SPAN, "c", "An");
        return 1;
 }
 
@@ -1032,7 +1029,7 @@ static int
 mdoc_cd_pre(MDOC_ARGS)
 {
        synopsis_pre(h, n);
-       print_otag(h, TAG_CODE, "cT", "Cd");
+       print_otag(h, TAG_CODE, "c", "Cd");
        return 1;
 }
 
@@ -1043,7 +1040,7 @@ mdoc_dv_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Dv", id);
+       print_otag(h, TAG_CODE, "ci", "Dv", id);
        return 1;
 }
 
@@ -1054,7 +1051,7 @@ mdoc_ev_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Ev", id);
+       print_otag(h, TAG_CODE, "ci", "Ev", id);
        return 1;
 }
 
@@ -1071,7 +1068,7 @@ mdoc_er_pre(MDOC_ARGS)
 
        if (id != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Er", id);
+       print_otag(h, TAG_CODE, "ci", "Er", id);
        return 1;
 }
 
@@ -1082,12 +1079,12 @@ mdoc_fa_pre(MDOC_ARGS)
        struct tag              *t;
 
        if (n->parent->tok != MDOC_Fo) {
-               print_otag(h, TAG_VAR, "cT", "Fa");
+               print_otag(h, TAG_VAR, "c", "Fa");
                return 1;
        }
 
        for (nn = n->child; nn; nn = nn->next) {
-               t = print_otag(h, TAG_VAR, "cT", "Fa");
+               t = print_otag(h, TAG_VAR, "c", "Fa");
                print_text(h, nn->string);
                print_tagq(h, t);
                if (nn->next) {
@@ -1118,11 +1115,11 @@ mdoc_fd_pre(MDOC_ARGS)
        assert(n->type == ROFFT_TEXT);
 
        if (strcmp(n->string, "#include")) {
-               print_otag(h, TAG_CODE, "cT", "Fd");
+               print_otag(h, TAG_CODE, "c", "Fd");
                return 1;
        }
 
-       print_otag(h, TAG_CODE, "cT", "In");
+       print_otag(h, TAG_CODE, "c", "In");
        print_text(h, n->string);
 
        if (NULL != (n = n->next)) {
@@ -1136,10 +1133,10 @@ mdoc_fd_pre(MDOC_ARGS)
                        cp = strchr(buf, '\0') - 1;
                        if (cp >= buf && (*cp == '>' || *cp == '"'))
                                *cp = '\0';
-                       t = print_otag(h, TAG_A, "cThI", "In", buf);
+                       t = print_otag(h, TAG_A, "chI", "In", buf);
                        free(buf);
                } else
-                       t = print_otag(h, TAG_A, "cT", "In");
+                       t = print_otag(h, TAG_A, "c", "In");
 
                print_text(h, n->string);
                print_tagq(h, t);
@@ -1166,7 +1163,7 @@ mdoc_vt_pre(MDOC_ARGS)
        } else if (n->type == ROFFT_HEAD)
                return 0;
 
-       print_otag(h, TAG_VAR, "cT", "Vt");
+       print_otag(h, TAG_VAR, "c", "Vt");
        return 1;
 }
 
@@ -1174,7 +1171,7 @@ static int
 mdoc_ft_pre(MDOC_ARGS)
 {
        synopsis_pre(h, n);
-       print_otag(h, TAG_VAR, "cT", "Ft");
+       print_otag(h, TAG_VAR, "c", "Ft");
        return 1;
 }
 
@@ -1195,7 +1192,7 @@ mdoc_fn_pre(MDOC_ARGS)
 
        ep = strchr(sp, ' ');
        if (NULL != ep) {
-               t = print_otag(h, TAG_VAR, "cT", "Ft");
+               t = print_otag(h, TAG_VAR, "c", "Ft");
 
                while (ep) {
                        sz = MIN((int)(ep - sp), BUFSIZ - 1);
@@ -1208,7 +1205,7 @@ mdoc_fn_pre(MDOC_ARGS)
                print_tagq(h, t);
        }
 
-       t = print_otag(h, TAG_CODE, "cT", "Fn");
+       t = print_otag(h, TAG_CODE, "c", "Fn");
 
        if (sp)
                print_text(h, sp);
@@ -1221,10 +1218,10 @@ mdoc_fn_pre(MDOC_ARGS)
 
        for (n = n->child->next; n; n = n->next) {
                if (NODE_SYNPRETTY & n->flags)
-                       t = print_otag(h, TAG_VAR, "cTs", "Fa",
+                       t = print_otag(h, TAG_VAR, "cs", "Fa",
                            "white-space", "nowrap");
                else
-                       t = print_otag(h, TAG_VAR, "cT", "Fa");
+                       t = print_otag(h, TAG_VAR, "c", "Fa");
                print_text(h, n->string);
                print_tagq(h, t);
                if (n->next) {
@@ -1271,8 +1268,10 @@ mdoc_skip_pre(MDOC_ARGS)
 static int
 mdoc_pp_pre(MDOC_ARGS)
 {
-
-       print_paragraph(h);
+       if ((n->flags & NODE_NOFILL) == 0) {
+               html_close_paragraph(h);
+               print_otag(h, TAG_P, "c", "Pp");
+       }
        return 0;
 }
 
@@ -1295,7 +1294,7 @@ mdoc_lk_pre(MDOC_ARGS)
        descr = link->next;
        if (descr == punct)
                descr = link;  /* no text */
-       t = print_otag(h, TAG_A, "cTh", "Lk", link->string);
+       t = print_otag(h, TAG_A, "ch", "Lk", link->string);
        do {
                if (descr->flags & (NODE_DELIMC | NODE_DELIMO))
                        h->flags |= HTML_NOSPACE;
@@ -1323,7 +1322,7 @@ mdoc_mt_pre(MDOC_ARGS)
                assert(n->type == ROFFT_TEXT);
 
                mandoc_asprintf(&cp, "mailto:%s", n->string);
-               t = print_otag(h, TAG_A, "cTh", "Mt", cp);
+               t = print_otag(h, TAG_A, "ch", "Mt", cp);
                print_text(h, n->string);
                print_tagq(h, t);
                free(cp);
@@ -1351,7 +1350,7 @@ mdoc_fo_pre(MDOC_ARGS)
                return 0;
 
        assert(n->child->string);
-       t = print_otag(h, TAG_CODE, "cT", "Fn");
+       t = print_otag(h, TAG_CODE, "c", "Fn");
        print_text(h, n->child->string);
        print_tagq(h, t);
        return 0;
@@ -1375,7 +1374,7 @@ mdoc_in_pre(MDOC_ARGS)
        struct tag      *t;
 
        synopsis_pre(h, n);
-       print_otag(h, TAG_CODE, "cT", "In");
+       print_otag(h, TAG_CODE, "c", "In");
 
        /*
         * The first argument of the `In' gets special treatment as
@@ -1394,9 +1393,9 @@ mdoc_in_pre(MDOC_ARGS)
                assert(n->type == ROFFT_TEXT);
 
                if (h->base_includes)
-                       t = print_otag(h, TAG_A, "cThI", "In", n->string);
+                       t = print_otag(h, TAG_A, "chI", "In", n->string);
                else
-                       t = print_otag(h, TAG_A, "cT", "In");
+                       t = print_otag(h, TAG_A, "c", "In");
                print_text(h, n->string);
                print_tagq(h, t);
 
@@ -1421,14 +1420,14 @@ mdoc_ic_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_CODE, "cTi", "Ic", id);
+       print_otag(h, TAG_CODE, "ci", "Ic", id);
        return 1;
 }
 
 static int
 mdoc_va_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_VAR, "cT", "Va");
+       print_otag(h, TAG_VAR, "c", "Va");
        return 1;
 }
 
@@ -1447,10 +1446,17 @@ mdoc_bf_pre(MDOC_ARGS)
 {
        const char      *cattr;
 
-       if (n->type == ROFFT_HEAD)
-               return 0;
-       else if (n->type != ROFFT_BODY)
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               html_close_paragraph(h);
                return 1;
+       case ROFFT_HEAD:
+               return 0;
+       case ROFFT_BODY:
+               break;
+       default:
+               abort();
+       }
 
        if (FONT_Em == n->norm->Bf.font)
                cattr = "Bf Em";
@@ -1473,7 +1479,7 @@ mdoc_ms_pre(MDOC_ARGS)
 
        if ((id = cond_id(n)) != NULL)
                print_otag(h, TAG_A, "chR", "permalink", id);
-       print_otag(h, TAG_SPAN, "cTi", "Ms", id);
+       print_otag(h, TAG_SPAN, "ci", "Ms", id);
        return 1;
 }
 
@@ -1496,13 +1502,21 @@ mdoc_pf_post(MDOC_ARGS)
 static int
 mdoc_rs_pre(MDOC_ARGS)
 {
-       if (n->type != ROFFT_BLOCK)
-               return 1;
-
-       if (n->prev && SEC_SEE_ALSO == n->sec)
-               print_paragraph(h);
-
-       print_otag(h, TAG_CITE, "cT", "Rs");
+       switch (n->type) {
+       case ROFFT_BLOCK:
+               if (n->sec == SEC_SEE_ALSO)
+                       html_close_paragraph(h);
+               break;
+       case ROFFT_HEAD:
+               return 0;
+       case ROFFT_BODY:
+               if (n->sec == SEC_SEE_ALSO)
+                       print_otag(h, TAG_P, "c", "Pp");
+               print_otag(h, TAG_CITE, "c", "Rs");
+               break;
+       default:
+               abort();
+       }
        return 1;
 }
 
@@ -1531,7 +1545,7 @@ mdoc_li_pre(MDOC_ARGS)
 static int
 mdoc_sy_pre(MDOC_ARGS)
 {
-       print_otag(h, TAG_B, "cT", "Sy");
+       print_otag(h, TAG_B, "c", "Sy");
        return 1;
 }
 
@@ -1541,7 +1555,7 @@ mdoc_lb_pre(MDOC_ARGS)
        if (SEC_LIBRARY == n->sec && NODE_LINE & n->flags && n->prev)
                print_otag(h, TAG_BR, "");
 
-       print_otag(h, TAG_SPAN, "cT", "Lb");
+       print_otag(h, TAG_SPAN, "c", "Lb");
        return 1;
 }
 
@@ -1679,9 +1693,15 @@ mdoc_quote_pre(MDOC_ARGS)
        case MDOC_Oo:
        case MDOC_Op:
                print_text(h, "\\(lB");
-               h->flags |= HTML_NOSPACE;
-               /* Cannot use TAG_SPAN because it may contain blocks. */
-               print_otag(h, TAG_IDIV, "c", "Op");
+               /*
+                * Give up on semantic markup for now.
+                * We cannot use TAG_SPAN because .Oo may contain blocks.
+                * We cannot use TAG_IDIV because we might be in a
+                * phrasing context (like .Dl or .Pp); we cannot
+                * close out a .Pp at this point either because
+                * that would break the line.
+                */
+               /* XXX print_otag(h, TAG_???, "c", "Op"); */
                break;
        case MDOC_En:
                if (NULL == n->norm->Es ||