Przeglądaj źródła

Fix slider when no article (#4375)

* Fix slider when no article
#fix https://github.com/FreshRSS/FreshRSS/issues/4373
https://github.com/FreshRSS/FreshRSS/pull/4291

* Use aside

* Forgotten

Co-authored-by: maTh <math-home@web.de>
Alexandre Alapetite 3 lat temu
rodzic
commit
0fc0060966

+ 7 - 7
app/views/configure/queries.phtml

@@ -59,16 +59,16 @@
 		</div>
 		<?php } ?>
 	</form>
+</main>
 
-	<?php $class = $this->displaySlider ? ' active' : ''; ?>
-	<a href="#" id="close-slider" class="<?= $class ?>">
-		<?= _i('close') ?>
-	</a>
-	<div id="slider" class="scrollbar-thin<?= $class ?>">
+<?php $class = $this->displaySlider ? ' active' : ''; ?>
+<a href="#" id="close-slider" class="<?= $class ?>">
+	<?= _i('close') ?>
+</a>
+<aside id="slider" class="scrollbar-thin<?= $class ?>">
 	<?php
 		if ($this->query != null) {
 			$this->renderHelper('configure/query');
 		}
 	?>
-	</div>
-</main>
+</aside>

+ 8 - 7
app/views/extension/index.phtml

@@ -77,15 +77,16 @@
 			</table>
 		</div>
 	<?php } ?>
-	<?php $class = isset($this->extension) ? ' active' : ''; ?>
-	<a href="#" id="close-slider" class="<?= $class ?>">
-		<?= _i('close') ?>
-	</a>
-	<div id="slider" class="scrollbar-thin<?= $class ?>">
+</main>
+
+<?php $class = isset($this->extension) ? ' active' : ''; ?>
+<a href="#" id="close-slider" class="<?= $class ?>">
+	<?= _i('close') ?>
+</a>
+<aside id="slider" class="scrollbar-thin<?= $class ?>">
 	<?php
 		if (isset($this->extension)) {
 			$this->renderHelper('extension/configure');
 		}
 	?>
-	</div>
-</main>
+</aside>

+ 9 - 6
app/views/index/normal.phtml

@@ -116,12 +116,8 @@ $today = @strtotime('today');
 	if ($nbEntries > 0):
 		call_user_func($this->callbackBeforePagination, $this, $nbEntries, $lastEntry);
 		$this->renderHelper('stream-footer');
-?><?php $class = $this->displaySlider ? ' active' : ''; ?>
-<a href="#" id="close-slider" class="<?= $class ?>">
-	<?= _i('close') ?>
-</a>
-<div id="slider" class="scrollbar-thin<?= $class ?>">
-</div></main><?php
+?>
+</main><?php
 	else:
 		ob_end_clean();	//Discard the articles headers, as we have no articles
 ?>
@@ -138,4 +134,11 @@ $today = @strtotime('today');
 </main>
 <?php endif; ?>
 
+<?php $class = $this->displaySlider ? ' active' : ''; ?>
+<a href="#" id="close-slider" class="<?= $class ?>">
+	<?= _i('close') ?>
+</a>
+<aside id="slider" class="scrollbar-thin<?= $class ?>">
+</aside>
+
 <?php if ($nbEntries > 0 && FreshRSS_Context::$user_conf->show_nav_buttons) $this->partial('nav_entries'); ?>

+ 7 - 7
app/views/stats/idle.phtml

@@ -48,16 +48,16 @@
 		<span class="alert-head"><?= _t('admin.stats.no_idle') ?></span>
 	</p>
 	<?php } ?>
+</main>
 
-	<?php $class = $this->displaySlider ? ' active' : ''; ?>
-	<a href="#" id="close-slider" class="<?= $class ?>">
-		<?= _i('close') ?>
-	</a>
-	<div id="slider" class="scrollbar-thin<?= $class ?>">
+<?php $class = $this->displaySlider ? ' active' : ''; ?>
+<a href="#" id="close-slider" class="<?= $class ?>">
+	<?= _i('close') ?>
+</a>
+<aside id="slider" class="scrollbar-thin<?= $class ?>">
 	<?php
 		if (isset($this->feed)) {
 			$this->renderHelper('feed/update');
 		}
 	?>
-	</div>
-</main>
+</aside>

+ 9 - 8
app/views/subscription/index.phtml

@@ -25,7 +25,7 @@
 			<a class="btn" href="<?= _url('subscription', 'index', 'error', '1') ?>"><?= _i('look') ?> <?= _t('sub.feed.show.error') ?></a>
 		</div>
 	<?php } ?>
-	
+
 	<div class="drop-section">
 		<?php
 			$signalError = false;
@@ -73,11 +73,13 @@
 			</div>
 		</div>
 	</div>
-	<?php $class = $this->displaySlider ? ' active' : ''; ?>
-	<a href="#" id="close-slider" class="<?= $class ?>">
-		<?= _i('close') ?>
-	</a>
-	<div id="slider" class="scrollbar-thin<?= $class ?>">
+</main>
+
+<?php $class = $this->displaySlider ? ' active' : ''; ?>
+<a href="#" id="close-slider" class="<?= $class ?>">
+	<?= _i('close') ?>
+</a>
+<aside id="slider" class="scrollbar-thin<?= $class ?>">
 	<?php
 		if (isset($this->feed)) {
 			$this->renderHelper('feed/update');
@@ -85,5 +87,4 @@
 			$this->renderHelper('category/update');
 		}
 	?>
-	</div>
-</main>
+</aside>