Merge pull request 'Fixed small tiny issue with the carousel having less the maximum number of visible items i.e. none in the hidden div' (#57) from older-blog-posts-fix into master
All checks were successful
🚀 Deploy website on push / 🎉 Deploy (push) Successful in 17s

Reviewed-on: #57
This commit is contained in:
Rohit Pai 2024-11-04 22:39:51 +00:00
commit 00851d37b8
3 changed files with 9 additions and 6 deletions

View File

@ -673,11 +673,6 @@ EOD;
*/ */
public function changeHTMLSrc(string $body, string $to, string $from): string public function changeHTMLSrc(string $body, string $to, string $from): string
{ {
// $body = preg_replace_callback('/>([^<]+)</', function($matches) {
// // Convert special characters to HTML entities
// return '>' . htmlentities(trim($matches[1]), ENT_QUOTES | ENT_HTML5, 'UTF-8') . '<';
// }, $body);
$htmlDoc = new DOMDocument(); $htmlDoc = new DOMDocument();
// Load the raw HTML content into DOMDocument // Load the raw HTML content into DOMDocument

File diff suppressed because one or more lines are too long

View File

@ -376,6 +376,14 @@ function carouselLoop(carouselInner, allItems)
visibleCards.push(cards[i]); visibleCards.push(cards[i]);
} }
if (allItems.children.length === 0)
{
// if there are no cards in the carousel, don't show the arrows
prev.style.visibility = 'hidden';
next.style.visibility = 'hidden';
return;
}
next.addEventListener('click', () => next.addEventListener('click', () =>
{ {
const firstCard = visibleCards.shift(); const firstCard = visibleCards.shift();