Closed Bug 27538 Opened 25 years ago Closed 24 years ago

Bookmarks without a title look terrible in menus

Categories

(SeaMonkey :: Bookmarks & History, defect, P3)

x86
Windows 98
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.0

People

(Reporter: hyatt, Assigned: paulkchen)

Details

Make a bookmark with no title.  It shows up as being only 2-3 pixels high when 
used in a menu.

We should probably create a title, e.g., (untitled), for any empty bookmarks and 
show that in the menu.
Steve?
Assignee: rjc → slamm
Wait, I was proposing a fix in the back end, not the front end.  The bookmarks 
look stupid in the tree view too.  I'm proposing that the RDF layer make up a 
title, e.g., (untitled), and then it would show up everywhere.  I don't think 
slamm should have to do this in the front end.
Reassigning as per Don
Assignee: slamm → hyatt
give them a title in the back end.
Assignee: hyatt → rjc
I'm not against seeing this fixed. However...

I believe that this should be fixed with CSS to ensure that menu items don't 
become 2-3 pixels high, instead of hacking the backend.

Why?  Due to aggregation, a user could end up aggregating in data (say, search 
results, or related links, or ...) that didn't specify a name, and we'd have the 
same problem.

A weak yet valid argument is that 4.x allows bookmarks without a title.  
Personally, I think having "(untitled)" has no more value than no title.

So, in terms of fixing the bookmark's menu being only a few pixels high, seems 
like CSS is the right answer.

David, opinions?
Assignee: rjc → hyatt
hey, couldn't a template rule do this?
.
Assignee: hyatt → ben
taking bug from Ben
Assignee: ben → pchen
Status: NEW → ASSIGNED
retargeting to mozilla1.0
Target Milestone: --- → mozilla1.0
Netscape Nav Triage team - marking as fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
bookmarks with no title now have -->no title, but at least they don't screw up the menus.
checked modern and classic 2001041004 builds
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.