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.

CommentFileSizeAuthor
#1 popups.js-528342.patch455 bytesjaperry
popupconfmessage.jpg29.33 KBjaperry

Comments

japerry’s picture

japerry
Location KOMK
commented
StatusFileSize
new popups.js-528342.patch455 bytes

Here is a simple patch to move the message down, but doesn't display a title.

jbrauer’s picture

jbrauer commented
Status: Active » Needs review
marc.groth’s picture

marc.groth commented
Version: 6.x-2.x-dev » 6.x-2.0-alpha5
Status: Needs review » Reviewed & tested by the community

Reviewed and tested. Patch works perfectly.

Can it be committed?

apanag’s picture

apanag commented

We should commit it.
It solves the problem with the drupal_set_messages, that they appear on the title.