Elliott Sprehn
Elliott Sprehn
There's a little edit link, but maybe not in the extension version.
This needs some API support, it's really expensive to fetch all the state to show per file level lgtm.
Can you give an example issue and what screen resolution your'e at? This isn't about non-programmers, this is about having an interface that's readable (ex. this bug tracker). Note that...
What do you mean scrolling is too slow?
Press the "e" key to expand all diffs, then just scroll with your trackpad. Jumping between chunks doesn't really make sense in the new tool but I'll consider it.
I really need a better API to get context from the server, I'm parsing the existing context responses, but it's very buggy. I'd be much better if I could just...
I don't think this feature is useful since it often has tons of rebase stuff in it. I don't think we should add it until there's more people asking.
I'll add this once Rietveld has a decent API for it. I don't want to do more screen scraping.
I fixed this.
Err, no I didn't, on Patchset 3 I see: Parse error: Unable to classify line: '";' the raw diff shows a line in the diff that doesn't have a +...