Skip to content

rendered the testimonials sections image properly as well as the first card - #551

Open
antedotee wants to merge 1 commit into
containers:mainfrom
antedotee:fix-testimonial-section
Open

rendered the testimonials sections image properly as well as the first card#551
antedotee wants to merge 1 commit into
containers:mainfrom
antedotee:fix-testimonial-section

Conversation

@antedotee

Copy link
Copy Markdown
Contributor

Before:
image

After:
image

Earlier, the leftmost card was rendering in half. Also, the image on the fourth card was not rendering. Apart from this, I have made padding between the arrows and the card consistent.

One clarification I want to ask: the post link on the second card is not available now, maybe he has deleted it. Should we now remove that link from the card or not?

Fixes #550

@antedotee

Copy link
Copy Markdown
Contributor Author

Requesting a friendly ping here since the PR has been raised since past 2 weeks @TomSweeneyRedHat @ashley-cui😅 , I hope you won't mind. I have also seen a lot of people raising PR for this, please look into it or close mine in case others have a better implementation😅.
Also, I have implemented preview link generation functionality to make the maintainer's work easier.
Here is the link in case you wanna have a look: #650

Hope you have a good day ahead 😅.

@ashley-cui

Copy link
Copy Markdown
Member

Sure, LGTM.

@ashley-cui

Copy link
Copy Markdown
Member

Apologies, could you squash you commits?

@antedotee

Copy link
Copy Markdown
Contributor Author

sure, will do 😅

@antedotee
antedotee force-pushed the fix-testimonial-section branch from e3a205c to 87e8b43 Compare August 24, 2026 17:47
…t card

Signed-off-by: antedotee <soniyadav2051982@gmail.com>
@antedotee
antedotee force-pushed the fix-testimonial-section branch from 87e8b43 to 5057551 Compare August 24, 2026 17:52
@TomSweeneyRedHat

Copy link
Copy Markdown
Member

LGTM
@ashley-cui I'll let you push the shiny green merge button if you think it's appropriate.

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.

Testimonials sections has few bugs

3 participants