[PATCH 8 of 8 V2] hgweb: add a "followlines" link to hover-box in annotate

Yuya Nishihara yuya at tcha.org
Wed Mar 1 09:52:57 EST 2017


On Tue, 28 Feb 2017 16:00:44 -0800, Gregory Szorc wrote:
> Also, the page it links to showing the revisions with links to a line range
> filtered diff is good but not great. I imagine developers will complain
> that it requires too many clicks to go to what they want (the actual line
> content). IMO it would be *really* useful if there were a single page
> showing multiple changesets and line range data for either/and a) raw line
> content b) annotate view c) diff view. That way, developers could scroll
> and see all versions of that line range. If you were looking at say the
> history of a function, this would be *extremely* useful compared to loading
> N pages. This improvement could be done as a follow-up and doesn't need to
> block this series.

When the page gets more dynamically rendered, will we still need the mdiff
filtering function? I guess filtering will be handled by js, and we'll need
to label all chunks instead of filtering them on server side.

I think the mdiff and patch.diff() APIs are the most controversial part in
this series. If we can go without them, that would be nice.


More information about the Mercurial-devel mailing list