Page MenuHomePhabricator

Unify layout of enhanced watchlist/recent changes
Closed, ResolvedPublic

Description

Author: FT2.wiki

Description:
The enhanced watchlist groups edits on a page together. At present it shows lines like:

Time Page (4 changes) . . (+bytes) . . (hist) [list of editors]

What you SHOULD get is:

Time Page (4 changes | hist) . . (+bytes) . . [list of editors]


Version: unspecified
Severity: enhancement

Details

Reference
bz11728

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:57 PM
bzimport set Reference to bz11728.
bzimport added a subscriber: Unknown Object (MLST).

While the above request is a little vague :) I think the point would be that this would make the grouped lines more consistent with the single-change lines which look like:

Time Page (diff; hist) . . (+bytes) Username (Talk | contribs) (comment)

Note that currently the grouped lines also use a different message for the history link (eg showing as longer "(Page history)"), another inconsistency which can be remedied by using the same message.

FT2.wiki wrote:

Yep, thats about what I meant :)

Created attachment 4642
Patch moving the history link to the corresponding place

This patch (against the current trunk) makes the history link appear after the “N changes” message, inside the parentheses, just like in the ungrouped case.

Attached:

(Changing summary to a more descriptive one, +keywords)