2128: Add a keyboard shortcuts to goto the head of webpage

suy****@gmai***** (Google Code) (Is this you? Claim this profile.)
June 8, 2011 
http://reviews.reviewboard.org/

What version are you running?
 Review Board 1.5.4

What's the URL of the page this enhancement relates to, if any?


Describe the enhancement and the motivation for it.
CR: Add a keyboard shortcuts to goto the head of webpage.
We can see the overview of the changes easily. We can chose a specify change in some file much more easily.
PR: the keyboard shortcuts didn't work in latest chrome, v12.0.

What operating system are you using? What browser?
windows xp.
chrome v12


Please provide any additional information below.
chipx86
#1 chipx86 
The very top? The Home key should suffice.
  • +NeedInfo
#2 suy****@gmai***** (Google Code) (Is this you? Claim this profile.) 
yes, you are right, it's my neglect. Home can suffice.

How about the PR?
keyboard shortcuts doesn't work on my latest chrome.
It can work well  before I update chrome this morning.
chipx86
#3 chipx86 
Sounds like a Chrome regression. It works in all other browsers, and we're using the standard eventing in the DOM. Do you know of other webapps that use shortcut keys that you can sanity check with?
david
#4 david 
The chrome key regression is also tracked in issue 2101.
#5 suy****@gmai***** (Google Code) (Is this you? Claim this profile.) 
Yes, I just have a try, seems the same problem. 
It's not a hotkey problem as I thought.
I'll close the issue, and we can track the problem in the previous issue.
david
#6 david
  • -NeedInfo
    +NotABug