#70 File links at the top point to an ambiguous anchor


The links to files at the top of a code review page go to an anchor with the same name as the path to the file.

Problem is, this anchor occurs twice in the HTML -- once at the top, once at the bottom.

Opera happens to choose the bottom one, so clicking on any file link will invariably take me to the bottom of the review for that file. Firefox presumably chooses the top. To remove the ambiguity, the one at the bottom should probably be removed.


  • rob_webset

    rob_webset - 2008-02-26

    Logged In: YES
    Originator: NO

    The simplest fix for this is:

    File: Render.pm
    Method: _coloured_mode_finish

    Remove the following code:

    if ($self->{fview} != -1) {
    # Show the current file header again for navigation purposes when
    # viewing a single file at a time.

    This would remove the bottom bar at the end of the single file view.

    This would also solve the problem that the top bar spans the whole of the width of the code review, and the bottom one just spans the viewable screen.

  • David Sitsky

    David Sitsky - 2008-03-17
    • status: open --> closed-fixed
  • David Sitsky

    David Sitsky - 2008-03-17

    Logged In: YES
    Originator: NO

    Fixed so only the top anchor is displayed - will be available in the next release.


Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

No, thanks