Hello, this patch is fixing a bug.
Can it be merged please?
16/08/2024 19:02, Robin Jarry:
> Thomas Monjalon, Aug 16, 2024 at 09:26:
> > Preformatted content must not be indented
> > because any space is kept in the output,
> > making the content wrongly indented.
> >
> > When aligning message headers to the left,
> > the new HTML code has been indented
> > including some preformatted content indented with two spaces.
> >
> > The fix is to remove the indent of the content.
> >
> > Fixes: fe34ab2ffad3 ("patch-detail: left align message headers")
>
> This should be grouped with your signed-off-by trailer without a blank
> line in between so that git-interpret-trailers(1) can parse it.
>
> >
> > Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
> > ---
> > patchwork/templates/patchwork/submission.html | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/patchwork/templates/patchwork/submission.html
> > b/patchwork/templates/patchwork/submission.html
> > index 85e7be4b..e924934f 100644
> > --- a/patchwork/templates/patchwork/submission.html
> > +++ b/patchwork/templates/patchwork/submission.html
> > @@ -262,7 +262,7 @@
> > <span class="message-date">{{ submission.date }} UTC</span>
> > </div>
> > <pre class="content">
> > - {{ submission|commentsyntax }}
> > +{{ submission|commentsyntax }}
> > </pre>
> > </div>
> >
>
> Hi Thomas, I had noticed this but thought it was a CSS issue.
>
> Looks good to me!
>
> Reviewed-by: Robin Jarry <ro...@jarry.cc>
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork
