Closed (duplicate)
Project:
Private Message Growl
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
27 May 2009 at 09:42 UTC
Updated:
16 Sep 2011 at 06:08 UTC
How about merging this with the jGrowl Module? I would like to have the Ability to override status messages to be displayed on jGrowl.
Comments
Comment #1
Dave.Ingram commentedHi there,
I had thought about extending this as a part of the jGrowl module at first, but the functionality is distinctly different enough that I decided against it. I assume you'll get errors trying to turn them both on at the same time, but you may be able to work around this by just commenting out the drupal_add_js or drupal_add_css files in the module that throws the errors.
Still, I'm not opposed to rolling in this functionality at some point, so I will leave this issue open for the time being and contact the maintainer of jGrowl module to see what his thoughts are on this.
Comment #2
naheemsays commentedThe jgrowl module seems rather limited in scope - fromk when lookingat it, it seems to only be for the site messages.
It will probably need to be refactored into a more general purpose module before allowing the privatemsg functionality to be merged.
Comment #3
Dave.Ingram commentedSteve McKenzie has graciously granted me CVS access to jGrowl. I'm going to look into this when I have some time, but have been quite busy this week. Anyone wanting to look into the code and start coming up with a plan is more than welcome to. I think we could factor the jgrowl module into one that basically just adds jgrowl to the page and maybe adds some helper functions if necessary, and then a separate module that handles system message alerts. Both would be relatively small, then pmgrowl could require the jgrowl_core module as a dependency.
Like I said, I'll look at this when I have time, but if anyone wants to lead the charge on this I'll gladly start committing patches to make it happen.
Comment #4
XerraX commentedi love to hear this, looking forward to it
Comment #5
naheemsays commentedfyi: I have found another duplicate of jgrowl I think: purr messages.
Comment #6
XerraX commentednice one
Comment #7
dqd#5 purr messages depends on purr plugin which doesn't support all older browsers like jGrowl does.I think for Drupal 7, Nice messages is the only maintained option for jGrowl drupal messages in the moment around.
I would rather merge ALL message popup providing modules into one, if so. That's what I thought last time and what I have asked around already.
Comment #8
berdirClosing as duplicate of #1281098: What about a jQuery popup plugin API for all modules needing popup calls