Page MenuHomePhabricator

Improve behavior of rollback if diff to last sighted version is from several authors
Closed, DeclinedPublic

Description

Author: pbirken

Description:
If you look at a diff to the last sighted version where several authors were involved, using the rollback button will revert the changes done by the last author and not mark the current version as sighted. This behavior is completely correct, since the edits by the other authors were not treated correctly.

However, this should be improved. Either by showing a warning before using the rollback (should be done anyhow, but is not part of flagged revisions, I know) or by showing a warning after the "Rollback completed" page is shown. Maybe the last page could even redirect to the new diff to last sighted version?


Version: unspecified
Severity: enhancement

Details

Reference
bz16467

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 10:29 PM
bzimport set Reference to bz16467.

The user should use Undo in this case.

pbirken wrote:

It is very difficult for users to change behavior, if they have no real chance of seeing that they did something wrong or even what to do about it.

pbirken wrote:

There is another problem with this. If the last of several authors was a vandal, the rollback is the useful choice, but then the interface does not provide the user with a useful way of flagging.

Both of the suggestions are too obtrusive/inconsistent I fear. There should be better ways to improve work flow.