[pgsql-www] Code review of the presskits - Mailing list pgsql-www

From Daniel Gustafsson
Subject [pgsql-www] Code review of the presskits
Date
Msg-id EC1DDD74-95CC-4DD4-A9AC-E5372C35FF91@yesql.se
Whole thread Raw
Responses Re: [pgsql-www] Code review of the presskits
List pgsql-www
When looking at the 9.6 presskit I noticed some incorrect HTML, which led me to
code review the presskits we have for related issues.  The attached patch
includes the following fixes and should bring all presskits to valid XHTML:

* Multiple </a> tags closing a link

* Missing </p> tags leading to nested paragraphs on the next <p>, or one too
  many </p> closing a non-existing paragraph.  Also sometimes empty <p></p>
  blocks.

* Incorrectly spelled tags and entities

* Missing <h1> (due to the <h1> being in the wrong place)

* Updating to use https for links to postgresql.org

* Missing alt=“” on images and unclosed <img> tags

* Missing <h2> for the Graphics section, or the <h2> inside the <a> which is
  illegal in XHTML

* <li> inside a <p>, also without a leading <ul> or <ol>

It’s a rather large patch, but since it’s mainly just a lot of the same I kept
it a single commit.  I can split it up into multiple patches in case that would
aid review.

cheers ./daniel


-- 
Sent via pgsql-www mailing list (pgsql-www@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-www

Attachment

pgsql-www by date:

Previous
From: Umair Shahid
Date:
Subject: Re: [pgsql-www] Updating Admin Cookbook to 9.5/9.6 ediiton
Next
From: "David G. Johnston"
Date:
Subject: Re: [pgsql-www] [BUGS] Bug 14592