New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improvements for the Default theme #282
Changes from 6 commits
fde940b
b4a29da
6956b84
3ae535f
25ed5df
2bf381c
2f8ea0f
7391947
66cdf21
65211ff
14c2562
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,13 @@ | ||
<nav class="navbar navbar-expand-lg navbar-dark bg-danger"> | ||
<nav class="navbar navbar-expand-lg navbar-dark bg-dark"> | ||
<div class="container"> | ||
<a class="navbar-brand" href="index.php">Admin</a> | ||
<button class="navbar-toggler" type="button" data-toggle="collapse" data-target="#navbarNav" aria-controls="navbarNav" aria-expanded="false" aria-label="<?php echo _('Toggle navigation'); ?>"> | ||
<span class="navbar-toggler-icon"></span> | ||
</button> | ||
<div class="collapse navbar-collapse" id="navbarNav"> | ||
<ul class="navbar-nav"> | ||
<li class="nav-item"> | ||
<a class="nav-link" href="index.php">Home</a> | ||
<a class="nav-link" href="../index.php">Home</a> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are we changing the path of index.php to the parent directory? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Right... Hope i made it clear :) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please leave the View Podcast button as it is, the home button is intended to access the homepage of the admin interface (with the news blog) |
||
</li> | ||
<li class="nav-item dropdown"> | ||
<a class="nav-link dropdown-toggle" href="#" id="navbarDropdown" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false"> | ||
|
@@ -59,12 +60,10 @@ | |
</li> | ||
</ul> | ||
<ul class="navbar-nav ml-auto"> | ||
<li class="nav-item"> | ||
<a class="nav-link" href="<?php echo $config['url']; ?>" target="_blank"><?php echo _('View Podcast'); ?></a> | ||
</li> | ||
<li class="nav-item"> | ||
<a class="nav-link" href="logout.php">Logout - <?php echo $_SESSION["username"]; ?></a> | ||
<li class="nav-item nav-pills"> | ||
<a class="nav-link active p-1" href="logout.php">Logout - <?php echo $_SESSION["username"]; ?></a> | ||
</li> | ||
</ul> | ||
</div> | ||
</div> | ||
</nav> |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,8 +3,9 @@ | |
|
||
<head> | ||
<title><?php echo htmlspecialchars($config["podcast_title"]); ?></title> | ||
<link rel="stylesheet" href="<?php echo htmlspecialchars($config["theme_path"]); ?>style/bootstrap.css"> | ||
<meta charset="utf-8"> | ||
<link rel="stylesheet" href="<?php echo htmlspecialchars($config["theme_path"]); ?>style/bootstrap.css"> | ||
<link rel="stylesheet" href="<?php echo htmlspecialchars($config["theme_path"]); ?>style/custom.css"> | ||
<link href="https://stackpath.bootstrapcdn.com/font-awesome/4.7.0/css/font-awesome.min.css" rel="stylesheet" integrity="sha384-wvfXpqpZZVQGK6TAh5PVlGOfQNHSoD2xbE+QkPxCAFlNEevoEH3Sl0sibVcOQVnN" crossorigin="anonymous"> <meta charset="utf-8"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This line is problematic.
Also please add a linebreak before the tag |
||
<meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
<link rel="shortcut icon" type="image/x-icon" href="favicon.ico"> | ||
</head> | ||
|
@@ -18,22 +19,25 @@ | |
<?php | ||
include "js.php"; | ||
include "jumbotron.php"; | ||
if (!isset($_GET["cat"])) { | ||
?> | ||
<ul> | ||
<?php | ||
foreach ($categories_xml as $item) { | ||
echo "<li><a href=\"categories.php?cat=" . $item->id . "\">" . $item->description . "</a></li>"; | ||
} | ||
?> | ||
</ul> | ||
<hr> | ||
<a href="categories.php?cat=all"><?php echo _('All Episodes'); ?></a> | ||
<?php | ||
} else { | ||
include 'listepisodes.php'; | ||
} | ||
?> | ||
echo '<div class="row">'; | ||
if (!isset($_GET["cat"])) { | ||
?> | ||
<div class="col"> | ||
<div class="list-group"> | ||
<a class="list-group-item list-group-item-action" href="categories.php?cat=all"><?php echo _('All Episodes'); ?></a> | ||
<?php | ||
foreach ($categories_xml as $item) { | ||
echo "<a class='list-group-item list-group-item-action' href=\"categories.php?cat=" . $item->id . "\">" . $item->description . "</a>"; | ||
} | ||
?> | ||
</div> | ||
</div> | ||
<?php | ||
} else { | ||
include 'listepisodes.php'; | ||
} | ||
echo '</div>' | ||
?> | ||
</div> | ||
</body> | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,6 +4,8 @@ | |
<head> | ||
<title><?php echo htmlspecialchars($config["podcast_title"]); ?></title> | ||
<link rel="stylesheet" href="<?php echo htmlspecialchars($config["theme_path"]); ?>style/bootstrap.css"> | ||
<link rel="stylesheet" href="<?php echo htmlspecialchars($config["theme_path"]); ?>style/custom.css"> | ||
<link href="https://stackpath.bootstrapcdn.com/font-awesome/4.7.0/css/font-awesome.min.css" rel="stylesheet" integrity="sha384-wvfXpqpZZVQGK6TAh5PVlGOfQNHSoD2xbE+QkPxCAFlNEevoEH3Sl0sibVcOQVnN" crossorigin="anonymous"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here as well |
||
<meta charset="utf-8"> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
<meta name="description" content="<?php echo htmlspecialchars($config["podcast_subtitle"]); ?>"> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not use a dark background honestly. I think the admin interface/bar should differ from the default theme