benjaminkott / benjaminkott/bootstrap_package

Full width image carousel

Open
#858 0 comments 4 reactions 0 assignees View on GitHub
feature
Dominant language
PHP
Stars
355
Forks
212
Avg merge
1h 4m
Merged PRs (30d)
31

Description

# Feature Request

## Is your feature request related to a problem? Please describe

One of the most common uses of the Carousel-CE is a picture carousel across the full width of the screen. At the moment, as far as I know, this is only possible by setting the images as background_image and the carousel to "no border". But this has three main disadvantages:

- background images are not indexed by Google.
- background images do not have an alt-tag, which could be used for SEO purposes
- the background images are automatically cropped to the size of the carousel. It is hardly possible for the editor to control where or how the image is cropped.

## Describe the solution you'd like

It should be possible to set images in the Carousel with a 100% width (instead of a fixed pixel width). The aspect ratio of the image should be maintained so that you have control over a possibly necessary crop in the image editor.
For example, the bootstrap carousel does it, with the class="d-block w-100".

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the Carousel-CE implementation and its current image rendering, then compare its behavior with the Bootstrap carousel example using d-block w-100. The change is complete when carousel images can use 100% width, preserve their aspect ratio, and retain usable alt text without relying on background images.

Written by the indexing model from the issue text.

Assessment

Tech stack
bootstrap, php
Domain
frontend, web-dev
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.