PATCH - Special:Search - invalid result list fixing
Closed, ResolvedPublic


invalid result list fixing

Replacing <ul> with <ol> due to "start" attribute presence.

Version: unspecified
Severity: normal

Attached: SpecialSearch.php.patch

bzimport added a subscriber: Unknown Object (MLST).
bzimport set Reference to bz13925.
Danny_B created this task.Via LegacyMay 1 2008, 10:37 PM
brion added a comment.Via ConduitMay 1 2008, 10:40 PM

We switched from <ol> to <ul> in part because outdated search index entries are silently dropped, and people whine less if they don't see the numbers failing to add up. ;)

But also because it's uglier.

Drop the start attribute rather than switching back to <ol>, probably.

Danny_B added a comment.Via ConduitMay 1 2008, 10:47 PM

removing invalid "start" attribute

OK, second version goes with removed invalid "start" attribute.

Choose the one you like ;-)

Attached: SpecialSearch.php.patch

brion added a comment.Via ConduitMay 1 2008, 11:14 PM


Add Comment