Re: [PATCH 3/5] gitweb: fix mobile page overflow across log/commit/blob/diff views

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Feb 15, 2026, 23:20 UTC
Message-ID: <CAPig+cSb=zzDJCoo91detBgfAi9p4a3R8sVc6iTXfzpbNxsNrg@mail.gmail.com>
In-Reply-To: <7b785b6064966aefe6883ce3c45e80bb74fe79f4.1770679038.git.gitgitgadget@gmail.com>

On Mon, Feb 9, 2026 at 6:17 PM Rito Rhymes via GitGitGadget <gitgitgadget@gmail.com> wrote:

Show 24 quoted lines

> On mobile-sized viewports, gitweb pages in log/commit/blob/diff views can
> overflow horizontally due to desktop-oriented paddings and fixed-width
> preformatted content.
>
> Add a shared mobile media query to rebalance those layouts: reduce or clear
> paddings in log/commit sections, keep header/search content within the
> viewport, and allow horizontal scrolling for preformatted blob/diff content
> instead of forcing page-wide overflow.
>
> Signed-off-by: Rito Rhymes <rito@ritovision.com>
> ---
> diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
> @@ -537,13 +538,6 @@ div.search {
>  div.projsearch {
>         text-align: center;
>         margin: 20px 0px;
> -       padding: 0 8px;
> -       box-sizing: border-box;
> -}
> -
> -div.projsearch input[type="text"] {
> -       max-width: 100%;
> -       box-sizing: border-box;
>  }

These lines were all added by patch [2/5], applying to all viewport sizes, but here in patch [3/5]...

Show 11 quoted lines

> @@ -691,3 +685,66 @@ div.remote {
> +@media (max-width: 768px) {
> +       div.projsearch {
> +               padding: 0 8px;
> +               box-sizing: border-box;
> +       }
> +
> +       div.projsearch input[type="text"] {
> +               max-width: 100%;
> +               box-sizing: border-box;
> +       }

...they are relocated to this @media query. It seems a bit odd to add the lines in one patch and then immediately relocate them in the next patch, and it's not clear why the series is constructed this way. I could, perhaps, understand having separate patches like this if the idea is to specially call out the existing "broken" behavior, but considering that the commit message of both patches talk about restricting the width of the input field to fit the viewport, I'm having trouble understanding why the patches are separate.