Reloading a page in the source browser redirects to another file

[noamtm@…]

It's hard to explain this one, so I'll jump directly to the steps to reproduce:

1. Go to ​http://trac.edgewall.org/browser/trunk/trac
2. Click a non-link area on the page
3. Hover the "admin" directory
4. Reload the page using the web browser's keyboard shortcut (ctrl-r or cmd-r)
5. Browser navigates to ​http://trac.edgewall.org/browser/trunk/trac#admin

Keep hovering and reloading, it will navigate to files and folders.

Since this behavior is not documented (or is it?), and I find it annoying, I assume it's a bug.

My site is running v1.0, hence the report below. For some reason I can't find the version number of YOUR site.

[Remy Blank]

Which browser are you using? I can't reproduce with Firefox, Chrome or Opera. But maybe I'm misunderstanding step 2 above. Where exactly do you click?

[anonymous]

I tested on Firefox, Chrome, Safari - on OSX. Just to make sure it also happens on IE, I switched to Windows. It does not show itself on any browser on Windows. Same for Firefox/OSX — when using F5 to refresh instead of cmd-r.

To recap: this bug seems to only happen on OSX, when refreshing the page using cmd-r. I'm not a JavaScript expert though, so can someone with a Mac take a look? I suspect it has something to do with the cmd key.

[Noam Tamim <noamtm@…>]

Sorry, anonymous in comment 2 is me, the reporter.

[Remy Blank]

Heh, I didn't know we had that :) Reproduced on a Mac. And it's even documented.

In the source browser, you can move the line selection cursor with j and k, and reload a folder with r (and if the folder wasn't open, it is opened). For some reason, cmd-r triggers the same action as r, and hence opens the folder where the cursor is located. If no cursor has been moved beforehand, it opens the folder pointed by the mouse cursor.

This is done in trunk/trac/htdocs/js/keyboard_nav.js​. There's a check at the top for the Ctrl key, to allow Ctrl+R to work. We should improve that check to also catch the cmd key. According to some sources, the .metaKey event attribute can be used for this. Actually, for good measure, we should probably catch all modifiers there.

[Ryan J Ollos]

Moved ticket assigned to next-stable-1.0.x since maintenance of 1.0.x is coming to a close. Please move the ticket back if it's critical to fix on 1.0.x.

[Ryan J Ollos]

Milestone renamed