GNU bug report logs -
#69815
[PATCH] Fix some CSS issues on elpa.gnu.org
Previous Next
Reported by: john muhl <jm <at> pub.pink>
Date: Fri, 15 Mar 2024 17:41:02 UTC
Severity: wishlist
Tags: patch
Done: Stefan Kangas <stefankangas <at> gmail.com>
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 69815 in the body.
You can then email your comments to 69815 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Fri, 15 Mar 2024 17:41:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
john muhl <jm <at> pub.pink>:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org.
(Fri, 15 Mar 2024 17:41:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Here are a few patches fixing some minor issues. The first one
makes the packages table always full-width and expands the footer
background so it contains all the footer text (see before.png).
I noticed that with a very narrow view the CSS drops the rank
column to help fit but it doesn’t help quite enough to prevent
horizontal overflow (narrow.png). It seems to me that the version
column is the least important so the second patch changes the
narrow view to drop it instead of the rank (after.png).
The last patch fixes the doctype for sub-pages. <!DOCTYPE HTML
PUBLIC> is not a valid doctype and so triggers quirks mode. Not a
big deal but probably better avoided.
[0001-Fix-some-CSS-issues.patch (text/x-patch, attachment)]
[0001-Show-rank-instead-of-version-on-narrow-displays.patch (text/x-patch, attachment)]
[0001-elpa-admin.el-elpaa-html-header-Fix-DOCTYPE.patch (text/x-patch, attachment)]
[before.png (image/png, attachment)]
[narrow.png (image/png, attachment)]
[after.png (image/png, attachment)]
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Sun, 17 Mar 2024 10:58:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 69815 <at> debbugs.gnu.org (full text, mbox):
john muhl <jm <at> pub.pink> writes:
> Here are a few patches fixing some minor issues. The first one
> makes the packages table always full-width and expands the footer
> background so it contains all the footer text (see before.png).
>
> I noticed that with a very narrow view the CSS drops the rank
> column to help fit but it doesn’t help quite enough to prevent
> horizontal overflow (narrow.png). It seems to me that the version
> column is the least important so the second patch changes the
> narrow view to drop it instead of the rank (after.png).
>
> The last patch fixes the doctype for sub-pages. <!DOCTYPE HTML
> PUBLIC> is not a valid doctype and so triggers quirks mode. Not a
> big deal but probably better avoided.
>
>>From f13cc711f0bf368f9acb1ae579ab9e078aefce1e Mon Sep 17 00:00:00 2001
> From: john muhl <jm <at> pub.pink>
> Date: Thu, 14 Mar 2024 13:22:52 -0500
> Subject: [PATCH] Fix some CSS issues
>
> * html/layout.css (footer): Remove fixed height so the background
> expands to encompass all footer content.
> (table): Make packages table always use full page width.
> ---
> html/layout.css | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/html/layout.css b/html/layout.css
> index 7d3be41..9d04958 100644
> --- a/html/layout.css
> +++ b/html/layout.css
> @@ -1,4 +1,4 @@
> -l/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
> +/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
>
> This file is free software: you can redistribute it and/or modify
> it under the terms of the GNU General Public License as published by
> @@ -74,7 +74,7 @@ html, body {
> height: 100%;
> margin: 0 auto -50px; /* the bottom margin is the negative value of the footer's height */
> }
> -footer, .push, .footer {
> +.push {
> height: 90px; /* .push must be the same height as .footer */
> }
>
> @@ -302,6 +302,10 @@ footer p, .footer p {
> line-height: 1.4em;
> }
>
> +table {
> + width: 100%;
> +}
> +
> @media screen and (max-width: 999px) {
> .container {
> width: 600px;
> @@ -313,10 +317,6 @@ footer p, .footer p {
> font-size: 1em;
> }
>
> - table {
> - width: 100%;
> - }
> -
> .container {
> width: 90%;
> padding: 40px 5%;
> --
> 2.41.0
Looks good.
>
>>From 5dafd0d3483336ff50b5039942bc35c41d30644f Mon Sep 17 00:00:00 2001
> From: john muhl <jm <at> pub.pink>
> Date: Thu, 14 Mar 2024 13:46:54 -0500
> Subject: [PATCH] Show rank instead of version on narrow displays
>
> * html/layout.css (td):
> (th): Hide version and right align rank column.
> ---
> html/layout.css | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/html/layout.css b/html/layout.css
> index 9d04958..02dfb87 100644
> --- a/html/layout.css
> +++ b/html/layout.css
> @@ -283,6 +283,11 @@ td {
> border-bottom: 1px solid #ddd;
> }
>
> +td:last-child,
> +th:last-child {
> + text-align: right;
> +}
> +
> footer, .footer {
> border-top: 1px solid #999;
> text-align:center;
> @@ -330,7 +335,7 @@ table {
> width: 300px;
> }
>
> - td:last-child, th:last-child {
> + td:nth-child(2), th:nth-child(2) {
> display: none;
> }
>
> --
> 2.41.0
Why not show both, and allow scrolling horizontally (x-overflow: auto)
if there is not enough space?
>
>
>>From 16547922cc9339e4f920ae62a1bee09c38625cbf Mon Sep 17 00:00:00 2001
> From: john muhl <jm <at> pub.pink>
> Date: Thu, 14 Mar 2024 16:54:45 -0500
> Subject: [PATCH] * elpa-admin.el (elpaa--html-header): Fix DOCTYPE.
>
> ---
> elpa-admin.el | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/elpa-admin.el b/elpa-admin.el
> index d4a9dcf..9475336 100644
> --- a/elpa-admin.el
> +++ b/elpa-admin.el
> @@ -1544,7 +1544,7 @@ readme file has an unconventional name"
> ;;; Make the HTML pages for online browsing.
>
> (defun elpaa--html-header (title &optional header head-extra)
> - (format "<!DOCTYPE HTML PUBLIC>
> + (format "<!DOCTYPE HTML>
What difference does this make? Or rather how is this related to the
other patches? That should be explained in the commit message.
> <html lang=\"en\" xml:lang=\"en\">
> <head>
> <title>%s</title>
--
Philip Kaludercic on peregrine
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Mon, 18 Mar 2024 02:34:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 69815 <at> debbugs.gnu.org (full text, mbox):
Philip Kaludercic <philipk <at> posteo.net> writes:
> john muhl <jm <at> pub.pink> writes:
>
>> Here are a few patches fixing some minor issues. The first one
>> makes the packages table always full-width and expands the footer
>> background so it contains all the footer text (see before.png).
>>
>> I noticed that with a very narrow view the CSS drops the rank
>> column to help fit but it doesn’t help quite enough to prevent
>> horizontal overflow (narrow.png). It seems to me that the version
>> column is the least important so the second patch changes the
>> narrow view to drop it instead of the rank (after.png).
>>
>> The last patch fixes the doctype for sub-pages. <!DOCTYPE HTML
>> PUBLIC> is not a valid doctype and so triggers quirks mode. Not a
>> big deal but probably better avoided.
>>
>>>From f13cc711f0bf368f9acb1ae579ab9e078aefce1e Mon Sep 17 00:00:00 2001
>> From: john muhl <jm <at> pub.pink>
>> Date: Thu, 14 Mar 2024 13:22:52 -0500
>> Subject: [PATCH] Fix some CSS issues
>>
>> * html/layout.css (footer): Remove fixed height so the background
>> expands to encompass all footer content.
>> (table): Make packages table always use full page width.
>> ---
>> html/layout.css | 12 ++++++------
>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/html/layout.css b/html/layout.css
>> index 7d3be41..9d04958 100644
>> --- a/html/layout.css
>> +++ b/html/layout.css
>> @@ -1,4 +1,4 @@
>> -l/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
>> +/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
>>
>> This file is free software: you can redistribute it and/or modify
>> it under the terms of the GNU General Public License as published by
>> @@ -74,7 +74,7 @@ html, body {
>> height: 100%;
>> margin: 0 auto -50px; /* the bottom margin is the negative value of the footer's height */
>> }
>> -footer, .push, .footer {
>> +.push {
>> height: 90px; /* .push must be the same height as .footer */
>> }
>>
>> @@ -302,6 +302,10 @@ footer p, .footer p {
>> line-height: 1.4em;
>> }
>>
>> +table {
>> + width: 100%;
>> +}
>> +
>> @media screen and (max-width: 999px) {
>> .container {
>> width: 600px;
>> @@ -313,10 +317,6 @@ footer p, .footer p {
>> font-size: 1em;
>> }
>>
>> - table {
>> - width: 100%;
>> - }
>> -
>> .container {
>> width: 90%;
>> padding: 40px 5%;
>> --
>> 2.41.0
>
> Looks good.
>
>>
>>>From 5dafd0d3483336ff50b5039942bc35c41d30644f Mon Sep 17 00:00:00 2001
>> From: john muhl <jm <at> pub.pink>
>> Date: Thu, 14 Mar 2024 13:46:54 -0500
>> Subject: [PATCH] Show rank instead of version on narrow displays
>>
>> * html/layout.css (td):
>> (th): Hide version and right align rank column.
>> ---
>> html/layout.css | 7 ++++++-
>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/html/layout.css b/html/layout.css
>> index 9d04958..02dfb87 100644
>> --- a/html/layout.css
>> +++ b/html/layout.css
>> @@ -283,6 +283,11 @@ td {
>> border-bottom: 1px solid #ddd;
>> }
>>
>> +td:last-child,
>> +th:last-child {
>> + text-align: right;
>> +}
>> +
>> footer, .footer {
>> border-top: 1px solid #999;
>> text-align:center;
>> @@ -330,7 +335,7 @@ table {
>> width: 300px;
>> }
>>
>> - td:last-child, th:last-child {
>> + td:nth-child(2), th:nth-child(2) {
>> display: none;
>> }
>>
>> --
>> 2.41.0
>
>
> Why not show both, and allow scrolling horizontally (x-overflow: auto)
> if there is not enough space?
>
>>
>>
>>>From 16547922cc9339e4f920ae62a1bee09c38625cbf Mon Sep 17 00:00:00 2001
>> From: john muhl <jm <at> pub.pink>
>> Date: Thu, 14 Mar 2024 16:54:45 -0500
>> Subject: [PATCH] * elpa-admin.el (elpaa--html-header): Fix DOCTYPE.
>>
>> ---
>> elpa-admin.el | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/elpa-admin.el b/elpa-admin.el
>> index d4a9dcf..9475336 100644
>> --- a/elpa-admin.el
>> +++ b/elpa-admin.el
>> @@ -1544,7 +1544,7 @@ readme file has an unconventional name"
>> ;;; Make the HTML pages for online browsing.
>>
>> (defun elpaa--html-header (title &optional header head-extra)
>> - (format "<!DOCTYPE HTML PUBLIC>
>> + (format "<!DOCTYPE HTML>
>
> What difference does this make? Or rather how is this related to the
> other patches? That should be explained in the commit message.
It’s unrelated. I just included it since it looked like a mistake.
If those pages require quirks mode then a better fix would be get
rid of that requirement. Checking now I see that they serve
non-conforming HTML so perhaps we should fix that first.
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Fri, 22 Mar 2024 13:11:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 69815 <at> debbugs.gnu.org (full text, mbox):
I looked into it some more and at least some of the invalid HTML
is generated by Org’s HTML export but they don’t consider that a
bug so I guess quirks mode is here to stay and the DOCTYPE patch
can be discarded.
john muhl <jm <at> pub.pink> writes:
> Philip Kaludercic <philipk <at> posteo.net> writes:
>
>> john muhl <jm <at> pub.pink> writes:
>>
>>> Here are a few patches fixing some minor issues. The first one
>>> makes the packages table always full-width and expands the footer
>>> background so it contains all the footer text (see before.png).
>>>
>>> I noticed that with a very narrow view the CSS drops the rank
>>> column to help fit but it doesn’t help quite enough to prevent
>>> horizontal overflow (narrow.png). It seems to me that the version
>>> column is the least important so the second patch changes the
>>> narrow view to drop it instead of the rank (after.png).
>>>
>>> The last patch fixes the doctype for sub-pages. <!DOCTYPE HTML
>>> PUBLIC> is not a valid doctype and so triggers quirks mode. Not a
>>> big deal but probably better avoided.
>>>
>>>>From f13cc711f0bf368f9acb1ae579ab9e078aefce1e Mon Sep 17 00:00:00 2001
>>> From: john muhl <jm <at> pub.pink>
>>> Date: Thu, 14 Mar 2024 13:22:52 -0500
>>> Subject: [PATCH] Fix some CSS issues
>>>
>>> * html/layout.css (footer): Remove fixed height so the background
>>> expands to encompass all footer content.
>>> (table): Make packages table always use full page width.
>>> ---
>>> html/layout.css | 12 ++++++------
>>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/html/layout.css b/html/layout.css
>>> index 7d3be41..9d04958 100644
>>> --- a/html/layout.css
>>> +++ b/html/layout.css
>>> @@ -1,4 +1,4 @@
>>> -l/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
>>> +/* Copyright (C) 2010-2022, 2024 Free Software Foundation, Inc.
>>>
>>> This file is free software: you can redistribute it and/or modify
>>> it under the terms of the GNU General Public License as published by
>>> @@ -74,7 +74,7 @@ html, body {
>>> height: 100%;
>>> margin: 0 auto -50px; /* the bottom margin is the negative value of the footer's height */
>>> }
>>> -footer, .push, .footer {
>>> +.push {
>>> height: 90px; /* .push must be the same height as .footer */
>>> }
>>>
>>> @@ -302,6 +302,10 @@ footer p, .footer p {
>>> line-height: 1.4em;
>>> }
>>>
>>> +table {
>>> + width: 100%;
>>> +}
>>> +
>>> @media screen and (max-width: 999px) {
>>> .container {
>>> width: 600px;
>>> @@ -313,10 +317,6 @@ footer p, .footer p {
>>> font-size: 1em;
>>> }
>>>
>>> - table {
>>> - width: 100%;
>>> - }
>>> -
>>> .container {
>>> width: 90%;
>>> padding: 40px 5%;
>>> --
>>> 2.41.0
>>
>> Looks good.
>>
>>>
>>>>From 5dafd0d3483336ff50b5039942bc35c41d30644f Mon Sep 17 00:00:00 2001
>>> From: john muhl <jm <at> pub.pink>
>>> Date: Thu, 14 Mar 2024 13:46:54 -0500
>>> Subject: [PATCH] Show rank instead of version on narrow displays
>>>
>>> * html/layout.css (td):
>>> (th): Hide version and right align rank column.
>>> ---
>>> html/layout.css | 7 ++++++-
>>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/html/layout.css b/html/layout.css
>>> index 9d04958..02dfb87 100644
>>> --- a/html/layout.css
>>> +++ b/html/layout.css
>>> @@ -283,6 +283,11 @@ td {
>>> border-bottom: 1px solid #ddd;
>>> }
>>>
>>> +td:last-child,
>>> +th:last-child {
>>> + text-align: right;
>>> +}
>>> +
>>> footer, .footer {
>>> border-top: 1px solid #999;
>>> text-align:center;
>>> @@ -330,7 +335,7 @@ table {
>>> width: 300px;
>>> }
>>>
>>> - td:last-child, th:last-child {
>>> + td:nth-child(2), th:nth-child(2) {
>>> display: none;
>>> }
>>>
>>> --
>>> 2.41.0
>>
>>
>> Why not show both, and allow scrolling horizontally (x-overflow: auto)
>> if there is not enough space?
>>
>>>
>>>
>>>>From 16547922cc9339e4f920ae62a1bee09c38625cbf Mon Sep 17 00:00:00 2001
>>> From: john muhl <jm <at> pub.pink>
>>> Date: Thu, 14 Mar 2024 16:54:45 -0500
>>> Subject: [PATCH] * elpa-admin.el (elpaa--html-header): Fix DOCTYPE.
>>>
>>> ---
>>> elpa-admin.el | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/elpa-admin.el b/elpa-admin.el
>>> index d4a9dcf..9475336 100644
>>> --- a/elpa-admin.el
>>> +++ b/elpa-admin.el
>>> @@ -1544,7 +1544,7 @@ readme file has an unconventional name"
>>> ;;; Make the HTML pages for online browsing.
>>>
>>> (defun elpaa--html-header (title &optional header head-extra)
>>> - (format "<!DOCTYPE HTML PUBLIC>
>>> + (format "<!DOCTYPE HTML>
>>
>> What difference does this make? Or rather how is this related to the
>> other patches? That should be explained in the commit message.
>
> It’s unrelated. I just included it since it looked like a mistake.
> If those pages require quirks mode then a better fix would be get
> rid of that requirement. Checking now I see that they serve
> non-conforming HTML so perhaps we should fix that first.
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Fri, 22 Mar 2024 15:11:01 GMT)
Full text and
rfc822 format available.
Message #17 received at 69815 <at> debbugs.gnu.org (full text, mbox):
john muhl <jm <at> pub.pink> writes:
> I looked into it some more and at least some of the invalid HTML
> is generated by Org’s HTML export but they don’t consider that a
> bug so I guess quirks mode is here to stay and the DOCTYPE patch
> can be discarded.
I think that could be something worth reporting to the org maintainers,
unless you have already done so?
--
Philip Kaludercic on peregrine
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#69815; Package
emacs.
(Fri, 22 Mar 2024 16:01:02 GMT)
Full text and
rfc822 format available.
Message #20 received at 69815 <at> debbugs.gnu.org (full text, mbox):
Philip Kaludercic <philipk <at> posteo.net> writes:
> john muhl <jm <at> pub.pink> writes:
>
>> I looked into it some more and at least some of the invalid HTML
>> is generated by Org’s HTML export but they don’t consider that a
>> bug so I guess quirks mode is here to stay and the DOCTYPE patch
>> can be discarded.
>
> I think that could be something worth reporting to the org maintainers,
> unless you have already done so?
I asked. It’s caused by authors not understanding how the
‘#+html:’ construct works and using to attempt to hide things in
the HTML output like:
#+html: <!--
more org stuff here
#+html: -->
https://lists.gnu.org/archive/html/emacs-orgmode/2024-03/msg00520.html
Severity set to 'wishlist' from 'normal'
Request was from
Stefan Kangas <stefankangas <at> gmail.com>
to
control <at> debbugs.gnu.org.
(Tue, 11 Feb 2025 19:44:02 GMT)
Full text and
rfc822 format available.
Reply sent
to
Stefan Kangas <stefankangas <at> gmail.com>:
You have taken responsibility.
(Tue, 11 Feb 2025 19:50:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
john muhl <jm <at> pub.pink>:
bug acknowledged by developer.
(Tue, 11 Feb 2025 19:50:02 GMT)
Full text and
rfc822 format available.
Message #27 received at 69815-done <at> debbugs.gnu.org (full text, mbox):
john muhl <jm <at> pub.pink> writes:
> Here are a few patches fixing some minor issues. The first one
> makes the packages table always full-width and expands the footer
> background so it contains all the footer text (see before.png).
>
> I noticed that with a very narrow view the CSS drops the rank
> column to help fit but it doesn’t help quite enough to prevent
> horizontal overflow (narrow.png). It seems to me that the version
> column is the least important so the second patch changes the
> narrow view to drop it instead of the rank (after.png).
>
> The last patch fixes the doctype for sub-pages. <!DOCTYPE HTML
> PUBLIC> is not a valid doctype and so triggers quirks mode. Not a
> big deal but probably better avoided.
It seems like we agreed to install these patches, except for the one
changing the DOCTYPE. So I did that, and I'm consequently closing this
bug.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org.
(Wed, 12 Mar 2025 11:24:14 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 day ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.