Bug 244347 Comment 116 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Magnus Melin [:mkmelin] from comment #115)
> 
> ::: mailnews/base/prefs/content/AccountManager.js
> @@ +1776,5 @@
> >      }
> >  
> > +    let accountTree = document.getElementById("accounttree");
> > +    accountTree.addEventListener("dragover", event => {
> > +      event.preventDefault();
> 
> what is this for?
> 
> Maybe it should only show drop allowed where it can, i.e. not the account
> manager sections (server settings, copies & folders etc.)
> 

By default, data/elements cannot be dropped in other elements. To allow a drop, we must prevent the default handling of the element.
Revision 1 by
on 
(In reply to Magnus Melin [:mkmelin] from comment #115)
>
> 
> what is this for?
> 
> Maybe it should only show drop allowed where it can, i.e. not the account
> manager sections (server settings, copies & folders etc.)
> 

By default, data/elements cannot be dropped in other elements. To allow a drop, we must prevent the default handling of the element.

Back to Bug 244347 Comment 116