Last modified: 2009-09-20 03:27:02 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T22722, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 20722 - FlaggedRevs note box (form) should contain current notes, to be easily changed
FlaggedRevs note box (form) should contain current notes, to be easily changed
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
FlaggedRevs (Other open bugs)
unspecified
All All
: Normal enhancement with 1 vote (vote)
: ---
Assigned To: Aaron Schulz
http://he.wikisource.org/wiki/%D7%A2%...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-18 13:40 UTC by Melancholie
Modified: 2009-09-20 03:27 UTC (History)
0 users

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Melancholie 2009-09-18 13:40:10 UTC
The FlaggedRevs notice box (meaning the editing form for editors: ID "wpNotes" ("mw-notebox")) should contain current notes (those that were given formerly, currently shown on bottom of page "User made the following notes reviewing this revision").

This would allow to easily (re-)adjust/change/(re-)use the existing, former "Observations or notes to display".

So, consider preloading the already used notes of the revision (or the most current?) into the form. Like this, people can remove some obsolete or add some further notes without having to write or copy everything into the form again before rating/reviewing.

At the moment the problem is even that if someone forgets to (re-)add notes to the form, the notes on bottom disappear, all.

If the rating is good enough to not show notes anymore (the form disappears then by JavaScript magic), the form should be blanked respectively the server should accept the notes. (not sure on this)

----
This is just a consideration, discussible (if this configuration is an advantage or not is a matter of how the note box is used (how many notes, how many changes on notes on re-rating, if 'notes' shall be permanent etc.), but I think this way it should be most suitable, as articles get improved step by step).

Maybe just create a *config variable*, so every community could decide whether to use this preload function or not!
Comment 1 Melancholie 2009-09-18 13:45:28 UTC
> the form should be blanked respectively the server
> should accept the notes.

... is missing a "not":
... should _not_ accept the notes. ;-)
Comment 2 Melancholie 2009-09-18 13:46:04 UTC
P.s.: The config variable should be True by default.
Comment 3 Aaron Schulz 2009-09-18 15:12:50 UTC
The diff-to-stable form now prefills it (as the tags already are) in r56604
Comment 4 Aaron Schulz 2009-09-20 03:22:09 UTC
More tweaks in r56660.
Comment 5 Aaron Schulz 2009-09-20 03:27:02 UTC
It will prefill in the cases that other review items prefill already.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links