Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-07 Thread Stefano Zacchiroli
On Thu, Nov 06, 2014 at 08:09:15PM -0500, Jason Pleau wrote: The attached patch implements your previous suggestions : Awesome, thanks! Last comments: 1) I've added to the new .js file a copyright header pointing to you. Can you confirm you're OK with contributing your code under AGPLv3,

Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-07 Thread Stefano Zacchiroli
On Fri, Nov 07, 2014 at 06:24:58PM +0100, Stefano Zacchiroli wrote: You can find attached a version of your patch that includes the copyright header (and fixes some whitespace issues that make Git cry :-)) Now with an attachment. -- Stefano Zacchiroli . . . . . . . z...@upsilon.cc

Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-07 Thread Jason Pleau
Hello Stefano, On 07/11/14 12:24 PM, Stefano Zacchiroli wrote: On Thu, Nov 06, 2014 at 08:09:15PM -0500, Jason Pleau wrote: The attached patch implements your previous suggestions : Awesome, thanks! Last comments: 1) I've added to the new .js file a copyright header pointing to you.

Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-06 Thread Stefano Zacchiroli
Hello again, Jason. To JavaScript hackers lurking here: your comments on this patch are most welcome! On Sat, Nov 01, 2014 at 11:37:56AM -0400, Jason Pleau wrote: I have attached a patch that solves this bug. Thanks a lot for this patch. It works like a charm on my local deployment. As I'm no

Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-06 Thread Jason Pleau
Hello Stefano, The attached patch implements your previous suggestions : On 06/11/14 02:22 PM, Stefano Zacchiroli wrote: Hello again, Jason. To JavaScript hackers lurking here: your comments on this patch are most welcome! Thanks a lot for this patch. It works like a charm on my local

Bug#761103: debsources: highlight #Lxxx lines by default

2014-11-01 Thread Jason Pleau
Hello ! I have attached a patch that solves this bug. From the commit message: It supports two formats: #L50 and #L50-L150 (The latter will highlight from line 50 to line 150). It also changes the hash if we click on a line, and if we click on a second line holding the SHIFT key, it will

Bug#761103: debsources: highlight #Lxxx lines by default

2014-09-10 Thread Stefano Zacchiroli
Package: qa.debian.org Severity: wishlist [ bug originally reported by Joachim Breitner ] URLs like http://sources.debian.net/src/cairo/1.12.14-4/boilerplate/cairo-boilerplate-xcb.c#L395 should highlight the line mentioned in the anchor by default, without requiring the user to pass ?hl.