Closed
Bug 467994
Opened 16 years ago
Closed 16 years ago
drag and drop subfolder to root of account is blocked/fails
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
VERIFIED
FIXED
Thunderbird 3.0b2
People
(Reporter: wsmwk, Assigned: iannbugzilla)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b3pre) Gecko/20081204 Thunderbird/3.0b1
drag and drop subfolder to root of account is blocked/fails. i.e. drag a local second level folder (subfolder) to local account to make it a first level folder
result: operation shows the drop is blocked
works if dragging subfolder to another folder
Comment 1•16 years ago
|
||
This is true on Mac too! Build 3.0b2pre
Comment 2•16 years ago
|
||
Regression from tb2, and quite limiting, as there's no way to move back a folder other than dragging and dropping, in case you move it accidentally.
The reason is that the new code checks .canFileMessages regardless if it is a message or folder - http://mxr.mozilla.org/comm-central/source/mail/base/content/folderPane.js#343
The old code used to do that check as well as a lot more.
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
This patch makes canDrop only check .canFileMessages when the drop types are messages.
Attachment #357240 -
Flags: review?(mkmelin+mozilla)
There are other checks we probably still need to do for messages that were in the old code:
targetFolder.isServer and checking not dropping into folder it is already in.
Updated•16 years ago
|
Attachment #357240 -
Flags: review?(mkmelin+mozilla) → review+
Comment 7•16 years ago
|
||
Comment on attachment 357240 [details] [diff] [review]
Only check .canFileMessages for messages patch v0.1 (Checkin: Comment 9)
Thx, r=mkmelin
Attachment #357240 -
Flags: superreview?(bienvenu)
Updated•16 years ago
|
Attachment #357240 -
Flags: superreview?(bienvenu) → superreview+
Comment 8•16 years ago
|
||
Comment on attachment 357240 [details] [diff] [review]
Only check .canFileMessages for messages patch v0.1 (Checkin: Comment 9)
thx for the patch!
Attachment #357240 -
Attachment description: Only check .canFileMessages for messages patch v0.1 → Only check .canFileMessages for messages patch v0.1 (Checkin: Comment 9)
Comment on attachment 357240 [details] [diff] [review]
Only check .canFileMessages for messages patch v0.1 (Checkin: Comment 9)
http://hg.mozilla.org/comm-central/rev/47815c10e304
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Target Milestone: --- → Thunderbird 3.0b2
You need to log in
before you can comment on or make changes to this bug.
Description
•