Page MenuHomePhabricator

Add option for editors in preferences to always show diff to latest flagged revision
Closed, ResolvedPublic

Description

Author: pbirken

Description:
It would be nice to have an option for editors in the preferences for the following case:

By default, logged in users always see the latest revision. In the case the latest revision is not flagged, the GUI shows this. If you want to flag that revision, you should first check the diff to the last flagged revision, which requires one more click. So, it would be nice to have an option in the preferences that will cause you to directly see this diff, instead of the draft version. This would save one click, which is nice enough.


Version: unspecified
Severity: enhancement

Details

Reference
bz20058

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:49 PM
bzimport set Reference to bz20058.

Hmm...how much demand is there for this?

pbirken wrote:

Back from holiday: I asked what people though about this and some said that they would use it. So, demand is not overwhelming, but definitely there.

hmm, what are the scenarios where this is considered most useful?

pbirken wrote:

This is if you just browse the Wikipedia and stumble on an article with revisions in need of flagging. Then, this saves a click and I believe that this click is the one that makes the difference between someone actually flagging that page or just continue browsing without having done so.

I also worry how this will affect large pages that already take ages to render.

pbirken wrote:

Well, I'm not sure. Pages with a lot of formulas can be a pain sometimes. But the page has to be rendered anyhow and I never noticed any problems with diff-rendering?

I'm not sure if it's as bad as enwiki then...pages like United State/Japan/India/Barrack Obama all have load time issues. Things can be rendering for 20+ sec. If diffs have to be loaded on top of that there is even more lag and a higher risk of timeouts.

pbirken wrote:

The difference could be the number of references, which is significantly smaller on de, compared to en. However, since this is supposed to just be an option in the preferences and I don't expect even the majority of the editors to chose this option, I doubt that this might become a serious issue.

Wait, "...instead of the draft version."? So the page would just be the diff and review form?

Done in r56014. Diff shows above page content.

pbirken wrote:

Ohoh, my fault! I didn't mean "instead", but "in addition". Sorry about that, so the discussion we were having was exactly the right one.

Re-closing (unless there is something else). The diff is above the content, but does not replace it as per #10.