Module "Cool message" provides a beautiful display of system messages (errors, warning, etc.).
These messages displaying at the top of the page using CSS3 and jQuery.
If you click on message then it will hide.

Screenshots:
- screenshot 1;
- screenshot 2.

Project Details

Project page: http://drupal.org/sandbox/azovsky/1300120
Git repository: git clone --branch master azovsky@git.drupal.org:sandbox/azovsky/1300120.git
Developed for: Drupal 7.x

Comments

greggles’s picture

Status: Needs review » Needs work

Those screenshots look great. It would be really helpful to add one to the project page (edit it and then click attach image) so people can see what this will look like. In general your project page could be spruced up http://drupal.org/node/997024 which will help users to find the module and use it.

Please change the line endings on all the files to be unix line endings (i.e. remove the carriage return "\r") character. from http://drupalcode.org/sandbox/azovsky/1300120.git/blob/refs/heads/master...

I looked for license issues and it's not clear to me what license the original css/jquery is released under.

I left a comment asking: http://www.red-team-design.com/cool-notification-messages-with-css3-jque...

The CSS should be updated to match Drupal standards for CSS. You may also want to work on making it unique from the original version.

I reviewed the code and didn't find any problems.

greggles’s picture

One other thought - you've got enough markup in the theme_status_messages_box function that it might make sense to change it to being a template and see if that looks easier to maintain or not.

azovsky’s picture

Status: Needs work » Needs review

I've updated the module:

  • fixed the line endings;
  • fixed CSS.

I think to make a separate template file for the message makes no sense. HTML markup is not too much.

And with the license all is well.

Attach a picture to this issue not obtained ...

Thanks, Greg!

13rac1’s picture

Status: Needs review » Needs work
  1. Project page needs work and should have a screenshot.
  2. I 2nd the HTML template file.
azovsky’s picture

Status: Needs work » Needs review

Updated Project page.

Added cool-messages.tpl.php for theming messages.

klausi’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

sreynen’s picture

Status: Reviewed & tested by the community » Fixed

Hi azovsky,

Thanks for your contribution and welcome to the community of project contributors on drupal.org.

I've granted you the git vetted user role which will let you promote this to a full project and also create new projects as either sandbox or "full" projects, at your discretion.

Now that you've experienced the full review process, please consider reviewing other projects that are still awaiting review. Anyone can help with reviews, following the guidelines.

azovsky’s picture

Many thanks to all!

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

Anonymous’s picture

Issue summary: View changes

Update descr.