Closed
Bug 347714
Opened 19 years ago
Closed 19 years ago
Go button is no longer a D&D target
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: zeniko, Assigned: rflint)
References
Details
(Keywords: fixed1.8.1, Whiteboard: [Fx2 theme change])
Attachments
(1 file)
1.27 KB,
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
... because the ondragover, ondragdrop and ondragexit attributes were (accidentally?) removed.
OTOH if that was an intentional change, the goButtonObserver object can be removed from browser.js.
Updated•19 years ago
|
Flags: blocking-firefox2?
Updated•19 years ago
|
Assignee: nobody → mconnor
Flags: blocking-firefox2? → blocking-firefox2+
Whiteboard: [Fx2 theme change]
Assignee | ||
Updated•19 years ago
|
Assignee: mconnor → rflint
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Comment 1•19 years ago
|
||
Dropping bookmarks/links onto the Go button as way of going to them seems like reasonable behavior, just as it is for dropping them onto the location bar and content window. Looks like this was just accidental, and we should fix it.
Comment 2•19 years ago
|
||
*** Bug 348402 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 3•19 years ago
|
||
Attachment #233536 -
Flags: review?(mconnor)
Comment 4•19 years ago
|
||
Comment on attachment 233536 [details] [diff] [review]
Patch
r+a=me for this branch-only regression from the theme landing
Attachment #233536 -
Flags: review?(mconnor)
Attachment #233536 -
Flags: review+
Attachment #233536 -
Flags: approval1.8.1+
Assignee | ||
Comment 5•19 years ago
|
||
Checked in.
You need to log in
before you can comment on or make changes to this bug.
Description
•