Can drag bookmarks into address book collection of Show All Bookmarks

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: john, Assigned: bugzilla-graveyard)

Tracking

({fixed1.8.1})

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.2) Gecko/20060310 Camino/1.0.0+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.2) Gecko/20060310 Camino/1.0.0+

This is bad. It makes no sense to do so, and after you do you cannot delete them. Quitting and relauching Camino gets rid of them.

Reproducible: Always

Steps to Reproduce:
1. Drag bookmarks (or maybe copy and paste as well) to the address book section.
2. Ow.
3.



Expected Results:  
Told me I'm an idiot for trying to do this.

Comment 1

13 years ago
Confirming.  You can't drop onto the collection, but you can drop onto the table once the collection is selected.  Definitely not so good.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Can drag bookmarks into address book section of Show All Bookmarks → Can drag bookmarks into address book collection of Show All Bookmarks
(Assignee)

Updated

13 years ago
Assignee: mikepinkerton → nobody
QA Contact: bookmarks
(Assignee)

Comment 2

13 years ago
Created attachment 224328 [details] [diff] [review]
invalidates drop if it's going to occur on a smart folder's outline view
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #224328 - Flags: review?(mozilla)

Comment 3

13 years ago
Comment on attachment 224328 [details] [diff] [review]
invalidates drop if it's going to occur on a smart folder's outline view

Code looks good to me. r+
Attachment #224328 - Flags: review?(mozilla) → review+
(Assignee)

Updated

13 years ago
Attachment #224328 - Flags: superreview?(mikepinkerton)
Comment on attachment 224328 [details] [diff] [review]
invalidates drop if it's going to occur on a smart folder's outline view

sr=pink
Attachment #224328 - Flags: superreview?(mikepinkerton) → superreview+
(Assignee)

Updated

13 years ago
Whiteboard: [needs checkin]

Comment 5

13 years ago
Fixed trunk and 1.8 branch
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED

Updated

13 years ago
Keywords: fixed1.8
Keywords: fixed1.8 → fixed1.8.1

Updated

13 years ago
Whiteboard: [needs checkin]
You need to log in before you can comment on or make changes to this bug.