Index: trunk/phase3/includes/PageHistory.php |
— | — | @@ -301,9 +301,12 @@ |
302 | 302 | '(' . $this->message['rev-delundel'] . ')' ); |
303 | 303 | // Otherwise, show the link... |
304 | 304 | } else { |
305 | | - $jsCall = 'updateShowHideForm('.$rev->getId().',this.checked)'; |
| 305 | + $id = $rev->getId(); |
| 306 | + $jsCall = "updateShowHideForm($id,this.checked)"; |
306 | 307 | $del = Xml::check( 'showhiderevisions', false, |
307 | | - $hidden + array('onchange' => $jsCall) ); |
| 308 | + $hidden + array( |
| 309 | + 'onchange' => $jsCall, |
| 310 | + 'id' => "mw-revdel-$id" ) ); |
308 | 311 | $query = array( |
309 | 312 | 'type' => 'revision', |
310 | 313 | 'target' => $this->mTitle->getPrefixedDbkey(), |
— | — | @@ -473,11 +476,15 @@ |
474 | 477 | */ |
475 | 478 | function diffButtons( $rev, $firstInList, $counter ) { |
476 | 479 | if( $this->linesonpage > 1 ) { |
477 | | - $radio = array( 'type' => 'radio', 'value' => $rev->getId() ); |
| 480 | + $id = $rev->getId(); |
| 481 | + $radio = array( 'type' => 'radio', 'value' => $id ); |
478 | 482 | /** @todo: move title texts to javascript */ |
479 | 483 | if( $firstInList ) { |
480 | 484 | $first = Xml::element( 'input', |
481 | | - array_merge( $radio, array( 'style' => 'visibility:hidden', 'name' => 'oldid' ) ) |
| 485 | + array_merge( $radio, array( |
| 486 | + 'style' => 'visibility:hidden', |
| 487 | + 'name' => 'oldid', |
| 488 | + 'id' => 'mw-oldid-null' ) ) |
482 | 489 | ); |
483 | 490 | $checkmark = array( 'checked' => 'checked' ); |
484 | 491 | } else { |
— | — | @@ -487,14 +494,20 @@ |
488 | 495 | $checkmark = array(); // We will check the next possible one |
489 | 496 | } else if( $counter == 2 || !$this->mOldIdChecked ) { |
490 | 497 | $checkmark = array( 'checked' => 'checked' ); |
491 | | - $this->mOldIdChecked = $rev->getId(); |
| 498 | + $this->mOldIdChecked = $id; |
492 | 499 | } else { |
493 | 500 | $checkmark = array(); |
494 | 501 | } |
495 | | - $first = Xml::element( 'input', array_merge( $radio, $checkmark, array( 'name' => 'oldid' ) ) ); |
| 502 | + $first = Xml::element( 'input', |
| 503 | + array_merge( $radio, $checkmark, array( |
| 504 | + 'name' => 'oldid', |
| 505 | + 'id' => "mw-oldid-$id" ) ) ); |
496 | 506 | $checkmark = array(); |
497 | 507 | } |
498 | | - $second = Xml::element( 'input', array_merge( $radio, $checkmark, array( 'name' => 'diff' ) ) ); |
| 508 | + $second = Xml::element( 'input', |
| 509 | + array_merge( $radio, $checkmark, array( |
| 510 | + 'name' => 'diff', |
| 511 | + 'id' => "mw-diff-$id" ) ) ); |
499 | 512 | return $first . $second; |
500 | 513 | } else { |
501 | 514 | return ''; |
Index: trunk/phase3/RELEASE-NOTES |
— | — | @@ -294,6 +294,7 @@ |
295 | 295 | Keyword set was not useful, and is ignored by modern search engines anway. |
296 | 296 | * (bug 19827) Special:SpecialPages title is "Upload file |
297 | 297 | * (bug 19355) Added .xhtml, .xht to upload file extension blacklist |
| 298 | +* (bug 19287) Workaround for lag on history page in Firefox 3.5 |
298 | 299 | |
299 | 300 | == API changes in 1.16 == |
300 | 301 | |