discuss@mandoc.bsd.lv
 help / color / mirror / Atom feed
* Not all section headers have id in HTML output
@ 2020-10-28 22:40 Aman Verma
  2020-10-30 13:35 ` Ingo Schwarze
  0 siblings, 1 reply; 2+ messages in thread
From: Aman Verma @ 2020-10-28 22:40 UTC (permalink / raw)
  To: discuss; +Cc: schwarze

Hi Ingo,

Thanks for the fix for the `pre` bug. I have another (possible)
bug to report:

In the HTML output for mdoc(7)[1] the first two section headers don't
have an id attribute. In fact, it seems that two other elements down the
page have taken the ids ("NAME" and "DESCRIPTION") that the first two
headers were supposed to have. This makes it impossible to link to those
headers and also breaks the table of contents generated with the `-O toc`
option to mandoc.

[1]: https://mandoc.bsd.lv/man/mdoc.7.html

Best regards,
Aman
--
 To unsubscribe send an email to discuss+unsubscribe@mandoc.bsd.lv


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Not all section headers have id in HTML output
  2020-10-28 22:40 Not all section headers have id in HTML output Aman Verma
@ 2020-10-30 13:35 ` Ingo Schwarze
  0 siblings, 0 replies; 2+ messages in thread
From: Ingo Schwarze @ 2020-10-30 13:35 UTC (permalink / raw)
  To: Aman Verma; +Cc: discuss

Hello Aman,

Aman Verma wrote on Wed, Oct 28, 2020 at 06:40:45PM -0400:

> Thanks for the fix for the `pre` bug. I have another (possible)
> bug to report:
> 
> In the HTML output for mdoc(7)[1] the first two section headers don't
> have an id attribute. In fact, it seems that two other elements down the
> page have taken the ids ("NAME" and "DESCRIPTION") that the first two
> headers were supposed to have.

That is true.  Automatic tagging of the .It macro tagged with
TAG_STRONG whereas .Sh only tagged with TAG_WEAK.

> This makes it impossible to link to those headers and also breaks the
> table of contents generated with the `-O toc` option to mandoc.

You are right; arguably, that is not ideal.

So i changed the priority from TAG_WEAK to TAG_STRONG for all
headers where the header text can be used as the fragment identifier
without any modification; that includes single-word titles, in
particular NAME and DESCRIPTION, see the patch below.

I do see value in having ...#DESCRIPTION always linking to the
description, just as you say.

I have also installed the changed code on https://man.openbsd.org/
such that the new behaviour can be easily inspected.

Thanks for the suggestion,
  Ingo


Log Message:
-----------
Promote section headers that can can be used unmodified as fragment 
identifiers from TAG_WEAK to TAG_STRONG, 
such that for example ...#DESCRIPTION always works.
Suggested by Aman Verma on the discuss@ list.

Modified Files:
--------------
    mandoc:
        man_validate.c
        mdoc_validate.c

Revision Data
-------------
Index: mdoc_validate.c
===================================================================
RCS file: /home/cvs/mandoc/mandoc/mdoc_validate.c,v
retrieving revision 1.387
retrieving revision 1.388
diff -Lmdoc_validate.c -Lmdoc_validate.c -u -p -r1.387 -r1.388
--- mdoc_validate.c
+++ mdoc_validate.c
@@ -2617,7 +2617,7 @@ post_section(POST_ARGS)
 			if ((nch = n->child) != NULL &&
 			    nch->type == ROFFT_TEXT &&
 			    strcmp(nch->string, tag) == 0)
-				tag_put(NULL, TAG_WEAK, n);
+				tag_put(NULL, TAG_STRONG, n);
 			else
 				tag_put(tag, TAG_FALLBACK, n);
 			free(tag);
Index: man_validate.c
===================================================================
RCS file: /home/cvs/mandoc/mandoc/man_validate.c,v
retrieving revision 1.154
retrieving revision 1.155
diff -Lman_validate.c -Lman_validate.c -u -p -r1.154 -r1.155
--- man_validate.c
+++ man_validate.c
@@ -324,7 +324,7 @@ post_SH(CHKARGS)
 					*cp = '_';
 			if (nc != NULL && nc->type == ROFFT_TEXT &&
 			    strcmp(nc->string, tag) == 0)
-				tag_put(NULL, TAG_WEAK, n);
+				tag_put(NULL, TAG_STRONG, n);
 			else
 				tag_put(tag, TAG_FALLBACK, n);
 			free(tag);
--
 To unsubscribe send an email to discuss+unsubscribe@mandoc.bsd.lv


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-10-30 13:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-28 22:40 Not all section headers have id in HTML output Aman Verma
2020-10-30 13:35 ` Ingo Schwarze

discuss@mandoc.bsd.lv

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.vuxu.org/mandoc-discuss

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 mandoc-discuss mandoc-discuss/ http://inbox.vuxu.org/mandoc-discuss \
		discuss@mandoc.bsd.lv
	public-inbox-index mandoc-discuss

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.vuxu.org/vuxu.archive.mandoc.discuss


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git