Closed
Bug 10198
Opened 25 years ago
Closed 12 years ago
Warn user if adding an existing bookmark.
Categories
(SeaMonkey :: Bookmarks & History, enhancement, P4)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
RESOLVED
WORKSFORME
Future
People
(Reporter: CodeMachine, Unassigned)
References
Details
(Keywords: helpwanted, Whiteboard: [2012 Fall Equinox])
Quoting a newsgroup posting:
---
Subject:
Bookmark management wish
Date:
Thu, 15 Jul 1999 15:00:23 +0100
From:
Alien Conspiracy <alien.conspiracy@dial.pipex.com>
Organization:
Another Netscape Collabra Server User
Newsgroups:
netscape.public.mozilla.wishlist
Hi All,
I don't know if this is a common problem or just me, but I often
indevertently bookmark pages that are already in my bookmark list. So it
would be nice if:
(1) There was some on-screen indication as to wether the page currently
being viewed is bookmarked or not.
(2) The 'add bookmark' command checked the existing bookmarks and warned
me if it's a duplicate.
Cheers!
---
This bug report concerns only the second, warning if adding a duplicate.
Severity: normal → enhancement
QA Contact: leger → claudius
Target Milestone: M15
Updated•25 years ago
|
Component: Autofill → XPApps
OS: Linux → All
Hardware: PC → All
Summary: Warn user if adding an existing bookmark. → [RFE]Warn user if adding an existing bookmark.
Comment 2•25 years ago
|
||
just doing some housekeeping on the fields.
Updated•25 years ago
|
Component: XPApps → Bookmarks
Comment 3•25 years ago
|
||
This seems like a nice feature... but it does not appear to be making the M15
checkpoint. We're in the stability freeze for M15 now... so I'm pushing this
off to at least M16
Target Milestone: M15 → M16
Comment 4•24 years ago
|
||
owner seems to be inactive, so i think this bug should be assigned back to
don@netscape.com
M16 has been out for a while now, these bugs target milestones need to be
updated.
Comment 6•24 years ago
|
||
Based on lack of response to Jesse Ruderman's comment above, assigning to
default owner and cc:ing previous owner to get bug back on somebody's radar.
Clearing M16 since M16 is past.
Assignee: alterego → slamm
Status: ASSIGNED → NEW
Target Milestone: M16 → ---
Comment 7•24 years ago
|
||
with workloads as they are there is very little chance any netscaper is going to get to this.
this would be a great spot for a mozilla contribution - adding helpwanted keyword.
Keywords: helpwanted
Comment 8•24 years ago
|
||
The "search" sidebar is now mysteriously and quietly excluding the "add to
bookmarks" option when you right-click on a link to an existing bookmark.
Nothing similar is happening when you right-click a link in the content area.
2000091508 Windows 98
Comment 9•24 years ago
|
||
Reassigning to default component owner since slamm isn't here anymore.
Assignee: slamm → ben
Comment 10•24 years ago
|
||
nav triage team:
Would be nice to have, but don't think we'll do it for beta. Also, need UE to
decide if this is worth doing
Keywords: nsbeta1-
Comment 11•24 years ago
|
||
Marking nsbeta1- bugs as future to get off the radar.
Target Milestone: --- → Future
Comment 12•24 years ago
|
||
See also bug 10197, "[RFE] Show whether the current page is in the user's
bookmarks."
Comment 13•23 years ago
|
||
*** Bug 32964 has been marked as a duplicate of this bug. ***
Comment 14•23 years ago
|
||
From the duplicate: nsIBookmarksService has a function, IsBookmarked(), that
returns PR_TRUE if a URL is already in the user's bookmarks.
Updated•23 years ago
|
Status: NEW → ASSIGNED
Updated•23 years ago
|
Priority: P3 → P4
Comment 15•23 years ago
|
||
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
Comment 17•23 years ago
|
||
nsbeta1+ per Nav triage team, ->1.0. should prevent adding duplicates, not just
warn.
Keywords: nsbeta1+
Target Milestone: Future → mozilla1.0
Comment 18•23 years ago
|
||
Trudelle:
> Should prevent adding duplicates, not just warn.
I hope you meant that as a temporary workaround for bug 51683 rather than as the
desired behavior.
Comment 19•23 years ago
|
||
Please update this bug with an [adt1] - [adt3] impact rating (or take it off the
list if it doesn't even rate adt3.) Thanks!
Comment 20•23 years ago
|
||
Jesse: yes, this is a temporary workaround; we do want to support URLs
bookmarked multiple times. ADT3
Whiteboard: [ADT3]
Updated•23 years ago
|
Comment 21•23 years ago
|
||
Since this RFE will prevent a lot of confusion (see bug 51683), I really think
it needs to get into 1.0. It seems simple enough to implement (says the
non-programmer;-) without creating a lot of new bugs.
I've also adding it as a blocker to bug 51683. Even though it doesn't literally
block it, it acts as a workaround to that bug.
Blocks: 51683
Updated•23 years ago
|
Summary: [RFE]Warn user if adding an existing bookmark. → [RFE] Warn user if adding an existing bookmark
Updated•23 years ago
|
Summary: [RFE] Warn user if adding an existing bookmark → [RFE]Warn user if adding an existing bookmark.
Summary: [RFE]Warn user if adding an existing bookmark. → Warn user if adding an existing bookmark.
Comment 22•22 years ago
|
||
Very handy. I'm surprized it isn't in yet. Heck, even my napster clone has
this functionality for downloading. =)
Updated•22 years ago
|
Keywords: mozilla1.3
Comment 23•22 years ago
|
||
This would become another worthless dialog which would annoy people who do want
the duplicate bookmark, and just be dismissed without being read by everyone
else. We don't have a warning on File->Quit for the same reason.
Resolving bug 10197 is a much better fix for this, and has the benefit of not
desensitizing users to dialogs we really do need. I think if bug 10197 were to
be resolved, the demand for this would vanish instantly.
Comment 24•22 years ago
|
||
As compromise, perhaps a function in the bookmark manager to check through the
bookmarks list and prompt at duplication might do the job nicely..
Comment 25•22 years ago
|
||
As a compromise, perhaps a function in the bookmark manager to check through the
bookmarks list and prompt at duplication might do the job nicely..
Comment 26•22 years ago
|
||
Isn't this bug blocked *BY* bug 51683 rather than a blocker thereof? Even
better, it could be blocked by bug 141227 instead.
In bug 51683, people are unable to create duplicate bookmarks -- forget about
warning them, dupes are what they want! The reason they can't create dupes is
because the current incarnation of the Bookmark database is unable to
differentiate between bookmarks with the same URL, ie., the behavior you're
asking for here is already de-facto.
Comment 27•22 years ago
|
||
I think bug 10197 will be a big performance hit and not better than a manual
bookmarks search.
The way IE does it is perfect, just check all bookmarks (files in IE) in the
folder you want to add in.
Comment 28•22 years ago
|
||
*** Bug 203857 has been marked as a duplicate of this bug. ***
Comment 29•21 years ago
|
||
*** Bug 212022 has been marked as a duplicate of this bug. ***
Comment 30•21 years ago
|
||
> This would become another worthless dialog which would annoy people who do want
> the duplicate bookmark, and just be dismissed without being read by everyone
> else.
Says the hardened commandline, anti-GUI Unix user. *I* would very much like
this implemented, and indeed would like a preference that *prevents* duplicate
bookmarks. I don't want more than one bookmark to the same URL and frankly can
see very little use for that feature. If you really want 2 identical bookmarks,
you could make it so that they can be seperated by being in different directories.
> We don't have a warning on File->Quit for the same reason.
When there is more than one tab open, Mozilla most certainly DOES give you a
warning :-)
> Resolving bug 10197 is a much better fix for this, and has the benefit of not
> desensitizing users to dialogs we really do need.
No. That bug's far less useful, and most users wouldn't notice some icon
telling them that the page is already bookmarked.
I suggest adding preference option: 'Allow duplicate bookmarks in the same
folder' (unchecked by default), and if it's unchecked, attempting to add a
duplicate bookmark to a folder will result in a dialog, similar to IE's, asking
whether the user would like to overwrite the existing bookmark.
Comment 31•21 years ago
|
||
*** Bug 220049 has been marked as a duplicate of this bug. ***
Comment 33•21 years ago
|
||
*** Bug 222928 has been marked as a duplicate of this bug. ***
Comment 34•21 years ago
|
||
*** Bug 228488 has been marked as a duplicate of this bug. ***
Comment 35•21 years ago
|
||
*** Bug 234079 has been marked as a duplicate of this bug. ***
Comment 36•20 years ago
|
||
*** Bug 270717 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Product: Browser → Seamonkey
Comment 37•20 years ago
|
||
*** Bug 272103 has been marked as a duplicate of this bug. ***
Comment 38•20 years ago
|
||
*** Bug 273426 has been marked as a duplicate of this bug. ***
Comment 39•20 years ago
|
||
*** Bug 281794 has been marked as a duplicate of this bug. ***
Comment 40•20 years ago
|
||
*** Bug 284997 has been marked as a duplicate of this bug. ***
Comment 41•20 years ago
|
||
*** Bug 291697 has been marked as a duplicate of this bug. ***
Comment 42•20 years ago
|
||
Note: several Firefox bugs were duped to this bug. Firefox bugs should be duped
to Bug 220049. Bug 222928 Bug 272103 Bug 281794 Bug 284997 Bug 291697 were the
Firefox bugs they are now dupes of Bug 220049.
Comment 43•20 years ago
|
||
interesting that the firefox bug 220049 is now marked as "wontfix" while this
one for mozilla is not... what's the difference between the two browsers
regarding to this functionality?
Comment 44•20 years ago
|
||
I'm disappointed that this was originaly flagged (today) as WONTFIX and now
continue to be worried that the new classification of enhancement with a
priority of P4 means it remains as a WONTFIX.
Comment 45•19 years ago
|
||
*** Bug 291844 has been marked as a duplicate of this bug. ***
Comment 46•19 years ago
|
||
For those not wanting to wait for the presence of this feature in the core there
is Bookmark Duplicate Detector 0.0.1 extension -
http://www.extensionsmirror.nl/index.php?showtopic=597 (with maxver bump it
would work on FF 1.0).
Comment 47•19 years ago
|
||
Bookmark Duplicate Detector doesn't work for me: it scans the _entire_
bookmarks collection for duplicates, while I need only to check for existing
bookmarks in the folder in which i'm adding the bookmark. Also in case of
existing bookmark it doesn't inform the user about the folder in which the
bookmarks is already present.
Sometimes I want to bookmark the same page in different folders, and this
extension doesn't help me at all.
Unfortunately this bug still need to be fixed.
Comment 48•19 years ago
|
||
*** Bug 281855 has been marked as a duplicate of this bug. ***
Comment 49•19 years ago
|
||
6 YEARS is a long time to wait on something that's seemingly one of the first
things that a programmer learns -- look for duplicates in an ASCII file. The
add-on isn't perfect in design nor does it work in all releases.
As it seems this is a futile request, I'm removing my address to save world-wide
Internet bandwidth.
Comment 50•19 years ago
|
||
I've just tried the extension "Bookmark Duplicate Detector" with FF 1.5 beta 1
but it doesnt' install. (Install script not found)
I've tried fixing the extension without success.
In any case the extension is really simple.
Why not adding this code directly to FF 1.5 ? It seems trivial.
function addBookmarkAs(aBrowser, aIsWebPanel)
{
var webNav = gBrowser.webNavigation;
var url = webNav.currentURI.spec;
var isDuplicate = isBookmarked(url);
if(isDuplicate) { if(! confirm("This URL is already in your Bookmarks. Do you
wish to add it as a duplicate entry?")) return; }
const browsers = aBrowser.browsers;
if (browsers && browsers.length > 1)
addBookmarkForTabBrowser(aBrowser);
else
addBookmarkForBrowser(aBrowser.webNavigation, aIsWebPanel);
}
function isBookmarked(url)
{
if( ! BMDS || ! RDF || ! NC_NS )
{
alert("Flagrant Error.");
return false;
}
var urlArc= RDF.GetResource(NC_NS+"URL");
var urlNode= RDF.GetLiteral(url);
return BMDS.hasArcIn(urlNode,urlArc);
}
Comment 51•18 years ago
|
||
This bug depends on bug 18152.
Updated•18 years ago
|
Assignee: bugs → nobody
QA Contact: claudius → bookmarks
Comment 52•13 years ago
|
||
Suggestion. Why not flash the Location Bar and then have an animated arrow appear on the bar that points to the Bookmark symbol at the right. This way the person will know that the page exists iu the bookmarks without having a dialog pop up to tell them.
Comment 53•12 years ago
|
||
From Bug 10197, Comment 37
"The "Star bookmark" quick bookmark panel in 2.1b2pre will tell you how many times you've bookmarked the current page."
Also, now you can't add duplicate bookmark, more in Bug 789468#c6, so I'm closing this as worksforme
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Whiteboard: [2012 Fall Equinox]
You need to log in
before you can comment on or make changes to this bug.
Description
•