Project

General

Profile

Actions

Patch #2367

closed

show view/annotate/download link on repositories/entries and repositories/annotate view

Added by Go MAEDA almost 16 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
Start date:
2008-12-20
Due date:
% Done:

0%

Estimated time:

Description

If I enter repositories/entry view by clicking by file name on repositories/revision view, I cannot move to repositories/annotate view directly and I always get frustrated.

This patch adds view/annotate/download link like same as repositories/changes.


Files

link_to_view_and_annotate.diff (2.6 KB) link_to_view_and_annotate.diff Go MAEDA, 2008-12-20 14:35
link_to_view_and_annotate_corrected-r2157.diff (3.48 KB) link_to_view_and_annotate_corrected-r2157.diff Corrected patch against r2157. Mischa The Evil, 2008-12-22 20:57
_link_to_functions.rhtml (634 Bytes) _link_to_functions.rhtml Go MAEDA, 2008-12-23 04:10
screenshot.png (22.7 KB) screenshot.png Go MAEDA, 2008-12-23 04:10
Actions #1

Updated by Mischa The Evil almost 16 years ago

Simple but very useful improvement... :thumbsup:

Though, your patch misses the new partial (app/views/repositories/_link_to_functions.rhtml). I'll upload a corrected patch-file against r2157 of the trunk.

Actions #2

Updated by Jean-Philippe Lang almost 16 years ago

  • Status changed from New to Closed
  • Target version set to 0.8.1

Very usefull indeed, it's committed in r2167.
Thanks.

Actions #3

Updated by Go MAEDA almost 16 years ago

Oh, I mistook to make patch. Thanks the new patch. And thank for JP's quick work.

But the partial I made is a little bit difrerent from it. Using 'link_to_if' instead of 'link_to', not to make link to same view. It helpful to avoid confusing novice user. Please see attached image.

Actions #4

Updated by Mischa The Evil almost 16 years ago

Go MAEDA wrote:

  • File _link_to_functions.rhtml added

But the partial I made is a little bit difrerent from it. Using 'link_to_if' instead of 'link_to', not to make link to same view. It helpful to avoid confusing novice user.

Indeed, that looks even better especially and probably for the novice Redmine user. Doesn't make much sense indeed to provide a link to the actual current view... Thanks for letting know about this difference...

Actions #5

Updated by Jean-Philippe Lang almost 16 years ago

  • Status changed from Reopened to Closed

Change committed in r2188. Thanks Go.

Actions

Also available in: Atom PDF