From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stepnem@smrk.net>
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net
X-Spam-Level: 
X-Spam-ASN: AS20473 45.76.80.0/20
X-Spam-Status: No, score=-3.2 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS
	shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6
Received: from mail.smrk.net (mail.smrk.net [45.76.87.244])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
	 key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA512)
	(No client certificate requested)
	by dcvr.yhbt.net (Postfix) with ESMTPS id 32FBA1F5CB
	for <meta@public-inbox.org>; Sat, 28 Sep 2024 19:29:15 +0000 (UTC)
Authentication-Results: dcvr.yhbt.net;
	dkim=pass (2048-bit key; unprotected) header.d=smrk.net header.i=@smrk.net header.a=rsa-sha256 header.s=20221002 header.b=LgeDIX8+;
	dkim-atps=neutral
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=smrk.net; s=20221002;
	t=1727551753;
	h=from:from:reply-to:subject:subject:date:date:message-id:message-id:
	 to:to:cc:cc:mime-version:mime-version:content-type:content-type:
	 content-transfer-encoding:content-transfer-encoding:
	 in-reply-to:in-reply-to:references:references;
	bh=lwVhIYoUhctGqGUgBK5nSpeVOr2AXlNbVW2WB3kbzmM=;
	b=LgeDIX8+FBisoHfAwjktvL6G1k00qK3kaba6QLnhr41F3ZcFPb1pIgxHQkT2qCrgoK1LXz
	1lxlPUgfxmyaJmaZ9XJWy6rTXnK0ldt3rVrdqvW4LZV0idoP5paXOZZdUVR+Z4j85iFNNP
	dq/BCoioO/XEatVCIlOUli/TF5TW/z9zj4RHNvHhw23rAEY1RGRQX+kXWgk3TxmdyE2r6I
	Kp3elOam6+2ReuEyJJWZmIsJLymYfZDdcclUccJsi6pkvic9VIywgOxHqfQ1rvqFF65P+r
	h0m7+y8+wsKFfRpzOyyhonzEHq2WzVwOEP029OOp9evlbxzo1OH/S9pJIHHG9w==
Received: from localhost (<unknown> [192.168.5.2])
	by smrk (OpenSMTPD) with ESMTPSA id 41e3eb23 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO);
	Sat, 28 Sep 2024 21:29:13 +0200 (CEST)
From: =?utf-8?B?xaB0xJtww6FuIE7Em21lYw==?= <stepnem@smrk.net>
To: Eric Wong <e@80x24.org>
Cc: meta@public-inbox.org,  "Robin H. Johnson" <robbat2@gentoo.org>
Subject: Re: [PATCH v2] www: allow specifying CSS @import or <link> tags
In-Reply-To: <20240928183935.M960376@dcvr> (Eric Wong's message of "Sat, 28
	Sep 2024 18:39:35 +0000")
References: <robbat2-20240901T045635-169490258Z@orbis-terrarum.net>
	<20240926005506.3703216-1-e@80x24.org>
	<20240926005506.3703216-5-e@80x24.org>
	<20240926183402Z.161922298-stepnem@smrk.net>
	<20240928183935.M960376@dcvr>
User-Agent: Gnus/5.13 (Gnus v5.13)
Date: Sat, 28 Sep 2024 21:29:13 +0200
Message-ID: <20240928192913Z.2177483813-stepnem@smrk.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
List-Id: <meta.public-inbox.org>

On Sat, 28 Sep 2024 18:39:35 +0000
Eric Wong wrote:

> diff --git a/Documentation/public-inbox-config.pod b/Documentation/public=
-inbox-config.pod
> index 50746b21..667f626f 100644
> --- a/Documentation/public-inbox-config.pod
> +++ b/Documentation/public-inbox-config.pod
> @@ -256,12 +256,34 @@ Default: :all
>  =3Ditem publicinbox.css
>=20=20
>  The local path name of a CSS file for the PSGI web interface.
> -May contain the attributes "media", "title" and "href" which match
> -the associated attributes of the HTML <style> tag.
> -"href" may be specified to point to the URL of a remote CSS file
> +Space-separated attributes (C<key=3Dval> or C<key=3D'val with spaces'>)
> +may be specified after the path on the same line; C<media>, C<title>
> +and C<href> are currently supported, and match the associated attributes
> +of the HTML C<E<lt>styleE<gt>> tag.
> +
> +C<href> may be specified to point to the URL of a remote CSS file
>  and the path may be "/dev/null" or any empty file.

This is indeed a better place for that sentence; the
displacement in v1 gave me a pause, but I didn't really
connect the dots...

> +public-inbox 2.0+ also supports a C<load> attribute with values
> +C<link> or C<import>; C<link> forces the use of a C<E<lt>linkE<gt>> tag
> +and disables inlining CSS into the C<E<lt>styleE<gt>> tag while
> +C<import> puts CSS C<@import> statements into the C<E<lt>styleE<gt>> tag.
> +C<import> appears necessary for some browser and C<media> query combinat=
ions.
> +
>  Multiple files may be specified and will be included in the
> -order specified.
> +order specified, but files specified via C<load=3Dimport> are
> +always handled first.

I guess saying "order given" or "given order" instead would
read better (avoid triple "specified" in a single sentence),
but that's just nit picking.

> +
> +Example:
> +
> +	[publicinbox]
> +		css =3D /path/to/216dark.css title=3D216dark \
> +		  media=3D'screen,(prefers-color-scheme:dark)'
> +
> +See the L<contrib/css
> +directory of the public-inbox
> +source|https://80x24.org/public-inbox.git/tree/contrib/css>
> +for more examples.

And an example is helpful (and I'd missed contrib/css, too).

Thanks,

  =C5=A0t=C4=9Bp=C3=A1n