Bookmarks sorted in folders

PECastro - July 29, 2005 - 00:28
Project:Bookmarks
Version:4.7.x-1.x-dev
Component:User interface
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Why not adding some more functionality to BookMars.
I like the module but I think its missing some content separations like separating things by areas like folders in FireFox or IE
You have your bookmarks all messed up in one page and say if you have 100 bookmarks you just can't group them in categories.
It could be fun to be able to categorize them...

#1

noa - August 25, 2005 - 06:50

I totally agree.. Categories and hit counter.. and should be a public module on the site..

#2

Don Robertson - August 29, 2005 - 07:37

I agree - categories would be good. Also like to see a 'Description' for the page.

#3

bigdogglen - October 14, 2005 - 05:59

Is anyone rolling on this? I think this would be an awesome feature. I'm no developer, but I'm thinking you would use a DHTML menu. Not sure if that's the best way to do it tho...

#4

daans - October 24, 2005 - 22:27

I'm also missing this. So at the moment I'm working on some code to give it a start. I think I will have a proof of concept (database and forms to fill the database) ready. So You'll be able to place links in a group/folder at the moment nothing is done with that information but that is from there not that hard to implement (but it is also theme dependent, so needs some proper thinking on how to implement it). I'll probably post an update in a few hours.

#5

daans - October 25, 2005 - 00:55
Status:active» needs work

I fineshed workin on the idea I had in mind what I missed about this module. I only have mysql server (and knowledge) so there's no postgres alteration file. I would love working on this (the only thing I miss so far in drupal is to have bookmarks organised in groups). With one easy setting it can be turned off (remove the corresponding fileds and forms and all links will placed in [Top Level]).

If people think it's a good start I would love to put more work in it. But I'm new to the drupal api. And the hardest part (for me) is still to come. Implementing the catagorized bookmarks correctly in the theme. So that the categories are actually used. So the following things work so far:

  • Add groups
  • Edit groups
  • Assign bookmarks to a group (the bookmark always stays in the group no matter what happens with the group)

Not working:

  • Deleting groups (beacuse a choice have to be made what happens with the bookmarks/groups in the group that is deleted, are they deleted too or are they moved to the parent group of the deleted group, or is this an option that can be set by the site maintainer or by individual users)
  • Using the groups in which the links are to display a categorized view of the bookmarks (never used the api to do this correctly with different themes)
AttachmentSize
groups4bookmarks.patch 10.87 KB

#6

deekayen - February 13, 2006 - 20:08
Version:4.4.x-1.x-dev» <none>
Assigned to:Anonymous» deekayen

How about giving the option at delete time to move the bookmarks or delete them with the group.

#7

deekayen - February 14, 2006 - 06:42
Title:Categories 4 Bookmarks» Bookmarks sorted in folders
Version:<none>» 4.7.x-1.x-dev
Component:Code» User interface

Since this won't be a tiny patch, relative to the size of the module, I've uploaded my work from tonight to my Drupal CVS sandbox: http://cvs.drupal.org/viewcvs/drupal/contributions/sandbox/deekayen/modu...

I used most of daans patch, but I changed instances of "group" to "folder". The bookmark listing page shows the folder name, but doesn't really group by it very well. When folders are deleted, it doesn't delete the child folders (and it should later), but it does re-parent the bookmarks in the folder to the top (not in a folder). I think in the end, it's still going to have to be an option for the user to delete all bookmarks in the folder or re-parent them since neither by itself is probably going to be acceptable to all.

I think to present the folders and bookmarks, it might be proper to consider a UI adjustment. Possibilities might be to use the new forms API collapsible fieldsets (for the AJAX) or the menu system's use of unordered lists, but that might mean lots of page refreshing to open/close folders. In both cases, that would mess up the option the table view has now to sort asc/desc by column. The forms API has a weight option, but that's not really the same as the table alpha ordering it has now. I could really use some help on ideas for implementation.

#8

deekayen - March 25, 2006 - 07:13
Assigned to:deekayen» Anonymous
Status:needs work» needs review

This patch for 4-7 introduces folders (implemented as fieldsets) and the option to display bookmark titles as links. The whole file is in my sandbox.

AttachmentSize
bookmarks_folders.patch 24.85 KB

#9

deekayen - March 31, 2006 - 06:47
Status:needs review» won't fix

Nobody seems to care. I'm forking.

#10

Steve Simms - August 2, 2006 - 15:23
Status:won't fix» active

Hi deekayen,

I just took over maintainership of this module, and would like to see some way of adding folders or categories to bookmarks, if it can be done cleanly and without breaking existing installations.

Would you be interested in merging your fork back in?

#11

deekayen - August 2, 2006 - 19:16

Bookmarks2 shouldn't break existing installations if you add the right stuff to the .install file, because I had to add the folders and password reminders without breaking an existing install. I implemented the features in bookmarks2 for a specific project that I'm done with, so if you want to merge the bookmarks2 stuff into the bookmarks cvs tree, I'll get rid of bookmarks2, but I'm not really interested in sending you a patch.

#12

uandrey - August 7, 2006 - 06:49

On my site I use this module. Bookmark based on node and I use taxonomy module for grouping bookmarks

AttachmentSize
bookmark.module 3.55 KB

#13

darthclue - June 4, 2008 - 15:26
Status:active» closed

The 6.x release supports folders.

 
 

Drupal is a registered trademark of Dries Buytaert.