Bug 1792276 Comment 3 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 
This was briefly addressed, but we had to backout the patch in bug 1786518, because it was causing too many edge problems. So in practice this never worked if not for a few versions. So it's technically a regression, but not really recent or high priority.

I think we could add a third argument to selectItems() to "ensureItemIsVisible" and then use that in PlacesController.showInFolder().
Revision 1 by
on 
This was briefly addressed, but we had to backout the patch in bug 1786518, because it was causing too many edge problems. So in practice this never worked if not for a few versions. It's technically a regression, but not really recent or high priority.

I think we could add a third argument to selectItems() to "ensureItemIsVisible" and then use that in PlacesController.showInFolder().

Back to Bug 1792276 Comment 3