My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 1541: "Expand N before/after" links don't work for me in Opera 12
3 people starred this issue and may be notified of changes. Back to list
Status:  Released
Owner:  ----
Closed:  Jul 3


Sign in to add a comment
 
Reported by matma.rex@gmail.com, Aug 24, 2012
Affected Version: 2.4.2

(I'm using Wikimedia's gerrit: https://gerrit.wikimedia.org/)

What steps will reproduce the problem?
1. Open a side-by-side diff view for any commit. (It's necessary to disable "Syntax Coloring" first; see  issue 878 .)
2. Click one of "Expand N before/after" links

I expect the lines to be loaded; instead, only a little blank space appears (probably where the lines are supposed to go) and an error is logged to the console. (Backtrace below.)

I am able to change the setting to include more lines of context, but that's a lousy workaround.

The backtrace itself is anything but helpful:

Uncaught exception: Error

  Error thrown at line 2299, column 99 in <anonymous function>() in https://gerrit.wikimedia.org/r/gerrit/74D0B070A6D7DD5313DD19D8824DF931.cache.html:
      throw b
Oct 19, 2012
Project Member #1 choro...@wikimedia.org
(No comment was entered for this change.)
Labels: Browser-Opera
Dec 10, 2013
#2 marcin.c...@gmail.com
Tracked at Wikimedia as https://bugzilla.wikimedia.org/show_bug.cgi?id=58252

It seems that it is related to the file size, for example:

https://gerrit.wikimedia.org/r/#/c/97675/5/wmf-config/InitialiseSettings.php
Jul 2, 2015
Project Member #3 edwin.ke...@gmail.com
Is still still an issue with the new change screen / new side-by-side diff screen?
Status: AwaitingInformation
Jul 2, 2015
#4 matma.rex@gmail.com
Ha, it seems to be fixed, both in Gerrit 2.11 (tested on gerrit-review.googlesource.com) and 2.8 (gerrit.wikimedia.org, when using new change screen).
Jul 3, 2015
Project Member #5 edwin.ke...@gmail.com
(No comment was entered for this change.)
Status: Released
Labels: FixedIn-2.8
Sign in to add a comment

Powered by Google Project Hosting