Closed Bug 271766 Opened 21 years ago Closed 21 years ago

Accel+D shouldn't remove bookmark (bookmarks manager)

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox1.5

People

(Reporter: asaf, Assigned: asaf)

References

Details

Attachments

(1 file)

...from the following reasons: * In the browser, accel+d *adds* bookmark * On mac, accel+d is expected to either dupe the bookmark or do nothing. (Do we want a dupe function?) * Is Accel+D meant anywhere else?
Blocks: 271496
> * Is Accel+D meant anywhere else? ...meant to delete something...
Assignee: vladimir → vladimir+bm
Attached patch Remove Accel+DSplinter Review
Assignee: vladimir+bm → bugs.mano
Status: NEW → ASSIGNED
Attachment #167816 - Flags: review?(mconnor)
Comment on attachment 167816 [details] [diff] [review] Remove Accel+D This doesn't belong, other than in the emacs-like keybindings in textareas, if those are even being used.
Attachment #167816 - Flags: review?(mconnor) → review+
Checking in mozilla/toolkit/content/editMenuOverlay.xul; /cvsroot/mozilla/toolkit/content/editMenuOverlay.xul,v <-- editMenuOverlay.xul new revision: 1.4; previous revision: 1.3 done Checking in mozilla/toolkit/locales/en-US/chrome/global/editMenuOverlay.dtd; /cvsroot/mozilla/toolkit/locales/en-US/chrome/global/editMenuOverlay.dtd,v <-- editMenuOverlay.dtd new revision: 1.3; previous revision: 1.2 done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox1.1
Status: RESOLVED → VERIFIED
Summary: Accel+D shoudn't remove bookmark (bookmarks manager) → Accel+D shouldn't remove bookmark (bookmarks manager)
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: