From: "Nícolas F. R. A. Prado" <nfraprado@collabora.com>
To: Jonathan Corbet <corbet@lwn.net>
Cc: linux-doc@vger.kernel.org, Mauro Carvalho Chehab <mchehab@kernel.org>
Subject: Re: [PATCH] docs: automarkup: Mark up undocumented entities too
Date: Tue, 3 Jun 2025 17:35:30 -0400 [thread overview]
Message-ID: <e6232bfc-092d-478c-b085-72bf4e9544f6@notapiano> (raw)
In-Reply-To: <87zfeojzr0.fsf@trenco.lwn.net>
On Tue, Jun 03, 2025 at 03:12:35PM -0600, Jonathan Corbet wrote:
> Nícolas F. R. A. Prado <nfraprado@collabora.com> writes:
>
> > To me the results look much better without these CSS rules, as they cause a
> > double underline.
> >
> > The current CSS already adds a dotted underline to reference links through the
> > following rule:
> >
> > a.reference {
> > border-bottom: 1px dotted #004B6B;
> > }
>
> OK, that is interesting ... I don't see that underline.
>
> Are you using the (default) alabaster theme? Alabaster explicitly sets
> it to "none", as can be seen on docs.kernel.org.
Yes. And I also see this same dotted underline on docs.kernel.org, for every URL
on that page. I've also double-checked this is the case when accessing from my
phone, and in incognito, so maybe this is something on your end?
To be clear, you don't see underlines on any URLs on docs.kernel.org?
You could find the CSS rule I mentioned above in
https://docs.kernel.org/_static/alabaster.css
>
> > So when you add this underline text-decoration to the .xref tags, the ones
> > inside <a> tags (valid xrefs) end up with two underlines.
> >
> > I've checked the result for both struct and functions and they work the same.
> >
> > So I suggest just dropping these CSS rules.
>
> We need to figure out why you are seeing something different. But I do
> want rules to distinguish just-plain-function from
> function-with-kerneldoc.
Maybe I wasn't clear, but on my end they are already rendered differently with
your change in automarkup.py, but without the CSS change. Both show up as bold
monospaced texts, but only in the case where the link is valid is there a <a>
tag, so only that one gets this dotted underline. When the xref doesn't exist
there's no underline.
Thanks,
Nícolas
next prev parent reply other threads:[~2025-06-03 21:35 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-03 17:20 [PATCH] docs: automarkup: Mark up undocumented entities too Jonathan Corbet
2025-06-03 19:44 ` Nícolas F. R. A. Prado
2025-06-03 21:12 ` Jonathan Corbet
2025-06-03 21:35 ` Nícolas F. R. A. Prado [this message]
2025-06-03 22:47 ` Jonathan Corbet
2025-06-03 23:04 ` Jonathan Corbet
2025-06-04 7:37 ` Mauro Carvalho Chehab
2025-06-04 13:19 ` Nícolas F. R. A. Prado
2025-06-04 14:22 ` Jonathan Corbet
2025-06-03 21:27 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e6232bfc-092d-478c-b085-72bf4e9544f6@notapiano \
--to=nfraprado@collabora.com \
--cc=corbet@lwn.net \
--cc=linux-doc@vger.kernel.org \
--cc=mchehab@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).