Open Bug 1388097 Opened 7 years ago Updated 2 years ago

Consider allowing insertTree to affect indexes

Categories

(Toolkit :: Places, enhancement, P3)

Product:
Toolkit
Component:
Places
Type:
enhancement

Tracking

()

Tracking Flags:
Tracking Status
firefox57 --- affected

People

(Reporter: standard8, Unassigned)

Details

In bug 1385733 I'm trying to use insertTree in place of insert for inserting a folder.

browser_bookmarkProperties_addFolderDefaultButton.js was failing as the folder was getting inserted in the wrong position, even though the index was being passed to insertTree.

For now, PT.NewFolder includes an extra update after insertion to set the correct index.
The risk is that a consumer passes bogus indices, that's also the reason it doesn't pick indices iirc.
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.