Re: my investigation into the issue users were reporting on dragging active->inactive (#1184826: Dragging Active - Inactive Doesn't Work) I ran into what looks to be an unrelated (somewhat minor), but a little confusing bug. If Sexybookmarks are active on the modules admin pages (siteurl.com/admin/settings/sexybookmarks*), the external JS called to display the appropriate icons there actually hides all of the inactive icons. Rendering the admin interface pretty unusable!

Looks unrelated to the other issue since it doesn't break JS just hides all inactives. But I'll link to this one from there just in case since it looks like people in that issue have a few different similar problems (likely site-specific...

I've done up a quick patch and attached it here, which rectifies the issue and I don't believe would have any ramifications - believe it's good to be applied to sites but would appreciate someone reviewing as I'm still learning drupal best practice etc :)

CommentFileSizeAuthor
admin-bug-patch.patch1.25 KBFreshy88

Comments

iamjon’s picture

iamjon commented
Status: Needs review » Reviewed & tested by the community

patch works as advirtised.

Freshy88’s picture

Freshy88 commented

Cheers for review and sorry for my slow response. Would be good to see this implemented into the module to ensure people don't hit the wierd issue that can be caused otherwise, but far from a priority item!

shareaholic’s picture

shareaholic commented
Issue summary: View changes

SexyBookmarks module is no longer being actively maintained. It was a port of Shareaholic for WordPress, and Shareaholic has since released an official Drupal module:

http://drupal.org/project/shareaholic

Please migrate to the official Shareaholic for Drupal module, which has updated buttons, styles, services, 100% mobile-ready and much more! You can learn more here: https://blog.shareaholic.com/drupal-module/

shareaholic’s picture

shareaholic commented
Version: 6.x-2.0 »
Status: Reviewed & tested by the community » Closed (won't fix)