public inbox for linux-doc@vger.kernel.org
 help / color / mirror / Atom feed
From: "Rito Rhymes" <rito@ritovision.com>
To: "Jonathan Corbet" <corbet@lwn.net>,
	"Rito Rhymes" <rito@ritovision.com>,
	"Mauro Carvalho Chehab" <mchehab@kernel.org>,
	<linux-doc@vger.kernel.org>
Cc: "Shuah Khan" <skhan@linuxfoundation.org>,
	<linux-kernel@vger.kernel.org>, <rdunlap@infradead.org>
Subject: Re: [PATCH v3] docs: wrap generated tables to contain small-screen overflow
Date: Wed, 25 Mar 2026 22:38:53 -0400	[thread overview]
Message-ID: <DHCD6BWJTLTB.VJWKPRWC2GHZ@ritovision.com> (raw)
In-Reply-To: <87mrzvg2qx.fsf@trenco.lwn.net>

> So this CSS perhaps makes sense, but..
> But why do you need to inject another <div>, creating a whole
> new extension to do so, rather than just applying the CSS
> directly to the <table> elements? I just gave that a try, and
> it would appear to work just fine.

I did try applying it directly to the <table> elements.

That works on some pages, but in my testing it introduces
regressions on others that do not happen with the wrapper
approach.

For example, on `/bpf/standardization/instruction-set.html` at
400px viewport width, applying CSS scrollability directly to
the table creates an extra outer border with an awkward gap and,
more importantly, causes cell content to wrap much more
aggressively as the viewport narrows. Instead of letting the
table remain a readable horizontally scrollable unit, the text
gets squeezed into very narrow stacked lines that are harder
to read.

Screenshots:
Double border with gap issue highlighted -
https://github.com/user-attachments/assets/0ddacc4e-edce-4c14-b129-0940e8014d3f
Text wrapping issue highlighted -
https://github.com/user-attachments/assets/c172e126-7a4c-4b3c-a01c-fd4c8f08bda6

On `/arch/arc/features.html`, the wrapping issue is not present
and it scrolls fine, but I still see the double-border issue
there. In that case there is no gap, so it mainly looks like a
thicker border.

The reason for the wrapper is that it gives one consistent
overflow treatment across the existing table cases without
changing the table element's layout behavior directly. In my
testing, applying the scrollability to the <table> itself
works for some pages but regresses others, while the wrapper
approach avoids needing targeted CSS rules for different
rendered table cases.

If helpful, you're welcome to add any screenshots as a
comment on this GitHub issue:
https://github.com/ritovision/linux-kernel-docs/issues/1

Rito

      reply	other threads:[~2026-03-26  2:39 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-21 13:38 [PATCH 0/1] docs: examples of pages affected by table overflow Rito Rhymes
2026-03-21 13:38 ` [PATCH 1/1] docs: wrap generated tables to contain small-screen overflow Rito Rhymes
2026-03-21 19:17   ` Randy Dunlap
2026-03-22 19:24 ` [PATCH v2 0/1] docs: examples of pages affected by table overflow Rito Rhymes
2026-03-22 19:24   ` [PATCH v2 1/1] docs: wrap generated tables to contain small-screen overflow Rito Rhymes
2026-03-23 15:37 ` [PATCH v3] " Rito Rhymes
2026-03-25 18:53   ` Jonathan Corbet
2026-03-26  2:38     ` Rito Rhymes [this message]

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=DHCD6BWJTLTB.VJWKPRWC2GHZ@ritovision.com \
    --to=rito@ritovision.com \
    --cc=corbet@lwn.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=skhan@linuxfoundation.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