GNU bug report logs - #53251
28.0.90; [PATCH] GNU ELPA: Add margin to 'pre' element

Previous Next

Package: emacs;

Reported by: "Y. E." <yet <at> ego.team>

Date: Fri, 14 Jan 2022 11:59:01 UTC

Severity: normal

Tags: patch

Found in version 28.0.90

Fixed in version 29.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 53251 in the body.
You can then email your comments to 53251 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#53251; Package emacs. (Fri, 14 Jan 2022 11:59:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to "Y. E." <yet <at> ego.team>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Fri, 14 Jan 2022 11:59:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Y. E. <yet <at> ego.team>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.90; [PATCH] GNU ELPA: Add margin to 'pre' element
Date: Fri, 14 Jan 2022 13:58:16 +0200
[Message part 1 (text/plain, inline)]
Minor style change to improve packages 'Full Description' output. Few of
the influenced pages:

- http://elpa.gnu.org/packages/cape.html
- http://elpa.gnu.org/packages/consult.html
- http://elpa.gnu.org/packages/corfu.html
- http://elpa.gnu.org/packages/csharp-mode.html
- http://elpa.gnu.org/packages/devdocs.html
- http://elpa.gnu.org/packages/embark.html
- http://elpa.gnu.org/packages/embark-consult.html

Among the pages I checked (many but not all), none was influenced
negatively.

[0001-html-layout.css-pre-Add-margin-bottom.patch (text/x-patch, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#53251; Package emacs. (Sun, 23 Jan 2022 12:32:02 GMT) Full text and rfc822 format available.

Message #8 received at 53251 <at> debbugs.gnu.org (full text, mbox):

From: Philip Kaludercic <philipk <at> posteo.net>
To: Y. E. <yet <at> ego.team>
Cc: 53251 <at> debbugs.gnu.org
Subject: Re: bug#53251: 28.0.90; [PATCH] GNU ELPA: Add margin to 'pre' element
Date: Sun, 23 Jan 2022 12:31:05 +0000
Y. E. <yet <at> ego.team> writes:

> Minor style change to improve packages 'Full Description' output. Few of
> the influenced pages:
>
> - http://elpa.gnu.org/packages/cape.html
> - http://elpa.gnu.org/packages/consult.html
> - http://elpa.gnu.org/packages/corfu.html
> - http://elpa.gnu.org/packages/csharp-mode.html
> - http://elpa.gnu.org/packages/devdocs.html
> - http://elpa.gnu.org/packages/embark.html
> - http://elpa.gnu.org/packages/embark-consult.html
>
> Among the pages I checked (many but not all), none was influenced
> negatively.

All these packages use org-mode READMEs, that are converted to HTML,
right?  To avoid unintended side-effects, it might be better to only
apply this to "pre.src", not all "pre" blocks.

-- 
	Philip Kaludercic




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#53251; Package emacs. (Mon, 24 Jan 2022 16:21:01 GMT) Full text and rfc822 format available.

Message #11 received at 53251 <at> debbugs.gnu.org (full text, mbox):

From: Y. E. <yet <at> ego.team>
To: Philip Kaludercic <philipk <at> posteo.net>
Cc: yet <at> ego.team, 53251 <at> debbugs.gnu.org
Subject: Re: bug#53251: 28.0.90; [PATCH] GNU ELPA: Add margin to 'pre' element
Date: Mon, 24 Jan 2022 18:20:49 +0200
[Message part 1 (text/plain, inline)]
Philip Kaludercic <philipk <at> posteo.net> writes:

> Y. E. <yet <at> ego.team> writes:
>
>> Minor style change to improve packages 'Full Description' output. Few of
>> the influenced pages:
>>
>> - http://elpa.gnu.org/packages/cape.html
>> - http://elpa.gnu.org/packages/consult.html
>> - http://elpa.gnu.org/packages/corfu.html
>> - http://elpa.gnu.org/packages/csharp-mode.html
>> - http://elpa.gnu.org/packages/devdocs.html
>> - http://elpa.gnu.org/packages/embark.html
>> - http://elpa.gnu.org/packages/embark-consult.html
>>
>> Among the pages I checked (many but not all), none was influenced
>> negatively.
>
> All these packages use org-mode READMEs, that are converted to HTML,
> right?  To avoid unintended side-effects, it might be better to only
> apply this to "pre.src", not all "pre" blocks.

Sounds good to me. I'm attaching the patch with this change.

[0001-html-layout.css-pre.src-Add-margin-bottom.patch (text/x-patch, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#53251; Package emacs. (Thu, 08 Sep 2022 13:53:02 GMT) Full text and rfc822 format available.

Message #14 received at 53251 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Y. E. <yet <at> ego.team>
Cc: Philip Kaludercic <philipk <at> posteo.net>, 53251 <at> debbugs.gnu.org
Subject: Re: bug#53251: 28.0.90; [PATCH] GNU ELPA: Add margin to 'pre' element
Date: Thu, 08 Sep 2022 15:52:33 +0200
Y. E. <yet <at> ego.team> writes:

> Sounds good to me. I'm attaching the patch with this change.

Thanks; pushed to the GNU web pages now.





bug marked as fixed in version 29.1, send any further explanations to 53251 <at> debbugs.gnu.org and "Y. E." <yet <at> ego.team> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Thu, 08 Sep 2022 13:53:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 07 Oct 2022 11:24:10 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 173 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.