304740 - Not able to add a bookmark with an empty title, while it is possible to make it empty afterwards

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Martijn Wargers (dead)]

This was mentioned here:
http://forums.mozillazine.org/viewtopic.php?t=298128&sid=71d245051c2fc3ec81f7dd2469924b7d

So this should be handled the same, both should not allow an empty name, or they
should allow it both.
My opinion would be to allow it in both places.

Comment 1

[Martijn Wargers (dead)]

Attachment: patch

Ok, this makes it work for me.
With this I am allowed to add a bookmark with an empty name.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

If you're getting rid of onFieldInput, you can remove gRequiredFields too, however that would remove the check for a keyword when using "Add a keyword for this search", I don't think that's wanted.

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/base/content/browser.js&rev=1.546&mark=5814#5808
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/components/bookmarks/content/addBookmark2.js&rev=1.38&mark=126#120

Comment 3

[qbxk]

wouldn't it be simpler to just comment out line 71 in addBookmark2.js - seems to do the job, with a whole lot less code.

that line in my 1.5.0.1 release reads:
   gRequiredFields.push(gName);

so... just don't do that...

for the record, this might seem like a pointless feature, but here's why i want it:  i like to have a whole bunch of links on my toolbar, but why not just have the "favicons" up there, i can fit so much more!  who need's text.  currently the procedure for adding a new "bookmark icon" is: 
   1) right-click, "bookmark this page..."
   2) "OK"
   3) right-click new bookmark (freqently on the "spillover" menu on the toolbar), "properties"
   4) erase "name"
   5) OK

changing this file would bring eliminate steps 3-5 but move step 4) to 1b)

coming soon: my bug report search for why you don't get a favicon for new bookmarks until the first time you click it (you get the default "blank page" icon at first)

Comment 4

[qbxk]

Attachment: Why I like bookmarks with no names.

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Fixed by bug 344459.