dcoughlin commandeered this revision.
dcoughlin edited reviewers, added: skomski; removed: dcoughlin.
dcoughlin added a comment.
Commandeering and closing this revision as it has been more than a year.
Repository:
rL LLVM
https://reviews.llvm.org/D13134
dcoughlin added a comment.
Thanks for the update. Unfortunately, we can't accept the mousetrap code. Any
code contributed to clang needs to be under the UIUC license. The original
copyright owner would need to contribute the code under that license.
I think it would be fine to revert to your p
skomski removed rL LLVM as the repository for this revision.
skomski updated this revision to Diff 36490.
skomski added a comment.
Included mousetrap to get keyboard shortcuts working on all international
keyboards.
Fixed behaviour without hash path present: Jump to #Path1 on j or k
http://revi
dcoughlin added a comment.
Thanks for the patch! This looks very useful. What browsers does the JavaScript
support?
One thing I noticed is that after immediately loading a new report page the 'j'
and 'k' shortcuts don't work for me. In Safari, I get:
TypeError: null is not an object (evaluat
skomski created this revision.
skomski added reviewers: dcoughlin, krememek, jordan_rose.
skomski added a subscriber: cfe-commits.
skomski set the repository for this revision to rL LLVM.
Adds three keyboard shortcuts to report.html
to make navigation faster:
Jump to next path: j
Jump to prev
