Cannot permanently delete Imported IE Favorites folder

VERIFIED FIXED in mozilla0.9.3


Bookmarks & History
19 years ago
14 years ago


(Reporter: lotek, Assigned: Ben Goodger (use ben at mozilla dot org for email))



Dependency tree / graph

Firefox Tracking Flags

(Not tracked)


(Whiteboard: relnote-user)


(8 attachments)



19 years ago
Run Mozilla. Delete the Imported IE Favorites folder from Bookmarks|Manage
Bookmarks. Close Mozilla. Run Mozilla. The folder is back.
Mozilla should not automatically import IE Favorites each time it's opened, but
it would be a good option to have in Bookmarks.


19 years ago
Target Milestone: M15

Comment 1

19 years ago
Hmmmm ... good point.  Steve, is it supposed to work like this?

Comment 2

19 years ago
In addition, the Favorites folder shouldn't be migrated if it's empty, which
currently happens.  I have an empty Favorites folder that won't go away...

Comment 3

19 years ago
In addition, if the Favorites folder is empty, it doesn't behave like other 
empty folders, which lose the little arrow on the right.  Instead, a tiny 1 or 2 
pixel menu pops out when selected.

Comment 4

19 years ago
(happens on Mac, too)
OS: Windows 95 → All
Hardware: PC → All
Summary: Cannot permanently delete Imported IE Favorites folder. → Cannot permanently delete Imported IE Favorites folder

Comment 5

19 years ago
Move to M16 for now ...
Target Milestone: M15 → M16

Comment 6

18 years ago
It's important you should be able to delete the IE Favorites folder for good. 
It's a good feature to import them when you install Mozilla, but if you have 
deleted it, the folder shouldn't be imported every time you start the Browser. 
It's really annoying for anyone who keeps track of their bookmarks.

Why not put the import of Favorites in the 'Import Utility' (where it logically 
should be) tool instead?

Comment 7

18 years ago
Also, you can't move the stupid thing. (Click and drag to rearrange doesn't work. 
It's rooted itself in there pretty well.)


18 years ago
Target Milestone: M16 → M18

Comment 8

18 years ago
*** Bug 39840 has been marked as a duplicate of this bug. ***

Comment 9

18 years ago
Move to M21 target milestone.
Target Milestone: M18 → M21
usability issue, worth dealing with before release in my oppinion
Keywords: rtm

Comment 11

18 years ago
To be blunt, for those of us who do not USE IE AT ALL, it's damn annoying.

I think I can see why this may have been a useful idea. People who use both 
browsers see their IE Bookmarks (I refuse to call them favorites) always current 
in Mozilla, but see nothing from Mozilla in IE, thus making Moz look smarter 
than IE. But, it never quits.

How about just a simple flag in the user prefs.js file?
user_pref("", false);

Comment 12

18 years ago
Stuff like this isn't going to get fixed between nsbeta3 and rtm. Changing 
Keywords: rtm → helpwanted, nsbeta3

Comment 13

18 years ago
nav triage team:
fyi, this is not "imported", it is a live feed of the folder on your desktop 
called "IE Favorites" so it is always up-to-date.  Vera is planning to add to 
documentation that the user can remove the folder from the disk drive and 
hopefully that will take care of the problem.

This is actually a feature request, so -
Whiteboard: [nsbeta3-]

Comment 14

18 years ago
Fine it's a feature request to supress a poorly implemented feature.

Relnote3, and RTM: "It is impossible to permanently delete this folder from 
your mozilla/netscape6 bookmarks" [have a nice day].
Keywords: relnote3, relnoteRTM

Comment 15

18 years ago

That doesn't solve the problem for people like me:
My brother uses IE, I use Mozilla. I don't want his bookmarks. This means that I 
can't remove the IE bookmarks folder, as you suggested as a solution to the 

There needs to be a way to turn accessing the IE bookmarks off.

Comment 16

18 years ago
A similar situation: I can't delete the IE Favorites folder on this computer. 
`You cannot move "Favorites" from the folder "System Folder", because you do not 
have the privilege to make changes.'

Filed bug 52431 about synchronizing Mozilla's bookmarks with those of another 
browser, instead of this hokey folder thing.

Comment 17

18 years ago
This could probably be fixed by hacking our magic entry so that when it is 
"deleted" it is marked with some attribute that prevents it from displaying.  
This would probably map to a preference that lives near the Go|Search|Bookmarks 

At first I thought that the correct thing to do would be to leave this flag in 
the bookmarks file, as I thought about the undelete method I considered it as a 
preference.  I don't know enough about how we handle magic folders to know if 
we could trap the delete event for just one.

Comment 18

18 years ago
The "Imported IE favorites" bookmark doesn't show a popup menu either. At the
very least right-clicking on it should display a popup allowing the user to
delete the folder (permenantly of course)

Comment 19

18 years ago
*** Bug 56640 has been marked as a duplicate of this bug. ***
Whiteboard: [nsbeta3-] → [nsbeta3-] relnote-user

Comment 20

18 years ago
this is covered in bug 56765

*** This bug has been marked as a duplicate of 56765 ***
Last Resolved: 18 years ago
Resolution: --- → DUPLICATE

Comment 21

18 years ago
It is not. Please keep this separate. Thank you.
Blocks: 56765
Resolution: DUPLICATE → ---

Comment 22

18 years ago
win32 build 110420 crashes when the IE favorites folder is deleted.  
Running on win98SE. Steps to reproduce:

1. Start Mozilla
2. Select Bookmarks|Manage Bookmarks
3. Select IE Favorites folder and hit delete key
4. Close Mozilla
5. Crash

Talkback did not kick in, but here is console log. Dr Watson attached.

Contents of console log:
has multiple monitor apis is 1
Move window by 580,484.5
New location for profile registry and user profile directories is -> 
start with profile: Default
ProfileManager : StartApprunner
profileName passed in: Default I am inside the initialize
 Hey : You are in QFA Startup 
(QFA)Talkback loaded Ok.
Setting content window
*** Pulling out the charset
Loading page specified via openDialog
in SetSecurityButton
Document loaded successfully
Hey : You are in QFA Shutdown 

Comment 23

18 years ago
Created attachment 18756 [details]
Dr Watson of crash

Comment 24

18 years ago
Reassigning 79 Bookmarks bugs to Ben.  I was told this was going to be done 
shortly about two months ago, but it clearly hasn't been.  I think that's long 
enough for all these bugs to remain assigned to nobody.

Feel free to filter all this spam into the trashcan by looking for this string 
in the message body: ducksgoquack
Assignee: slamm → ben

Comment 25

18 years ago
reported 11/14/2000 by Jacob Arnold at
Keywords: nsmac2

Comment 26

18 years ago
reported 11/15/2000 by Jim Thomason at

Comment 27

18 years ago
reported 11/15/2000 by Steve Palm at

Comment 28

18 years ago
Removing myself from list of cc's. I hope this gets fixed!

Comment 29

18 years ago
This was a common complaint from Netscape 6 users, and should be addressed for 
Severity: minor → normal
Keywords: nsbeta3, relnote3 → nsbeta1
Whiteboard: [nsbeta3-] relnote-user → relnote-user
reported 01/08/2001 by Art Hughes on n.p.m.wishlist ("various issues", no.3)
Netscape Nav Triage Team: adding german to the cc list.  we need to design this 

Comment 32

18 years ago
nav triage team:

Yeah, we need to solve this issue, but not now. Marking nsbeta1-
Keywords: nsbeta1 → nsbeta1-
Target Milestone: --- → Future

Comment 33

18 years ago
*** Bug 68926 has been marked as a duplicate of this bug. ***
reported 2/16/2001 by Jerry Park on n.p.m.seamonkey

Not now? Sorry, but I can't follow. This is in the top 12 of the voting charts,
and sounds like it has a good reward/effort ratio. Re-nominating by replacing
the nsbeta1- keyword with the nsbeta1 keyword, per keyword description. Please
have a second look. If it get's nsbeta1- again, I'll stop whining.
Keywords: nsbeta1- → nsbeta1

Comment 35

18 years ago
Created attachment 25587 [details] [diff] [review]
Proposed patch

Comment 36

18 years ago
Above patch adds a preference for importing the Favorites on startup. The UI 
for the prefs needs a bit of cross-platform work (presumably Unix doesn't need 
the pref, and BeOS needs to refer to NetPositive bookmarks?) but this can be 
done if the approach is sound. Review?

Comment 37

18 years ago
sounds like IEFavorites is the wrong term.
ben, can we rename it systemFavorites?

Import is inaccurate, let's use Include.

this whitespace change is bad:
-  </titledbox>              
+  </titledbox>             
if you're going ^ to change trailing whitespace please trim all of it.

asside from my wording nits, this patch looks great.

german: thoughts about Include?
Keywords: approval, mozilla0.9, patch, review

Comment 38

18 years ago
Renaming of IE Favorites is currently bug 68858.

Anyone in a position to test on a Mac?

Comment 39

18 years ago
I can test it, but I need a binary...

Comment 40

18 years ago
+<!ENTITY  IEFavorites.label             "IE Favorites">

We should probably use `Internet Explorer™' rather than `IE', because more people 
will know what `Internet Explorer' is than what `IE' is. (Abbreviations are 
generally bad UI, except for units of measurement.) I don't think we should use 
`system favorites', for cross-platform reasons -- Internet Explorer favorites 
aren't system favorites on either Mac OS or Solaris (where IE is also present), 
and it's probably not worth doing platform-specific stuff for this bug until this 
fix is made obsolete by the fix for bug 56765.

+<!ENTITY  IEFavoritesExplanation.label  "&brandShortName; can create a bookmark 
to your IE Favorites on startup.">

It's not a bookmark, it's a folder full of bookmarks. And they're not created on 
startup, they're dynamic based on the contents of the Favorites folder.

+<!ENTITY  importIEFavorites.label       "Import IE Favorites on startup">

This is the only entity you should need. Having a group box, and in-line help 
text, just for a single checkbox which has a relatively simple function, seems 
rather extreme.

[/] Show Internet Explorer™ "favorites" in their own bookmarks folder

Comment 41

18 years ago
>It's not a bookmark, it's a folder full of bookmarks. And they're not created 
>startup, they're dynamic based on the contents of the Favorites folder.

I think that what I said describes what actually happens - although not 
necessarily what it looks like to the user. There *is* a single bookmark 
created, which points to the location of the IE Favorites folder on disk - you 
can create one yourself, or for other directories, such as the desktop.

The reason I mentioned startup in the preference text is that unchecking the 
preference does not instantly delete your IE Favorites folder, you have to 
restart for that to happen.

Comment 42

18 years ago
"Create bookmark folder containing Internet Explorer favorites when &brandname; 
starts up" ?

>This is the only entity you should need. Having a group box, and in-line help 
text, just for a single checkbox which has a relatively simple function, seems 
rather extreme.

It is consistent with the current bookmark prefs though. Which group box should 
this new pref be included in?

Comment 43

18 years ago
As long as the UI isn't lying, it's what it looks like to the user (rather than 
how it is implemented) which is important.

> Which group box should this new pref be included in?

It shouldn't. Prefs in group boxes should be the exception, rather than the rule.

Comment 44

18 years ago
Created attachment 26054 [details] [diff] [review]
Revised patch incorporating UI suggestions

Comment 45

18 years ago
adding self to CC, timeless wants me to modify the patch

Comment 46

18 years ago
Created attachment 26113 [details] [diff] [review]
previous patch with IE --> system and pref added to all.js

Comment 47

18 years ago
I note that
1) the UI will still refer to Internet Explorer (pref-bookmarks.dtd)
2) nsBookmarksService.cpp still refers to the pref
browser.bookmarks.import_IE_favorites not browser.bookmarks.import_system_favorites
3) there's an unrelated history change in there

Comment 48

18 years ago
Created attachment 26120 [details] [diff] [review]
Revised patch

Comment 49

18 years ago
That's what I get for posting a patch when tired, sorry for the spam.
1) I think the UI strings should be platform dependent because BEOS is 
mentioned in nsBookmarksService.cpp but there are no entities.  Ask timeless 
for the specifics.
2) Fixed (thanks for the heads up)
3) Removed

Comment 50

18 years ago
re: 1. I originally suggested that this bug would need cross-platform work
(NetPositive Bookmarks on BeOS, not applicable on Linux etc). but Matthew Thomas
thought that "it's probably not worth doing platform-specific stuff for this bug".

Comment 51

18 years ago
*** Bug 70127 has been marked as a duplicate of this bug. ***

Comment 52

18 years ago
The latest patch:
*   still refers to `import_system_favorites' rather than `import_ie_favorites',
    potentially causing confusion with any future pref which really does include
    the system favorites rather than the Internet Explorer favorites;
*   still talks about creating the folder `on startup', when as far as the user
    is concerned that is irrelevant.

Neither of those are huge problems, though.

Comment 53

18 years ago
mpt: yes the idea is to refer to system favorites, not ie favorites, because we 
were abusing the elements to have IE Favorites refer to netpositive bookmarks 
which makes no sense.

Note to self, I owe this patch some love for the preference ui. [refactor dtd 
values into platform files]

Comment 54

18 years ago
mpt: If we do not mention startup, then it looks as though unchecking the 
preference will instantly delete the Imported IE Favorites folder; it will not.

Comment 55

18 years ago
mpt: there are three possible solutions to the "on startup" problem:

- add a text that explains that the changes will take effect the next time he 
restarts the browser
- when the user checks/unchecks this option, bring up a pop-up telling her/him 
when the changes will take effect
- actually delete/insert the bookmark when the user checks/unchecks the option

I prefer the third solution, but I don't know if the insertion can be done...

Comment 56

18 years ago
As a user I would like the option to import them, move them around, and so on, 

without having them be in a "ie favorites" folder.

Comment 57

18 years ago
 That's more closely related to bug 56765 then this one.

Comment 58

18 years ago
*** Bug 70822 has been marked as a duplicate of this bug. ***

Comment 59

18 years ago
i haven't looked at the actual text of the patch but can't you just change the wording from
'on startup' to 'after restart'?  I think most folks get the ramifications of that. (i.e. not now but later)

Comment 60

18 years ago
I didn't have to close Mozilla for it to crash on me --all I did was try to 
delete the IE folder.  After hitting the delete option in the editing menu, 
Watson showed up with a 'c0000005 (access violation)'.

This is on NT with build 2001031008

--C. Labombard
The fix to allow the IE Favourites folder to be turned off is (as far as I can 
see) to set up the pref, and then add code to check for it in the tests on line 
4914 (Mac) and 4217 (Windows) of nsBookmarksService.cpp. 

So, we can have a UI option which immediately sets a pref. Then, the next time 
the Bookmarks menu is pulled down, they'll be gone. (If they are cached, we 
merely call ReadBookmarks again.)


Comment 62

18 years ago
Gerv: s/4914/4194. And isn't that pretty much what the current patch does?

>Then, the next time the Bookmarks menu is pulled down, they'll be gone.

Doesn't happen. (because the Imported IE Favorites directory is saved out as 
part of the bookmarks?)

Are there actually any problems with the latest patch? What is stopping it 
being reviewed and checked in?

Comment 63

18 years ago
*** Bug 74195 has been marked as a duplicate of this bug. ***

Comment 64

17 years ago
nav triage team:

This is not a Netscape priority for beta, marking nsbeta1-. The mozilla community 
is more than welcome to fix this bug, though. ;-)
Keywords: nsbeta1 → nsbeta1-

Comment 65

17 years ago
51 votes and a patch - what more does the "mozilla community" have to do?

Although the patch itself now needs

-  <checkbox id="importSystemFavorites" value="&importSystemFavorites.label;"
+  <checkbox id="importSystemFavorites" label="&importSystemFavorites.label;"

following the XUL syntax changes.
> what more does the "mozilla community" have to do?

0. agree that with the patch applied things are better than without
1. get a review and a super-review
2. check it in

Someone probably needs to volunteer to drive this to the finish. To indicate
that you are willing to do this, just take the bug (reassign it to yourself).
Ben, can you review or comment, please? Or recommend someone else for review?

Comment 67

17 years ago
*** Bug 76548 has been marked as a duplicate of this bug. ***
This requires some more thought. Placing into .9.2. as it is a popular bug 
however and causes some bookmarks usability issues. 
Keywords: mozilla0.9
Target Milestone: Future → mozilla0.9.2
Here's the solution I want: 

No preferences panel UI. This is more of a view option than a checkbox in 

Show the folder by default. Allow the user to delete it, perform a delete and 
then update the pref. Have a menu item in the view menu maybe, with platform 

[x] Internet Explorer Favorites (Windows and Mac). 
[ ] NetPositive Favorites (BeOS)

Not sure what the work is like for Mac and BeOS, but should be fairly 
straightforward for Windows. 

Checking the item will recreate the folder. 

I'm not going to get to this before .9.2, and then outliner work may take 
priority, but someone who produces a patch along these lines will be greatly 
appreciated ;) Please note that I want an XP solution as that's the way I'd do 
it myself. 

Comment 70

17 years ago
*** Bug 77316 has been marked as a duplicate of this bug. ***

Comment 71

17 years ago
*** Bug 78160 has been marked as a duplicate of this bug. ***
Created attachment 32742 [details]
changes to fix bug (back end + FE removal)
This is not actually a patch, (well, it was extracted from a larger one). 
It's possible to hand-apply. Implements support required from 
nsBookmarksService.cpp for removal of system favorites folders. Adds the same 
pref from previous patches to disable the adding of the folder on startup. Hooks 
up a set of this pref into the delete function in bookmarksOverlay.js. 

Still left: add a checkbox to the 'System' panel in preferences to allow this to 
be re-activated. 
This patch should also prevent the addition of multiple favorites folders, as it 
does a case insensitive compare for filename. 
Ben - great!

> This is not actually a patch, (well, it was extracted from a larger one). 
> It's possible to hand-apply. 

Does this mean that:
a) You are working on this in another patch and it'll get checked in as part of
that soon, or
b) You want someone else to pick this code up, make a proper patch and add the
pref to the prefs UI somewhere?


Comment 76

17 years ago
Can we not call NetPositive bookmarks IEFavorites?

Comment 77

17 years ago
*** Bug 79512 has been marked as a duplicate of this bug. ***

Comment 78

17 years ago
marking mostfreq: 9 dupes + the press reports.
Keywords: mostfreq
*** Bug 80071 has been marked as a duplicate of this bug. ***


17 years ago
Keywords: mozilla1.0

Comment 80

17 years ago
nav triage team:

Not an ns stopper, marking future
Target Milestone: mozilla0.9.2 → Future

Comment 81

17 years ago
Oh man. This bug has 70 votes, 9 dupes, and was one of the oft-commented on 
"features" of NS 6 that people disliked. Why why why?

Comment 82

17 years ago
i tried to use my other 9 votes for this bug, but it won't let me vote for it 
more than once.  the sooner this is fixed, the better, as far as i'm concerned.  
i'd prefer a few of the crashers i get over this.

Comment 83

17 years ago
This is one of those things that keeps me from using Mozilla as my main 
browser. It's so annoying...

And it's been there SOOOOOO long.

Please change this behavior.

Make it an option (Would you like to import you IE bookmarks?)

Comment 84

17 years ago
I thought this was mostly done, and was only lacking a UI for it.  If that's 
true, how do I change my prefs to get rid of the IE favorites?

Comment 85

17 years ago
I like the fact that you have an up-to-date version of your IE bookmarks, which
can be useful if you use both browsers. Although it would be nice to make it
either optional, and/or allow adding/moving/copying/deleting to/from the IE
bookmarks folder.

Comment 86

17 years ago
Ben, where are we up to with your patch?

I have a UI patch if people are interested. It adds "Internet Explorer 
Favorites" (or the platform equivalent) to the Bookmark Manager/View menu.


17 years ago
Blocks: 81552

Comment 87

17 years ago
I follow Simon's lead in asking 'why' here.  Ben is a busy guy, but I don't see 
how this bug is less important than the other plussed bugs on his plate.  Are 
people expected to complain more about bug 75470 than this?  Or 68985 or 81213?

Renominating; this should also be catfood, because more than a couple reviews 
mentioned it.  I would be happy to help out in fixing this (I'll talk to Ben 
about where he is on this...)
Keywords: nsbeta1- → nsCatFood
Target Milestone: Future → ---
Pessimistically 0.9.3, as I already have a patch
Target Milestone: --- → mozilla0.9.3
nav triage: not sure why this is catfood. its a nice to have - hence scheduled
into m0.9.3. if someone wants to take it, thats fine too ;). 
Keywords: nsCatFood → nsCatFood-

Comment 90

17 years ago
SEMI-SPAM: Vishi is not sure why this is nsCatFood??? Doesn't he know what that
means??? Hasn't he seen all the CC's, all the votes??? All the comments for this
bug??? Vishi, if this isn't a "serious user *satisfaction* issue" (not user
"usability") and your managers don't slap you upside the you-know-what, then
Mozilla will surely fail - steamrolling over such obvious user satisfaction
issues is no good omen. Please remove the #*§%& minus sign from nsCatFood!!!
END-OF-SPAM, and apologies for loosing my "cool" (it's 3:00 AM).
Created attachment 36896 [details] [diff] [review]
diff version of earlier patch.
So I've had this patch in my tree for some time now, and it'd be a little hard 
to remove it, and since the code collides with other nsbeta1+ fixes I have, I'd 
like to just check it in. I don't see any harm, it seems to work. Need review, 
super review. 

Comment 93

17 years ago
r=hixie, with two minor comments: first, your diff is screwed up (one of the
files is in there twice??) and second, dude, remove the tabs...

Comment 95

17 years ago
apart from the horrrrrible indentation this patch looks good to me as well,
unfortunately I could not test it as I build only on linux, and bill knows that
there is no such thing as favourites on linux 
Tabs are a long standing issue in this file (you can thank rjc). Since this 
isn't really a large modification to the file, I've chosen not to start a fight 
in my text editor (as it'd eventually end up in a war, with many casualties, 
botched braces and failed compilations). 

I plan on reformatting the file function by function as I make localized 

As for the double patch, oops, that'll teach me to use >> too liberally ;) 

Comment 97

17 years ago
a= for checkin to the trunk.
(on behalf of drivers)
Patch checked in.
Last Resolved: 18 years ago17 years ago
Resolution: --- → FIXED

Comment 99

17 years ago
I can't seem to delete any of my bookmarks now. As soon as I click on one, the
"delete" button is disabled, and never re-enables.

Comment 100

17 years ago
Skewer, that's bug 83642. It's unrelated to this bug because bug 83642 showed up
a few days ago.

Comment 101

17 years ago
Is this really fixed, if that there is no UI for it? Or is there a UI patch on 
the way?

Comment 102

17 years ago
Created attachment 37007 [details] [diff] [review]
Patch to enable Delete for the IE Favorites folder
Oops, forgot about that. Delete key and Delete button on toolbar work though.
Can we take the details to a new bug to prevent spam? 

Comment 104

17 years ago
Sounds like a good idea.

Ben, I have a more complete UI patch if you are interested in seeing it.

Comment 105

17 years ago
Works for me 2001060504 Win98. I don't remember how I deleted it but it hasn't
come back.
verified fixed with win2k build 20010605.. (CVS opt)
BTW: how can I get the IE Bookmarks back :-)

good question!
Open your prefs.js file and remove the pref for

Not the best solution, so file a new bug on me to create a UI for this. mpt has
suggested the system panel in the past, but lets leave that for a new bug. 

Matthew Wilson, could you also file a bug on me for your patch? Thanks. 


17 years ago
Blocks: 84281

Comment 108

17 years ago
OK, I created bug 84281 for "Prefs UI to show/hide Imported IE Favorites folder".

That way, we can get the IE favs back if we ever change our minds or
accidentally deleted them ;)

Comment 109

17 years ago
Has this regressed on newer builds ? (2001122603)


17 years ago
Blocks: 120814


17 years ago
No longer blocks: 120814

Comment 110

17 years ago
Sorry if I missed the obvious, but this bug is marked "fixed" and "closed" back
in 2001 for target milestone 0.9.3, and here I am in 2002 with a brand new
milestone 0.9.9 (build 2002031104 on Win98SE) and I still don't know how to
*permanently* delete that stupid IE favorite folder Mozilla keeps creating each
time I start it...
Yes, I did read through the thread, and yes, I did notice a handfull of patches,
but I don't think installing a 0.9.3 patch on my 0.9.9 build would be a good
idea. And AFAIK usually a patch gets integrated into future versions. What am I

Comment 111

17 years ago
Please see bug 84272. If that isn't what you want, please open a new bug. 
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.