From 6aa638adc419fb541c4e259ae30a49cb03aad37a Mon Sep 17 00:00:00 2001
From: Fabian Helfer <fhelfer@databay.de>
Date: Fri, 30 Jan 2026 15:39:17 +0100
Subject: [PATCH 1/2] =?UTF-8?q?[Fix]=200047065=20UI:=20Lightbox=20carousel?=
 =?UTF-8?q?=20nav=20=E2=80=93=20use=20data-target=20and=20type=3D"button",?=
 =?UTF-8?q?=20not=20href?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html    | 4 ++--
 components/ILIAS/UI/tests/Component/Modal/LightboxTest.php    | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html b/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
index 642332a907f1..99f500f00ea0 100755
--- a/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
+++ b/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
@@ -31,11 +31,11 @@ <h1 class="modal-title">{TITLE}</h1>
 					</div>
 
 					<!-- BEGIN controls -->
-					<button class="left carousel-control btn-link" href="#{ID_CAROUSEL3}" role="button" type="button" data-slide="prev">
+					<button class="left carousel-control btn-link" data-target="#{ID_CAROUSEL3}" type="button" type="button" data-slide="prev">
 						<span class="glyphicon glyphicon-chevron-left" aria-hidden="true"></span>
 						<span class="sr-only">Previous</span>
 					</button>
-					<button class="right carousel-control btn-link" href="#{ID_CAROUSEL3}" role="button" type="button" data-slide="next">
+					<button class="right carousel-control btn-link" data-target="#{ID_CAROUSEL3}" type="button" type="button" data-slide="next">
 						<span class="glyphicon glyphicon-chevron-right" aria-hidden="true"></span>
 						<span class="sr-only">Next</span>
 					</button>
diff --git a/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php b/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
index e587b2760eee..3e976fb95d3a 100755
--- a/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
+++ b/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
@@ -183,11 +183,11 @@ protected static function getExpectedMixedPagesHTML(): string
 						</div>
 					</div>
 
-					<button class="left carousel-control btn-link" href="#id_1_carousel" role="button" type="button" data-slide="prev">
+					<button class="left carousel-control btn-link" data-target="#id_1_carousel" type="button" type="button" data-slide="prev">
 						<span class="glyphicon glyphicon-chevron-left" aria-hidden="true"></span>
 						<span class="sr-only">Previous</span>
 					</button>
-					<button class="right carousel-control btn-link" href="#id_1_carousel" role="button" type="button" data-slide="next">
+					<button class="right carousel-control btn-link" data-target="#id_1_carousel" type="button" type="button" data-slide="next">
 						<span class="glyphicon glyphicon-chevron-right" aria-hidden="true"></span>
 						<span class="sr-only">Next</span>
 					</button>

From 79d5a073a9ec6fc51848d5f077a5e8a74694c93c Mon Sep 17 00:00:00 2001
From: Fabian Helfer <fhelfer@databay.de>
Date: Wed, 4 Feb 2026 09:42:41 +0100
Subject: [PATCH 2/2] [Fix] 0047065 UI: Lightbox carousel remove data-target

---
 .../ILIAS/UI/src/Implementation/Component/Modal/Renderer.php  | 4 +---
 .../ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html    | 4 ++--
 components/ILIAS/UI/tests/Component/Modal/LightboxTest.php    | 4 ++--
 3 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/components/ILIAS/UI/src/Implementation/Component/Modal/Renderer.php b/components/ILIAS/UI/src/Implementation/Component/Modal/Renderer.php
index 3f987d347bea..e74e38553795 100755
--- a/components/ILIAS/UI/src/Implementation/Component/Modal/Renderer.php
+++ b/components/ILIAS/UI/src/Implementation/Component/Modal/Renderer.php
@@ -262,11 +262,9 @@ protected function renderLightbox(Component\Modal\Lightbox $modal, RendererInter
             $first = false;
         }
         if (count($pages) > 1) {
-            $tpl->setCurrentBlock('controls');
-            $tpl->setVariable('ID_CAROUSEL3', $id_carousel);
+            $tpl->touchBlock('controls');
             $tpl->parseCurrentBlock();
         }
-        $tpl->setVariable('ID_CAROUSEL4', $id_carousel);
         return $tpl->get();
     }
 
diff --git a/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html b/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
index 99f500f00ea0..5a0081dd84b4 100755
--- a/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
+++ b/components/ILIAS/UI/src/templates/default/Modal/tpl.lightbox.html
@@ -31,11 +31,11 @@ <h1 class="modal-title">{TITLE}</h1>
 					</div>
 
 					<!-- BEGIN controls -->
-					<button class="left carousel-control btn-link" data-target="#{ID_CAROUSEL3}" type="button" type="button" data-slide="prev">
+					<button class="left carousel-control btn-link" type="button" data-slide="prev">
 						<span class="glyphicon glyphicon-chevron-left" aria-hidden="true"></span>
 						<span class="sr-only">Previous</span>
 					</button>
-					<button class="right carousel-control btn-link" data-target="#{ID_CAROUSEL3}" type="button" type="button" data-slide="next">
+					<button class="right carousel-control btn-link" type="button" data-slide="next">
 						<span class="glyphicon glyphicon-chevron-right" aria-hidden="true"></span>
 						<span class="sr-only">Next</span>
 					</button>
diff --git a/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php b/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
index 3e976fb95d3a..06fc05d966b9 100755
--- a/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
+++ b/components/ILIAS/UI/tests/Component/Modal/LightboxTest.php
@@ -183,11 +183,11 @@ protected static function getExpectedMixedPagesHTML(): string
 						</div>
 					</div>
 
-					<button class="left carousel-control btn-link" data-target="#id_1_carousel" type="button" type="button" data-slide="prev">
+					<button class="left carousel-control btn-link" type="button" data-slide="prev">
 						<span class="glyphicon glyphicon-chevron-left" aria-hidden="true"></span>
 						<span class="sr-only">Previous</span>
 					</button>
-					<button class="right carousel-control btn-link" data-target="#id_1_carousel" type="button" type="button" data-slide="next">
+					<button class="right carousel-control btn-link" type="button" data-slide="next">
 						<span class="glyphicon glyphicon-chevron-right" aria-hidden="true"></span>
 						<span class="sr-only">Next</span>
 					</button>