Look below at a screenshot.
drupal_set_message is setting the messages in the title. I think it'd be better if it was in the body of the popup, with the title of the previous popup.
For example: if you create a new page, the title would say 'Create Page' or similar on the popup that has the message.
If its not desired for some reason to put the message inside the body of the popup, we should change the facebook theme so the colors don't clash.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | popups.js-528342.patch | 455 bytes | japerry |
| popupconfmessage.jpg | 29.33 KB | japerry |
Comments
Comment #1
japerryHere is a simple patch to move the message down, but doesn't display a title.
Comment #2
jbrauer commentedComment #3
marc.groth commentedReviewed and tested. Patch works perfectly.
Can it be committed?
Comment #4
apanag commentedWe should commit it.
It solves the problem with the drupal_set_messages, that they appear on the title.