Message info Subject:[Bug 33928] HTML page title should say we are looking at a diff Date:Sun, 18 Mar 2012 16:02:49 +0000

Amir E. Aharoni <> changed:

What |Removed |Added
Keywords|need-review |patch
CC| |
| |l

--- Comment #2 from Amir E. Aharoni <> 2012-03-18 16:02:49 UTC ---
I agree that this is a bug that should be fixed. I also didn't like that the
title of the diff page is the same as of the page itself.

Patch 10238 looks good to me functionally, but:
* wfMessage() is better than wfMsg(), see
* setPageTitle() would be better inside showDiffPage()
* Documentation for the new message must be added in MessagesQqq.php. Please
mention that it's the title of the HTML page.
* Please consider adding the message to MessagesAs.php, too :)

Configure bugmail:
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

Wikibugs-l mailing list