As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 509954 - Make F5 reload on other platforms than Windows
: Make F5 reload on other platforms than Windows
Product: SeaMonkey
Classification: Client Software
Component: UI Design (show other bugs)
: Trunk
: x86 Linux
: -- normal (vote)
: seamonkey2.1b2
Assigned To: Jens Hatlak (:InvisibleSmiley)
Depends on:
  Show dependency treegraph
Reported: 2009-08-12 07:20 PDT by Robert Kaiser
Modified: 2010-11-03 05:57 PDT (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

make F5 Reload [Checkin: comment 6] (1.78 KB, patch)
2010-11-02 15:37 PDT, Jens Hatlak (:InvisibleSmiley)
neil: review+
neil: superreview+
Details | Diff | Splinter Review

Description User image Robert Kaiser 2009-08-12 07:20:48 PDT
[Hrm, we don't have a Bugzilla component for browser window issues. Too bad.]

I'm not sure where it was, but I saw people surprised that F5 didn't reload on Linux SeaMonkey, as most Linux desktop apps nowadays have adopted that keybinding.

Looking at I see that Firefox enables it on all platforms, only accel-F5 is non-Mac-only for the browser while shows that view source has both defined for everyone - Stefan, do you know the reason for the exception?

I think we should go and move our definitions from to a general place to follow suit(e) and also expectations users seem to have nowadays.
Comment 1 User image Robert Kaiser 2009-08-12 07:31:57 PDT
As a side note, bug 79397 and bug 129823 are competing for the same key in mailnews ;-)
Comment 2 User image Stefan [:stefanh] 2009-08-12 08:31:15 PDT
Accel+F5 on mac enables Voice Over (bug 307061). I think the viewsource part was just missed (bug 230693).
Comment 3 User image Jens Hatlak (:InvisibleSmiley) 2010-11-02 14:37:14 PDT
I think we should really have this. Just checked Firefox 3.5, Minefield, Konqueror, Opera, and Chromium, all on Kubuntu. All support F5 = Reload.

Firefox has F5 on all platforms, and Shift+F5 on all but Mac (cf. To achieve the same, we have two options:
a) do it like FF, i.e. add it to navigatorOverlay.xul, preprocess that file (currently it isn't, cf. and use an #ifndef XP_MACOSX rule there
b) add it to the individual platform files that don't have it yet (mac/, unix/), leaving out Accel+F5 for Mac.

Neil, feel free to un-CC yourself but please answer the above question first.
Comment 4 User image Stefan [:stefanh] 2010-11-02 15:25:23 PDT
Firefox having F5 on Mac is probably a mistake that no-one has discovered yet, no need to repeat that. We have Accel+R in the menu and that should be enough.

I can btw probably answer for Neil - b) is the way to go, we already have the files for doing this, so there's no need to pre-process ;-)
Comment 5 User image Jens Hatlak (:InvisibleSmiley) 2010-11-02 15:37:52 PDT
Created attachment 487740 [details] [diff] [review]
make F5 Reload [Checkin: comment 6]

So, after Stefan's feedback, let's only add Unix support (Win already has it, from where I copied it). No changes made to the shortcuts file since no entry exists there for Windows either and it's only a secondary shortcut which doesn't appear in the menu.
Comment 6 User image Jens Hatlak (:InvisibleSmiley) 2010-11-03 05:18:31 PDT
Comment on attachment 487740 [details] [diff] [review]
make F5 Reload [Checkin: comment 6]
Comment 7 User image Robert Kaiser 2010-11-03 05:57:29 PDT
(In reply to comment #4)
> Firefox having F5 on Mac is probably a mistake that no-one has discovered yet

Could you please file it there, then? Thanks!

Note You need to log in before you can comment on or make changes to this bug.