see attached patch.

The tooltip for the Tag subscriptions gets a bit borked as there are potentially multiple tables of terms across multiple vocabularies and the theme_table_select_header_cell() function ends up adding Javascript for the tooltip each time...

but other than that this is a useful feature especially if you have a lot of subscriptions (which if you have autosubscribe on would certainly be the case for a busy site).

CommentFileSizeAuthor
#1 notifications.admin_.inc_.227837.patch622 bytesjaydub
#1 notifications_tags.module.227837.patch633 bytesjaydub
notifications_tags.module.patch650 bytesjaydub
notifications.admin_.inc_.patch622 bytesjaydub

Comments

jaydub’s picture

jaydub commented
StatusFileSize
new notifications_tags.module.227837.patch633 bytes
new notifications.admin_.inc_.227837.patch622 bytes

re-rolled off of latest -dev

jose reyero’s picture

jose reyero commented
Status: Needs review » Fixed

Thanks, works great.

Anonymous’s picture

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

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