unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH 0/2] build-aux/list-packages.scm: fix HTML errors.
@ 2013-08-15 15:41 Cyril Roelandt
  2013-08-15 15:41 ` [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element Cyril Roelandt
  2013-08-15 15:41 ` [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element Cyril Roelandt
  0 siblings, 2 replies; 5+ messages in thread
From: Cyril Roelandt @ 2013-08-15 15:41 UTC (permalink / raw)
  To: guix-devel

These two (tivial) patches fix small errors in the HTML generated by the
build-aux/list-packages.scm script.

Regards,
Cyril Roelandt.
---
Cyril Roelandt (2):
  list-packages: remove deprecated "height" attribute on "td" element.
  list-packages: remove useless "language" attribute of "script" element

 build-aux/list-packages.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element.
  2013-08-15 15:41 [PATCH 0/2] build-aux/list-packages.scm: fix HTML errors Cyril Roelandt
@ 2013-08-15 15:41 ` Cyril Roelandt
  2013-08-15 16:41   ` Ludovic Courtès
  2013-08-15 15:41 ` [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element Cyril Roelandt
  1 sibling, 1 reply; 5+ messages in thread
From: Cyril Roelandt @ 2013-08-15 15:41 UTC (permalink / raw)
  To: guix-devel

See http://www.w3.org/TR/html5-diff/#obsolete-attributes for more information.

* build-aux/list-packages.html(package-sxml)[description-id]: remove "height"
  attribute for "td" elements.
---
 build-aux/list-packages.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-aux/list-packages.scm b/build-aux/list-packages.scm
index d060787..5ca2323 100755
--- a/build-aux/list-packages.scm
+++ b/build-aux/list-packages.scm
@@ -103,7 +103,7 @@ exec guile -l "$0"                              \
                   (title "Link to the Guix package source code"))
                ,(package-name package) " "
                ,(package-version package)))
-        (td (@ (colspan "2") (height "0"))
+        (td (@ (colspan "2"))
             (a (@ (href "javascript:void(0)")
                   (title "show/hide package description")
                   (onClick ,(format #f "javascript:show_hide('~a')"
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element
  2013-08-15 15:41 [PATCH 0/2] build-aux/list-packages.scm: fix HTML errors Cyril Roelandt
  2013-08-15 15:41 ` [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element Cyril Roelandt
@ 2013-08-15 15:41 ` Cyril Roelandt
  2013-08-15 16:42   ` Ludovic Courtès
  1 sibling, 1 reply; 5+ messages in thread
From: Cyril Roelandt @ 2013-08-15 15:41 UTC (permalink / raw)
  To: guix-devel

See http://www.w3.org/TR/html5-diff/#changed-attributes for more information.

* build-aux/list-packages.html (insert-js): remove "language" attribute, useless for the
* "script" element.
---
 build-aux/list-packages.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-aux/list-packages.scm b/build-aux/list-packages.scm
index 5ca2323..a1bea69 100755
--- a/build-aux/list-packages.scm
+++ b/build-aux/list-packages.scm
@@ -206,7 +206,7 @@ color:#fff;
 (define (insert-js)
   "Return the JavaScript for the list-packages page."
   (format #t
-"<script language=\"javascript\" type=\"text/javascript\">
+"<script type=\"text/javascript\">
 // license: CC0
 function show_hide(idThing)
 {
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element.
  2013-08-15 15:41 ` [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element Cyril Roelandt
@ 2013-08-15 16:41   ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2013-08-15 16:41 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> skribis:

> See http://www.w3.org/TR/html5-diff/#obsolete-attributes for more information.
>
> * build-aux/list-packages.html(package-sxml)[description-id]: remove "height"
>   attribute for "td" elements.

OK!

Ludo’.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element
  2013-08-15 15:41 ` [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element Cyril Roelandt
@ 2013-08-15 16:42   ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2013-08-15 16:42 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> skribis:

> See http://www.w3.org/TR/html5-diff/#changed-attributes for more information.
>
> * build-aux/list-packages.html (insert-js): remove "language" attribute, useless for the
> * "script" element.

(Extraneous star here.)

OK!

Ludo’.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-08-15 16:47 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-15 15:41 [PATCH 0/2] build-aux/list-packages.scm: fix HTML errors Cyril Roelandt
2013-08-15 15:41 ` [PATCH 1/2] list-packages: remove deprecated "height" attribute on "td" element Cyril Roelandt
2013-08-15 16:41   ` Ludovic Courtès
2013-08-15 15:41 ` [PATCH 2/2] list-packages: remove useless "language" attribute of "script" element Cyril Roelandt
2013-08-15 16:42   ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).