Previous code for theme_links() gave only 'first' OR 'last' to each list element. But if there is only one item in the list, then it is both the first AND the last item and should have *both* classes. This patch fixes this problem.

CommentFileSizeAuthor
#4 li-first-and-last_0.patch900 bytesChrisKennedy
li-first-and-last.patch745 bytesjjeff

Comments

jjeff’s picture

jjeff commented

For reference:

http://drupal.org/node/89818

edmund.kwok’s picture

edmund.kwok commented
Status: Active » Needs review
coreb’s picture

coreb commented
Version: x.y.z » 5.x-dev

Moving out of the "x.y.z" queue to a real queue.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new li-first-and-last_0.patch900 bytes

Works fine.

Minor tweaks in the updated patch: synced with HEAD, re-rolled from root, added period to the comment, and fixed potential E_ALL warning.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)