This is a really minor issue (for most). I need a class on the "Delete message" link when viewing a message thread. Unfortunately the links don't go through the theme system (bad!) until they're all in a UL string. So unless I want to A) recreate the entire list or B) use a regex, I'm pretty well out of luck.

This patch does not fix the larger problem of theming the links, but it does add a class to the delete link so I can place an "X" icon next it with CSS.

CommentFileSizeAuthor
privatemsg_delete_class.patch431 bytesquicksketch

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Hm. I could move the adding of the message links in my api patch from the view_alter hook to the load hook. That does make sense anyway and would allow to change/alter them.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Hm no, after thinking a bit more about it, it's not really useful because the links are only needed when the message is displayed.

litwol’s picture

litwol commented
Status: Reviewed & tested by the community » Fixed

Commited.

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.