<<TableOfContents>>

= Short description =

The edit warn message that some one else does already edit the page should become more noticable e.g. another color, blinking text or by playing a warning sound.

= Patch for Moin1.9.0beta3 =
== General aims ==
General aims of the given patch is to make following messages better noticable (since they could cause a lot of trouble to the user):
 * Warn of potential conflicts:
  * Message "The lock you held timed out. Be prepared for editing conflicts!"
  * Message "This page was opened for editing or last previewed at .. by ... You should refrain from editing this page for at least another .. minute(s), to avoid editing conflicts. To leave the editor, press the Cancel button."
 * Warn of existing conflicts:
  * Message "Someone else deleted this page while you were editing!"
  * Message "Someone else changed this page while you were editing!"
  * Message "Someone else saved this page while you were editing! Please review the page and save then. Do not save this page as it is!"

When trying to make something better noticable, this should adhere to basic accessibility guidelines. Therefore, '''blinking text cannot be used''', see http://www-03.ibm.com/able/guidelines/java/javaavoidblnk.html and http://www.scsk12.org/SCS/webhelp/accessibility/blinking.htm. The given patch thus just styles the warning messages in another color, with bigger fonts and in bold. Blue font color is used for potential conflicts, red color is used for existing conflicts.

== Explanation of the patch ==
 * !PageEditor.py
  * got rid off widget.status; used instead several calls of theme.add_msg.
  * needed to find a way to mark potential conflicts; introduced var "edit_lock" for that. edit_lock=3 warns about these potential conflicts 
  * some cosmetics
 * theme/_init_.py
  * needed to find a way to supress "clear message" link
  * changed rendering of output from "<p><div ..>...</div></p>" to "<div><p>...</p><div>
  * some cosmetics
 * theme code
  * modern.py: supress link "clear message" when rendering editor header
  * common.css: added css for styling the messages
  * Note: up to now only modern-theme changed

== Patch ==

{{attachment:patch.diff}}

= Discussion =
Yes, this would be quite important in my eyes, however for conflict_msg, edit_lock_message, draft_message there are not seperate css-classes yet. All get wrapped into class "status" by the status widget -- -- OliverSiemoneit <<DateTime(2009-09-06T20:42:44+0100)>>

= Related topics =

MoinMoinBugs/NoMsgForEditConflict

----
CategoryFeatureRequest