Opened 6 years ago

Closed 6 years ago

#3610 closed defect (bug) (fixed)

Archives & Categories should have list items even if empty

Reported by: nickshanks Owned by: anonymous
Priority: highest omg bbq Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: has-patch
Cc:

Description

Strict doctypes disallow <ul>No categories</ul> should be <ul><li>No categories</li></ul>
Same with archives.

Attachments (2)

archives.diff​ (5.3 KB) - added by nickshanks 6 years ago.
straight forward i think
archives2.diff​ (5.0 KB) - added by nickshanks 6 years ago.
use this patch if committing for 2.1

Download all attachments as: .zip

Change History (4)

nickshanks — 6 years ago

straight forward i think

nickshanks — 6 years ago

use this patch if committing for 2.1

comment:1   nickshanks — 6 years ago

  • Milestone changed from 2.2 to 2.1
  • Priority changed from normal to highest omg bbq

Use "archives2.diff" if committing in time for 2.1

The first patch "archives.diff" contains a '/' removal for text/html servers. This causes XHTML-served-as-text/html to fail to validate, and therefore makes the "Valid XHTML" validator link a bit wrong, so I don't recommend committing that for 2.1 (it should go in for 2.2 though). If the second patch is used, I will create a third that only contains the / change (and update the diff on bug #3609 instead of this one—it already does the same thing elsewhere).

Setting priority to high per rboren's request.

comment:2   ryan — 6 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4774]) Check style. Props nickshanks. fixes #3610

Note: See TracTickets for help on using tickets.