Closed
Bug 355396
Opened 18 years ago
Closed 18 years ago
cmd-up takes you up a path of the URL instead of to top of page
Categories
(SeaMonkey :: UI Design, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: IDontUseMozillaAnyMore, Assigned: neil)
References
Details
(Keywords: fixed-seamonkey1.1.1, regression)
Attachments
(1 file)
(deleted),
patch
|
samuel.sidler+old
:
review+
csthomas
:
review+
kairo
:
approval-seamonkey1.1.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b2) Gecko/20060821 SeaMonkey/1.1a
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b2) Gecko/20060821 SeaMonkey/1.1a
Pressing command-up arrow causes webpage to be with a different page from the same site. Previously it took you to the top of the current page. Command-down arrow still takes you to the bottom of the current page. The existing behavior is an absolute must for laptop users who do not have 'home' and 'end' keys.
Reproducible: Always
Steps to Reproduce:
1. Go to a page
2. Press command-up arrow
Actual Results:
Page is replaces, usually with the main page for that site.
Expected Results:
Page should remain and you should be taken to the top of the page.
Comment 1•18 years ago
|
||
I can confirm this behavior, but am changing the summary a bit.
In SeaMonkey 1.0.x, pressing cmd-up would take you to the top of the page, if you had scrolled it.
In SeaMonkey 1.1 and trunk, pressing cmd-up takes you up a directory in the path. For example, if you were at http://localhost/testing/123/, cmd-up would take you to http://localhost/testing/.
This appears to be a regression, but I'm sure it's actually the casualty of a new feature. Unfortunately, I can't find the bug that added this behavior. As such, I'm not confirming it because there may have been discussion already.
Keywords: regression
Summary: Command-up arrow causes webpage to be replaced. Should take you to top of page. → cmd-up take you up a path of the URL instead of to top of page
Assignee | ||
Comment 2•18 years ago
|
||
Sorry, when I fixed bug 33684 I didn't realise Cmd+Up was already in use.
What key would you like it to be, if any? (You can choose any key you like because it goes in platformNavigationBindings.xul).
Comment 3•18 years ago
|
||
(Cmd+Up also moves tab left)
How about Cmd+Shift+Up ? I don't think it's used anywhere.
Comment 4•18 years ago
|
||
Hmm, maybe Alt+Up works as well.
Comment 5•18 years ago
|
||
Technically, cmd-shift-up used to for selecting "from here to the top the of page". Alt/option- or ctrl- up seem reasonable though.
alt/option-up go to the top of textareas though, so as long as it doesn't interfere with that (which it shouldn't because focus is different), then it should be good.
Assignee | ||
Comment 6•18 years ago
|
||
It's actually alt-up on the other platforms, but I changed it to cmd-up to match the back and forward keys... are we agreed on alt-up?
Comment 7•18 years ago
|
||
(In reply to comment #6)
> It's actually alt-up on the other platforms, but I changed it to cmd-up to
> match the back and forward keys... are we agreed on alt-up?
>
Alt+Up sounds good to me. If you bother to update Help on trunk (otherwise I can), Alt is Opt on mac ;-)
Assignee | ||
Comment 8•18 years ago
|
||
Attachment #254462 -
Flags: review?(samuel.sidler)
Attachment #254462 -
Flags: review?(cst)
Comment 9•18 years ago
|
||
Comment on attachment 254462 [details] [diff] [review]
Proposed patch
I don't build SeaMonkey, but this change looks good to me.
r=me
Attachment #254462 -
Flags: review?(samuel.sidler) → review+
Attachment #254462 -
Flags: review?(cst) → review+
Assignee | ||
Updated•18 years ago
|
Status: NEW → ASSIGNED
Component: Keyboard: Navigation → XP Apps: GUI Features
Product: Core → Mozilla Application Suite
Assignee | ||
Updated•18 years ago
|
Attachment #254462 -
Flags: approval-seamonkey1.1.1?
Assignee | ||
Comment 10•18 years ago
|
||
Fix checked in to the trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Summary: cmd-up take you up a path of the URL instead of to top of page → cmd-up takes you up a path of the URL instead of to top of page
Comment 11•18 years ago
|
||
Comment on attachment 254462 [details] [diff] [review]
Proposed patch
a=me for SM1.1.1 (one more needed)
Assignee | ||
Comment 12•18 years ago
|
||
(In reply to comment #11)
>(From update of attachment 254462 [details] [diff] [review])
>a=me for SM1.1.1 (one more needed)
I don't think it is, so I've checked it in to the branch.
Keywords: fixed-seamonkey1.1.1
Comment 13•18 years ago
|
||
Comment on attachment 254462 [details] [diff] [review]
Proposed patch
a+ per correct notion on Comment #12
Attachment #254462 -
Flags: approval-seamonkey1.1.1? → approval-seamonkey1.1.1+
You need to log in
before you can comment on or make changes to this bug.
Description
•