Edgewall Software
Modify

Opened 13 years ago

Last modified 6 years ago

#6709 new enhancement

Extended 'view diff' feature does not update the browsing position

Reported by: Emmanuel Blot Owned by:
Priority: low Milestone: next-major-releases
Component: version control/changeset view Version: 0.11b1
Severity: minor Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When a changeset is browsed (/changeset/xxx) and that the max size/file limit is reached, the changeset diff is not rendered: only the changeset log message and the list of modified files are rendered. This is the expected behaviour.

A handy feature is that each modified file appears in the list with a "view diff" clickable link. When such a link is selected, the diff of the file is rendered, again with the log message of the same changeset.

However the browser shows the regular diff page, without scrolling down to the actual diff section.

When the log message is a bit long, the user experience is a bit weird: it seems that the link led to the same page as the full changeset: the same long log message is rendered, and the selected diff does not appear in the browsing window.

It would be nice to add an anchor to the diff section so that the browser directly jump to the actual diff, that is the info he selected from the changeset view.

Attachments (0)

Change History (6)

comment:1 by Christian Boos, 13 years ago

Milestone: 0.11.1

What you suggest sounds a reasonable thing to do, as a first step.

As as second step, we might want to have in-place expansion of those diffs ;-)

(oh, didn't even notice it was you manu ;-) )

in reply to:  1 comment:2 by Emmanuel Blot, 13 years ago

Replying to cboos:

(oh, didn't even notice it was you manu ;-) )

Will you believe it tooks me a couple of minutes to figure out what's going on with those diffs ? ;-)

In-place diff with AJAX would be even better, I did not think about this feature, you're right.

Another note (OOT) about diff: the tabular vs. unified button seems a bit awkward: the user needs to click on a grayed button to activate the alternative view mode - grey UI items are normally used to depict a disabled feature. I have no alternative proposition in mind but it could be improved using another color, at least. Thanks.

comment:3 by Christian Boos, 13 years ago

Milestone: 0.11.20.12

comment:4 by Remy Blank, 11 years ago

Milestone: 0.12next-minor-0.12.x

Nice idea, but should not block 0.12.

comment:5 by Christian Boos, 11 years ago

Milestone: next-minor-0.12.xnext-major-0.1X

comment:6 by Ryan J Ollos, 6 years ago

Owner: Christian Boos removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.