Page MenuHomePhabricator

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

Description

invalid result list fixing

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


Version: unspecified
Severity: normal

Attached:

Details

Reference
bz13925

Related Objects

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:10 PM
bzimport set Reference to bz13925.
bzimport added a subscriber: Unknown Object (MLST).

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.

removing invalid "start" attribute

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

Choose the one you like ;-)

Attached: