Hello. First of all let me tell you that I find your plugin wonderful. It's all I wanted for links to appear.
I did like the idea though, that the links should appear categorized, and, even better, with the collapsible fields that appeared on the admin section.
So .... I took the opportunity to play a little bit with the module (I'm NOT an expert on php coding, just trying to find my way in it). I managed to do what I wanted to do, but I think it still needs some coding/fixing. Nevertheless, I found it a good opportunity to share it with you.
The module has alternations on lines: 768, 771, 777, 783-830 (I'm mentioning this for you to be able to trace it down).
Unfortunately I don't have module live to see a demo, since I'm still setting it up on my test server.

Again, thanks!

PS. (Remove the extension .patch from the attachment)

CommentFileSizeAuthor
#10 weblinks31.patch21.74 KBnancydru
weblinks.module.patch45.21 KBvangelisp

Comments

ThePiston’s picture

what exactly did you change?

rmiddle’s picture

adding to my ToDo list.

Thanks
Robert

vangelisp’s picture

Yes, apparently, the way I've wrote it is non-understandable: It creates collapsible links.

The patch does have some issues though. If it finds a subcategory, it doesn't care to make it collapsible, it spits it out as it is. A demo of the collapsible weblinks is here :
http://www.motoridersclub.com/en/weblinks

PS. And forgive me for this huge delay on my response!

rmiddle’s picture

That is pretty cool. Need to looking into this once I get the category code fixed.

Thanks
Robert

rmiddle’s picture

Title: Some modifications on the module » Make /weblinks page collapsible.
nancydru’s picture

Assigned: Unassigned » nancydru

I should be able to include this as part of the overall theming work.

The funny thing is that the overview page is already collapsible field sets...

rmiddle’s picture

I know NancyDru was working on this need to check on it.

Thanks
Robert

nancydru’s picture

Still at it

nancydru’s picture

Status: Needs work » Needs review

Here's the 5.x version of a patch to be tested. It is based on the most recent -dev version. The patch also includes some refactoring and revisions in the blocks.

nancydru’s picture

StatusFileSize
new21.74 KB

Uh, oh, another blonde attack...

nancydru’s picture

Status: Needs review » Fixed

Committed to both branches.

nancydru’s picture

Status: Fixed » Closed (fixed)