Skip to content

Feature/image carousel caption#1229

Merged
KingYes merged 12 commits intoelementor:release/1.2.0from
kobizz:feature/image-carousel-caption
Jan 26, 2017
Merged

Feature/image carousel caption#1229
KingYes merged 12 commits intoelementor:release/1.2.0from
kobizz:feature/image-carousel-caption

Conversation

@kobizz
Copy link
Copy Markdown
Contributor

@kobizz kobizz commented Jan 26, 2017

No description provided.

$slides[] = '<div class="slick-slide"><div class="slick-slide-inner">' . $image_html . '</div></div>';
$image_caption = $this->get_image_caption( $attachment );

$slides[] = '<div><figure class="slick-slide-inner">' . $image_html . '<figcaption class="elementor-image-carousel-caption">' . $image_caption . '</figcaption></figure></div>';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add slick-slide class

@drawcard
Copy link
Copy Markdown
Contributor

Nice one - thank you for incorporating this! I'm glad it's helped everyone.

When I get a free moment I may look into adding the option for absolute positioning of captions, so they can overlay the image (rather than take up space at the bottom). That is probably just a matter of writing additional CSS classes for figcaption and toggling them on or off in the editor UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants