The default bug view has changed. See this FAQ.

Make F5 reload on other platforms than Windows

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
UI Design
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1b2
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
[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 http://mxr.mozilla.org/comm-central/source/mozilla/browser/base/content/browser-sets.inc#263 I see that Firefox enables it on all platforms, only accel-F5 is non-Mac-only for the browser while http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/components/viewsource/content/viewSource.xul#123 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 http://mxr.mozilla.org/comm-central/source/suite/browser/win/platformNavigationBindings.xul#16 to a general place to follow suit(e) and also expectations users seem to have nowadays.
(Reporter)

Comment 1

8 years ago
As a side note, bug 79397 and bug 129823 are competing for the same key in mailnews ;-)

Comment 2

8 years ago
Accel+F5 on mac enables Voice Over (bug 307061). I think the viewsource part was just missed (bug 230693).
(Assignee)

Comment 3

7 years ago
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. browser-sets.inc). 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. jar.mn) 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.
Assignee: nobody → jh
Status: NEW → ASSIGNED

Comment 4

7 years ago
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 ;-)
(Assignee)

Comment 5

7 years ago
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.
Attachment #487740 - Flags: superreview?(neil)
Attachment #487740 - Flags: review?(neil)

Updated

7 years ago
Attachment #487740 - Flags: superreview?(neil)
Attachment #487740 - Flags: superreview+
Attachment #487740 - Flags: review?(neil)
Attachment #487740 - Flags: review+
(Assignee)

Comment 6

7 years ago
Comment on attachment 487740 [details] [diff] [review]
make F5 Reload [Checkin: comment 6]

http://hg.mozilla.org/comm-central/rev/656d8167f3a2
Attachment #487740 - Attachment description: make F5 Reload → make F5 Reload [Checkin: comment 6]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
(Reporter)

Comment 7

7 years ago
(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!
You need to log in before you can comment on or make changes to this bug.