The default bug view has changed. See this FAQ.

Access key for expander button only works once

RESOLVED FIXED in seamonkey2.1b3

Status

SeaMonkey
Bookmarks & History
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

Trunk
seamonkey2.1b3

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
When viewing the properties for a bookmark or livemark, an expander button appears to allow you to view a minimal or complete set or properties. This button has an access key, but the key only works once.
(Assignee)

Comment 1

6 years ago
Created attachment 522845 [details] [diff] [review]
Proposed patch

The label expects the access key to be set on the controlled element, so when the toggle code in bookmarksManager.js sets the access key, there are now two access keys, one on the label, one on the button. If the two keys are different, then both keys work to uncollapse the pane! However if the two keys are the same, or if the pane is collapsed again, then the accelerator code detects a conflict and refuses to activate the button. The solution is to set the original access key on the button, where the label can overwrite it.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #522845 - Flags: review?(iann_bugzilla)

Updated

6 years ago
Attachment #522845 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 2

6 years ago
Pushed changeset 9ab5ebcff8fa to comm-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Depends on: 647877
Target Milestone: --- → seamonkey2.1b3
Version: unspecified → Trunk
(Assignee)

Updated

6 years ago
No longer depends on: 647877
You need to log in before you can comment on or make changes to this bug.