Don't render diffs by default
Closed, ResolvedPublic

Description

Author: mortgat

Description:
Many requests on the servers are done in order to watch changes (whether it is
vandalism or not).
When users check differences between versions, they don't necessarily want to
have the most recent version being displayed.
It would be a good enhancement if any logged user could choose: displaying the
changed version or not?


Version: unspecified
Severity: enhancement

Details

Reference
bz3446
bzimport set Reference to bz3446.
bzimport added a subscriber: Unknown Object (MLST).
bzimport created this task.Sep 12 2005, 3:36 PM

robchur wrote:

I thought that the History page allows you to choose whichever diff you wish to
see? The reason for the most recent change being shown under something like
Recent Changes is because, logically, that is what the user expects to see.
Could you explain further if I've misunderstood?

mortgat wrote:

I think there is a misunderstanding!
When you compare two versions of an article, you can see what's changed between
both articles seeing their code and you see also the rendering corresponding
with the last code.
That's true when you click "diff" in the RC, and when you compare two versions
of an article via the History.

I'd like to disable the rendering in my individual preferences, since the
servers are overwhelmed with such requests only to check the nature of recent
Changes.

Nice day

Benoit Mortgat

robchur wrote:

Oh, I see. Yes, I'd agree with that logic.

dake.cdx wrote:

This would be very useful indeed. You don't need to see the whole page to track
down vandalism and such.

Siegel wrote:

I agree this would be useful. see bug #3365 and
<http://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%
29#Bare_diffs.3F>

lupin.wp wrote:

patch without user pref

This patch adds the diffonly flag which suppresses the article display if
present.

Attached: diffonly.patch

ezyang wrote:

*** Bug 3365 has been marked as a duplicate of this bug. ***

ezyang wrote:

Bug 208 may be related (the external API), but I don't think they have anything
to do with each other. Lupin, would you be able to extend user preferences to
make diffonly apply globally?

lupin.wp wrote:

add user pref

There's an option in Special:Preferences with this patch, under "Misc". You can
override that setting using diffonly=0 or diffonly=1 in the url.

attachment nodiff ignored as obsolete

lupin.wp wrote:

patch with user pref

Oops, forgot Messages.php - apologies.

Attached: nodiff

brion added a comment.Feb 6 2006, 6:34 AM

I'm leery of adding preference options; they complicate things.

francesco.cosoleto wrote:

patch with user pref and "show diff with article" link

I added a "Show diff with article" link in diff generated page with diffonly
flag enabled. Pref or not pref, I think "diffonly" it's fine feature.

Attached: diffonlypatch.txt

angus wrote:

*** Bug 8488 has been marked as a duplicate of this bug. ***

ayg wrote:

*** Bug 6936 has been marked as a duplicate of this bug. ***

Fixed in r19141 based on a combination of Francesco's patch with a similar
change I'd written myself.

So how does it work? How can this be implemented into our regular wiki usage?

rotemliss wrote:

(In reply to comment #16)

So how does it work? How can this be implemented into our regular wiki usage?

This the the commit description:

(bug 3446) Add user preference to hide page content below diffs, can be
overridden by adding diffonly=1 or diffonly=0 to the URL of the diff page

I think it's clear enough.

francesco.cosoleto wrote:

Thank you. (Not "Francesco's patch", but Lupin)

Add Comment