Issue status update for http://drupal.org/node/18329 Project: Drupal Version: cvs Component: base system Category: feature requests Priority: normal Assigned to: Anonymous Reported by: Steven Updated by: Steven Status: patch Attachment: http://drupal.org/files/issues/confirm_1.patch (28.2 KB) Sorry the patch was still a bit dirty. Here's a better one. Steven Previous comments: ------------------------------------------------------------------------ March 3, 2005 - 20:30 : Steven Attachment: http://drupal.org/files/issues/confirm_0.patch (28.55 KB) This patch creates a themable function theme_confirm() which is used to confirm actions everywhere across Drupal (mostly 'delete', but e.g. used for 'reset' in the menu system too). I also took the liberty of changing a bunch of messages to be more helpful in their wording. A plain message looks like this: http://acko.net/dumpx/confirm2.png (the red is from the theme) but some messages also display more info (e.g. mass delete of nodes shows a list of titles as well). The idea is taken from the Apple human interface guidelines: there is a large question, with some more explanation in smaller type below about what the effects might be of this action. The actions are not labeled "yes/no", but are verbs that mean something. This is much easier to end-users than trying to figure out what "Yes" is ("Are you sure you don't want to abort quitting the program and losing all your work?"). The 'cancel' item is a link rather than a button, because it does not perform an action, while the confirm button does. The menu admin especially got some improvements (to distinguish between menus and menu items in confirmation messages) while node deletion was changed to support drupal_goto and drupal_set_message. I discussed this patch with Chris Messina, he might do some more work on the display after 4.6, but in the meantime this patch is already a good usability and code improvement.