New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Blame view doesn't have line numbers #45

Open

GoogleCodeExporter opened this issue Mar 20, 2015 · 1 comment

Labels

auto-migrated Priority-Medium Type-Defect

Collaborator

GoogleCodeExporter commented Mar 20, 2015

ex.

https://chromium.googlesource.com/chromium/blink.git/+blame/87e70381aca484f1700e
a23e72f5adc78dc4581c/Source/core/inspector/InspectorPageAgent.cpp

compared to

https://src.chromium.org/viewvc/blink/trunk/Source/core/inspector/InspectorPageA
gent.cpp?annotate=172074

Line numbers are really important, this makes dealing with stack traces at 
specific revisions very difficult.

Original issue reported on code.google.com by esprehn@chromium.org on 21 Apr 2014 at 9:42

The text was updated successfully, but these errors were encountered:

GoogleCodeExporter added Type-Defect Priority-Medium auto-migrated labels

Collaborator Author

GoogleCodeExporter commented Mar 20, 2015

Line numbers used to be provided by prettify. I turned off prettify because it 
was slowing down rendering, even compared to prettify without blame. But I 
think I found the cause of that issue so we can turn prettify back on.

If I'm wrong and performance is still poor, we'll have to consider alternatives 
(client and/or server-side) for line numbers. I agree this feature is important.

Original comment by dborowitz@google.com on 21 Apr 2014 at 9:51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment