Closed Bug 101174 Opened 23 years ago Closed 17 years ago

IE favourites need indication of their read only nature in their UI

Categories

(SeaMonkey :: Bookmarks & History, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX
Future

People

(Reporter: dave532, Unassigned)

References

Details

I've created this bug as a spin off of bug 56765, that bug is a RFE but many of the bugs duped against it are really bugs not enhancement requests. Most of the bugs duped against 56765 are because normal bookmark operations on IE favs fail without any error message. This is definitely a bug with severity of at least 'normal'. There's two ways of fixing this: 1) Provide an indication in the UI that IE favs are special (i.e. disable the delete options from the toolbar/context menu, provide a dialog if someone tries to add a bookmark into the IE folder or make IE favs their default bookmark folder) - this bug can track the progress of implementing this. 2) fix bug 56765 meaning there's no difference to the user between Netscape style bookmarks and IE favourites. In fact if this bug is fixed the user should be able to set Mozilla just to use the IE favourites folder as its only bookmark source for those who use IE and Moz interchangeably. Anyway this bug is definitely not a dup of 56765, but if that bug is fixed then this one becomes invalid, but I propose this as a temporary solution which will be used until (if) the other bug is fixed.
See also bug 68858, "Rename 'IE Favorites' to better reflect read-only nature".
Yeah, renaming the folder would be a good first step, so I'll mark that bug as a dependency, however as the favs aren't really imported bug 68858 should be fixed whether or not bug 56765 is implemented, however this bug only needs to be fixed if it's decided to wontfix bug 56765 or as a temporary solution until it is fixed.
Depends on: 68858
could we make ie favorites grey instead of black (or say 50% hue?)
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1
Paul Chen is now taking Bookmarks bugs. For your convenience, you can filter email notifications caused by this by searching for 'ilikegoats'.
Assignee: ben → pchen
Status: ASSIGNED → NEW
This should be a dupe of 68858? 'IE favorites folder (read-only)' .. kinda like the addressbook implemenatation from outlook folders that are really just viewed books view, not actually a book. This is the same kinda thing.
mass reassign of pchen bookmark bugs to ben
Assignee: pchen → ben
having just given up on the bookmark manager and resorted to editing the bookmarks.html file instead, having attempted to copy bookmarks from the IE favourites, i found this bug. seems to me "read-only" isn't an ideal description - the bookmark manager offers a "copy" for IE favourites, but it doesn't actually work properly (should that be a separate bug?) - they are evidently strictly for navigating to the page the IE favourite points to. also i notice the milestone on this bug is set to 1.1, and bug 68858 that this depends on doesn't have a milestone at all... :(
I agree with comment 8. Let's just gray out the "delete" option on IE bookmarks while in the manager. If there's ever a context menu for bookmarks, let's gray out "delete" on that menu for IE bookmarks, too. This is the simplest and most effective solution.
Blocks: 120814
Of course, simply greying-out the "Delete" button in Bookmark Manager doesn't address: - drag-and-drop of shortcuts (within Bookmark Manager) - File (add) a shortcut to the "IE Favorites" folder - renaming etc. What do the programmers want to do on this? Bug 68858 seems awfully easy to implement; it could just as easily be un-done if/when a more sophisticated solution is created.
retargeting
Target Milestone: mozilla1.1alpha → Future
Flags: blocking1.7?
Flags: blocking1.7? → blocking1.7-
Product: Browser → Seamonkey
Assignee: bugs → nobody
QA Contact: claudius → bookmarks
They are real imported bookmarks now
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.