Project

General

Profile

Actions

Bug #4002

closed

[ce] Got whole page inlined at commit list when creating PR

Added by Johannes Bornhold over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
09.06.2016
Due date:
% Done:

0%

Estimated time:
Sorting:
Commit Number:
Affected Version:

Description

When creating a PR and doing the following steps in order i got the whole page inlined at that position where normally the commit list is shown.
Added a screenshot wich displays the bug.

Steps to reproduce:

  1. Open a new PR
  2. On the "new-PR" page click 'Show detailed compare.' link.
  3. Close the new tab which shows the detailed compare again.
  4. On the "new-PR" page click on any commit link in the commit list.
  5. Use the back button of the browser.

Used Environment:

  • Browser: Google Chrome Version 51.0.2704.79 (64-bit)
  • Rhodecode Enterprise: Revision 43455ea49a37

Files

rc-pr-bug.png (280 KB) rc-pr-bug.png Johannes Bornhold, 09.06.2016 12:49

Related issues 1 (1 open0 closed)

Related to RhodeCode CE/EE - Feature #4000: Make compare more functionalNewMarcin Kuzminski [CTO]08.06.2016

Actions
Actions #1

Updated by Johannes Bornhold over 8 years ago

  • Description updated (diff)
Actions #2

Updated by Johannes Bornhold over 8 years ago

  • Description updated (diff)
Actions #3

Updated by Marcin Kuzminski [CTO] over 8 years ago

Actions #4

Updated by Marcin Kuzminski [CTO] over 8 years ago

  • Target version set to v4.2
Actions #5

Updated by Daniel D over 8 years ago

Sounds like a caching issue, the compare url is the same as the ajax request url and the browser cache probably loads it as is rather than do a new request.

Actions #6

Updated by Daniel D over 8 years ago

  • Assignee set to Daniel D
Actions #8

Updated by Marcin Kuzminski [CTO] over 8 years ago

  • Target version changed from v4.2 to v4.1
Actions #9

Updated by Marcin Kuzminski [CTO] over 8 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF