Closed Bug 271766 Opened 18 years ago Closed 18 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: 18 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.