[9front] mothra: fix rendering of <samp> tag

[9front] mothra: fix rendering of <samp> tag

[Kristo]

Hi all,

Mothra does not currently render text inside <samp> tags inline
similar to <code>, but instead treats them like <pre> which is actually
incorrect behavior. The following small patch should fix this issue.

diff 9d43029ff984435111eff658308a44b4f3eee1cc uncommitted
--- a/sys/src/cmd/mothra/rdhtml.c
+++ b/sys/src/cmd/mothra/rdhtml.c
@@ -982,6 +982,7 @@
 			g.state->size=NORMAL;
 			break;
 		case Tag_code:
+		case Tag_samp:
 			g.state->font=CWIDTH;
 			g.state->size=NORMAL;
 			break;
@@ -1113,7 +1114,6 @@
 		case Tag_xmp:
 			htmlerror(g.name, g.lineno, "<%s> deprecated", tag[g.tag].name);
 		case Tag_pre:
-		case Tag_samp:
 			g.state->indent=0;
 			g.state->pre=1;
 			g.state->font=CWIDTH;
@@ -1203,7 +1203,6 @@
 		case Tag_listing:
 		case Tag_menu:
 		case Tag_ol:
-		case Tag_samp:
 		case Tag_title:
 		case Tag_ul:
 		case Tag_xmp:

Re: [9front] mothra: fix rendering of <samp> tag

[ori]

Do you have an example site or html snippet that
I can compare against?

Quoth Kristo <kristo.ilmari@gmail.com>:
> Hi all,
> 
> Mothra does not currently render text inside <samp> tags inline
> similar to <code>, but instead treats them like <pre> which is actually
> incorrect behavior. The following small patch should fix this issue.
> 
> diff 9d43029ff984435111eff658308a44b4f3eee1cc uncommitted
> --- a/sys/src/cmd/mothra/rdhtml.c
> +++ b/sys/src/cmd/mothra/rdhtml.c
> @@ -982,6 +982,7 @@
>  			g.state->size=NORMAL;
>  			break;
>  		case Tag_code:
> +		case Tag_samp:
>  			g.state->font=CWIDTH;
>  			g.state->size=NORMAL;
>  			break;
> @@ -1113,7 +1114,6 @@
>  		case Tag_xmp:
>  			htmlerror(g.name, g.lineno, "<%s> deprecated", tag[g.tag].name);
>  		case Tag_pre:
> -		case Tag_samp:
>  			g.state->indent=0;
>  			g.state->pre=1;
>  			g.state->font=CWIDTH;
> @@ -1203,7 +1203,6 @@
>  		case Tag_listing:
>  		case Tag_menu:
>  		case Tag_ol:
> -		case Tag_samp:
>  		case Tag_title:
>  		case Tag_ul:
>  		case Tag_xmp:
> 

Re: [9front] mothra: fix rendering of <samp> tag

[Kristo]

Hi,

> Do you have an example site or html snippet that
> I can compare against?

It's actually a rather unused tag, but here's this small
html file from mozillas documentation:

https://yari-demos.prod.mdn.mozit.cloud/en-US/docs/Web/HTML/Element/samp/_sample_.Basic_example.html

My apologies, I should've included this in the first email.

Re: [9front] mothra: fix rendering of <samp> tag

[ori]

Quoth Kristo <kristo.ilmari@gmail.com>:
> Hi,
> 
> > Do you have an example site or html snippet that
> > I can compare against?
> 
> It's actually a rather unused tag, but here's this small
> html file from mozillas documentation:
> 
> https://yari-demos.prod.mdn.mozit.cloud/en-US/docs/Web/HTML/Element/samp/_sample_.Basic_example.html
> 
> My apologies, I should've included this in the first email.
> 

No worries -- looks good to me.