Feature #2835

clickable/linkable line #'s while browsing the repo

Added by Alex U almost 9 years ago. Updated about 8 years ago.

Status:ClosedStart date:2009-02-25
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:UI
Target version:0.8.2
Resolution:Fixed

Description

A user on the #redmine channel requests clickable/linkable line numbers while repo browsing. Here is a simple patch for the views and a little css tweak.

worksforme :)

linkable_line_numbers.patch Magnifier (2.19 KB) Alex U, 2009-02-25 01:56

linenumbers_without_css.jpg (30.5 KB) Mischa The Evil, 2009-02-26 23:25

linenumbers_with_css.jpg (30 KB) Mischa The Evil, 2009-02-26 23:25

Associated revisions

Revision 2527
Added by Jean-Philippe Lang almost 9 years ago

Clickable/linkable line #'s while browsing the repo or viewing a text file (#2835).

Revision 2555
Added by Jean-Philippe Lang almost 9 years ago

Change to line #'s links in file/repo viewer (#2835).

History

#1 Updated by Mischa The Evil almost 9 years ago

+1, it seems to me that the chosen implementation has a pretty low severity in contrast to a the highly improved useability it provides...

#2 Updated by Jean-Philippe Lang almost 9 years ago

  • Status changed from New to Resolved
  • Target version set to 0.8.2
  • Resolution set to Fixed
Committed in r2527 with sligth changes:
  • title attribute removed (hardcoded english word)
  • change to scm.css removed (not related to this feature)

Thanks.

#3 Updated by Mischa The Evil almost 9 years ago

Jean-Philippe Lang wrote:

Committed in r2527 with sligth changes:
  • ...
  • change to scm.css removed (not related to this feature)

I am in the opinion that it ís related in the sense of the styling of the feature. As such: I think these selectors maybe shouldn't be declared in source:/trunk/public/stylesheets/scm.css. Instead they seem to belong to the source:/trunk/public/stylesheets/application.css.

See the following screenshots:

Without the CSS style-declarations (as committed) With the CSS style-declarations (like the patch)
As you see the changes are:
  • on hover -> underlining removed
  • color -> instead of default link-color a more sophisticated gray-teint is used.

Though, consider it just a minor CSS-improvement... ;)

Thanks for your fast response on this issue...

#4 Updated by Jean-Philippe Lang almost 9 years ago

Yes, sorry. The style is committed in r2555.

Instead they seem to belong to the source:/trunk/public/stylesheets/application.css

scm.css is included when browsing a repo or viewing a file. So I don't think this style is needed in application.css
Am I wrong?

#5 Updated by Jean-Philippe Lang almost 9 years ago

  • Status changed from Resolved to Closed

Merged in 0.8-stable in r2556.

#6 Updated by Jean-Philippe Lang about 8 years ago

  • Estimated time deleted (0.00)

Also available in: Atom PDF