Splitting this out from #1516446: Autocomplete for internal links for nivo slider link fields
Lengthy descriptions/captions lead to overflow in the document area see attached screen shot where I've highlighted the full text to clarify the overflow issue. I'm not sure if truncation and/or "read more" is a good solution here due to the various sizes of the markup the user may include. Also this is for the homepage editorial review by an actual person is likely the best solution.
From #1516446: Autocomplete for internal links for nivo slider link fields lightsurge indicates it would be nice to have a read more link if there is a nivo slider link provided. Right now if a nivo-slider link is present the image is clickable and takes you to the destination. It may not be clear to users that some slides are clickable while others may not be. Adding the read more link makes this more clear, but likely clutters up the slideshows.
Comment | File | Size | Author |
---|---|---|---|
#2 | overflow_readmore-1528702-2.patch | 2.55 KB | lightsurge |
julio-nivo-slider-caption-overflow.png | 443.21 KB | jgraham |
Comments
Comment #1
lightsurge CreditAttribution: lightsurge commentedI think in terms of the overflow, it might work quite well to place a maximum string length on the Description field that the caption gets its content from, and disable/strip markup, essentially making it a plain text field... otherwise it could easily end up not very pretty.
Then as far as a read more link goes... A form_validate could be added in the slideshow feature to add a simple read more link to the slides with something like:
and css to position it:
Comment #2
lightsurge CreditAttribution: lightsurge commentedThis patch
This approach would also solve #1535568: Nowhere to set a slide heading
Here's a screenshot of the outcome with what would have been overflowing text and no read more link:
I sort of thought all of this might be better done in the slideshow feature, though, unless you're planning on having the ability to create a slideshow of all galleries rather than just the front page one?
Comment #3
penguininja CreditAttribution: penguininja commentedThis looks great, though I haven't had a chance to test yet. Yes, ideally this would be in the slideshow feature. I'll check this out further as soon as I have time.
Comment #4
jgraham CreditAttribution: jgraham commentedCommitted 5da19d1, a modified version of the patch in 2
Thanks lightsurge! Patches really help get things moving along.
Comment #5
lightsurge CreditAttribution: lightsurge commentedDoes text_summary() allow for links etc being in there? Bit lazy to ask rather than to look, but that's great, wish I'd known about that function ;-)
Comment #6
jgraham CreditAttribution: jgraham commentedyeah, text_summary() you're pretty much at the whim of the filter format of the field. In this case filtered_html.