Posted by redsisok on April 1, 2009 at 7:10pm
Jump to:
| Project: | Slide Show CS |
| Version: | 6.x-2.x-dev |
| Component: | Miscellaneous |
| Category: | feature request |
| Priority: | normal |
| Assigned: | davebv |
| Status: | closed (fixed) |
Issue Summary
Hi,
I am really new with drupal and I have just installed SlideShowCS, so I am a bit worried that my request it's actually caused by the fact that I couldn't find the functionality that I am looking for.
Anyway, I was looking at the last example of the slideshow given in this page http://www.gruppo4.com/~tobia/cross-slide.shtml and I can see that from there it is possible to click on the images as they have an "href" attribute.
If this is not already possible on the Drupal version I would like to request it, as it could be a great feature to include in this module.
thanks.
Comments
#1
Do not worry, this is not implemented in this module. I think is a good idea for the module to add this parameter, so let's start a discussion about what to do with that field:
Proposals:
a) Link to picture
b) Totally customized link in the block settings for each picture
c) ...
I am looking forward to hear what you would expect to enter in the href field.
#2
As a first draft I think just having the ability to add our own urls for each image would be fine. We are using Slide Show CS to display upcoming event ads, and just having the ability to link to an event's page would be just fine. I would think that always leaving the ability to just define a static href for an image would be a good thing, even if some other dynamic options were added later.
Josh
#3
Ok, that is what I will do. I do not have so much time right now, so it will take a little bit long. Anyway if any of you are able to make a patch, I can test it, otherwise, wait until I find a gap to work on it.
thanks for the suggestion!
#4
Ok! I am uploading a patch with this functionality implented. Just added a field and then you may enter the url you want to link the picture to.
This will be in the 2.x-dev shortly.
#5
Ok, it should be now in the dev version. Please, wait until the packages are generated, try it and report any problems.
If this works as expected, I will create a new official release.
#6
Please, wait for the next release, I found a small bug which may prevent long urls to be entered.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.