Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.If a spotlight slide has a title without a description, it can sometimes wrap to a second line:
This is because .panopoly-spotlight-label has a width of 75% of its parent, .panopoly-spotlight-wrapper, which doesn't have any width set. :-)
I think it should just be a matter of setting that to width: 100%.
It looks like this problem was introduced in #2163985: Too narrow Spotlight headline area.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | Skærmbillede 2015-06-10 kl. 22.34.30.png | 440.41 KB | lsolesen |
| #5 | panopoly_widgets-spotlight-label-width-2499289-5.patch | 1.68 KB | dsnopek |
Comments
Comment #1
dsnopekThis turned out to be a little more convoluted than I had hoped. Just doing:
... didn't work, because it could make the label wider than the image, and really, we want the label to fit to the text content in it.
So, here is a patch with a solution that works for me, but since it's not terribly straight forward, I'd like to get some additional testing on it!
@lsolesen: Since this changes the fix for your site in #2163985: Too narrow Spotlight headline area, can you try this patch and see if it breaks your site? Thanks!
Comment #2
cboyden commentedI've tested this on a child distribution and it's working as expected.
Comment #3
lsolesen commented@dsnopek - the first slide on vies.dk shows how it looks with the patch.
Comment #4
dsnopekHrm. So, it doesn't quite work as it should on vies.dk. Here's the screenshot:
I'll try and come up with something else...
Comment #5
dsnopekWow, who would have thought this would be so hard?! The fun of browser sizing. :-)
Previous iterations worked by shrinking the label box if it got to be greater than 85% of the width. This patch instead attempts to grow the description box by 50px if it's smaller than the label (by adding a right margin to the label).
Let's see if this one works for everyone!
Comment #6
cboyden commentedThis patch works nicely on our child distro.
Comment #7
lsolesen commentedThis seems to work on vies.dk (actually I think better than even before applying this patch.
Comment #9
dsnopekAwesome! Thanks, @cboyden and @lsolesen. :-)
Committed!