Redesign package page, add gallery

This commit is contained in:
rubenwardy 2022-02-09 19:10:28 +00:00
parent 2a82c08d8b
commit 2866589109
3 changed files with 167 additions and 182 deletions

@ -0,0 +1,17 @@
// @author rubenwardy
// @license magnet:?xt=urn:btih:1f739d935676111cfff4b4693e3816e664797050&dn=gpl-3.0.txt GPL-v3-or-Later
window.addEventListener("load", event => {
document.querySelectorAll(".gallery").forEach(gallery => {
const primary = gallery.querySelector(".primary-image img");
const images = gallery.querySelectorAll("a[data-image]");
images.forEach(image => {
const imageFullUrl = image.getAttribute("data-image");
image.removeAttribute("href");
image.addEventListener("click", event => {
primary.src = imageFullUrl;
})
});
});
});

@ -1,35 +1,21 @@
.gallery { .gallery-thumbnails {
list-style: none; list-style: none;
padding: 0; margin: 0;
margin: 0 0 2em; padding: 0.5rem 0;
overflow: auto hidden; overflow: auto hidden;
display: flex;
flex-direction: row;
align-items: center;
gap: 0.5rem;
li, li a { li {
display: block;
list-style: none; list-style: none;
margin: 0; margin: 0;
padding: 0; padding: 0;
}
li {
display: inline-block;
vertical-align: middle;
margin: 5px;
padding: 0;
a {
display: block;
text-decoration: none;
&:hover {
text-decoration: none;
}
}
}
.gallery-image {
position: relative; position: relative;
&:hover img { &:hover img, .active img {
filter: brightness(1.1); filter: brightness(1.1);
} }
} }

@ -11,14 +11,15 @@
<meta name="og:title" content="{{ package.title }}"/> <meta name="og:title" content="{{ package.title }}"/>
<meta name="og:description" content="{{ package.short_desc }}"/> <meta name="og:description" content="{{ package.short_desc }}"/>
<meta name="description" content="{{ package.short_desc }}"/> <meta name="description" content="{{ package.short_desc }}"/>
<meta name="og:url" content="{{ package.getURL("packages.view", absolute=True) }}"/> <meta name="og:url" content="{{ package.getURL('packages.view', absolute=True) }}"/>
{% if package.getMainScreenshotURL() %} {% if package.getMainScreenshotURL() %}
<meta name="og:image" content="{{ package.getMainScreenshotURL(absolute=True) }}"/> <meta name="og:image" content="{{ package.getMainScreenshotURL(absolute=True) }}"/>
{% endif %} {% endif %}
{% endblock %} {% endblock %}
{% block scriptextra %} {% block scriptextra %}
<script src="/static/video_embed.js"></script> <script async src="/static/video_embed.js"></script>
<script async src="/static/gallery.js"></script>
{% endblock %} {% endblock %}
{% macro render_license(license) %} {% macro render_license(license) %}
@ -31,7 +32,7 @@
{% block download_btn %} {% block download_btn %}
{% if release %} {% if release %}
<a class="btn btn-block btn-download" rel="nofollow" download="{{ release.getDownloadFileName() }}" <a class="btn btn-block btn-lg btn-download" rel="nofollow" download="{{ release.getDownloadFileName() }}"
href="{{ package.getURL('packages.download') }}"> href="{{ package.getURL('packages.download') }}">
<div> <div>
{{ _("Download") }} {{ _("Download") }}
@ -60,7 +61,7 @@
{{ 0 / 0 }} {{ 0 / 0 }}
{% endif %} {% endif %}
<p class="text-center mt-1 mb-4"> <p class="text-center mt-1 mb-0">
<a href="{{ installing_url }}"> <a href="{{ installing_url }}">
<small> <small>
<i class="fas fa-question-circle mr-1"></i> <i class="fas fa-question-circle mr-1"></i>
@ -83,130 +84,41 @@
{% elif not package.media_license.is_foss %} {% elif not package.media_license.is_foss %}
{% set package_warning=_("Non-free media") %} {% set package_warning=_("Non-free media") %}
{% endif %} {% endif %}
<style>
.bg-banner {
background-size: cover;
background-repeat: no-repeat;
background-position: center;
position: absolute;
top: 0;
left: 0;
right: 0;
height: 65vh;
z-index: -1;
}
</style>
<main> <main>
{% set cover_image = package.cover_image.url or package.getMainScreenshotURL() %} {% set cover_image = package.cover_image.url or package.getMainScreenshotURL() %}
<header class="jumbotron pb-3" {% if cover_image %}
style="background: linear-gradient(rgba(0, 0, 0, 0.4), rgba(0, 0, 0, 0.7)), url('{{ cover_image }}'); <div style="position:relative;">
background-size: cover; <div class="bg-banner" style="background: linear-gradient(rgba(34, 34, 34, 0.7), rgba(34, 34, 34, 1)), url('{{ cover_image }}');
background-repeat: no-repeat; background-size: cover;
background-position: center;"> background-repeat: no-repeat;
<div class="container"> background-position: center;"></div>
<div class="btn-group float-right mb-4"> </div>
{% if package.checkPerm(current_user, "EDIT_PACKAGE") %} {% endif %}
<a class="btn btn-primary" href="{{ package.getURL('packages.create_edit') }}"> <header class="container pt-3 mb-4">
<i class="fas fa-pen mr-1"></i> <div class="row align-items-center">
{{ _("Edit") }} <h1 class="col my-0 display-4">
</a>
{% endif %}
{% if package.checkPerm(current_user, "MAKE_RELEASE") %}
<a class="btn btn-primary" href="{{ package.getURL('packages.create_release') }}">
<i class="fas fa-plus mr-1"></i>
{{ _("Release") }}
</a>
{% endif %}
{% if package.checkPerm(current_user, "DELETE_PACKAGE") or package.checkPerm(current_user, "UNAPPROVE_PACKAGE") %}
<a class="btn btn-danger" href="{{ package.getURL('packages.remove') }}">
<i class="fas fa-trash mr-1"></i>
{{ _("Remove") }}
</a>
{% endif %}
</div>
<h1 class="display-3">
{{ package.title }} {{ package.title }}
</h1> </h1>
<p class="lead"> <div class="col-md-3 text-right">
{{ package.short_desc }} {% if package.type == package.type.GAME %}
</p> <a href="{{ package.getURL('packages.game_hub') }}" class="btn btn-lg btn-block btn-primary">
{{ _("View content for game") }}
<p>
{% if package.dev_state.name == "LOOKING_FOR_MAINTAINER" or package.dev_state.name == "DEPRECATED" %}
<span class="badge badge-warning" title="{{ package.dev_state.get_desc() }}">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ package.dev_state.value }}
</span>
{% endif %}
{% if package_warning %}
<a class="badge badge-danger" href="/help/non_free/">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ package_warning }}
</a> </a>
{% endif %} {% endif %}
{% for warning in package.content_warnings %}
<a class="badge badge-warning" rel="nofollow" href="/help/content_flags/"
title="{{ warning.description }}">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ warning.title }}
</a>
{% endfor %}
{% if package.dev_state.name == "WIP" %}
<span class="badge badge-info" title="{{ package.dev_state.get_desc() }}">
<i class="fas fa-tools" style="margin-right: 0.3em;"></i>
{{ _("Work in Progress") }}
</span>
{% endif %}
{% for t in package.tags %}
<a class="badge badge-primary" rel="nofollow"
title="{{ t.description or '' }}"
href="{{ url_for('packages.list_all', tag=t.name) }}">
{{ t.title }}
</a>
{% endfor %}
</p>
<div class="info-row row" style="margin-top: 2rem;">
<div class="btn-group-horizontal col">
<a class="btn" href="{{ url_for('users.profile', username=package.author.username) }}" title="{{ _("Author") }}">
<img src="{{ package.author.getProfilePicURL() }}" style="max-height: 1em; filter: none">
<span class="count">
{{ package.author.display_name }}
</span>
</a>
{% if release %}
<a class="btn" rel="nofollow" href="{{ package.getURL("packages.download") }}" title="{{ _("Downloads") }}"
download="{{ release.getDownloadFileName() }}">
<i class="fas fa-download"></i>
<span class="count">{{ package.downloads }}</span>
</a>
{% endif %}
<a class="btn" href="{{ url_for('threads.list_all', pid=package.id) }}" title="{{ _("Threads") }}">
<i class="fas fa-comment-alt"></i>
<span class="count">{{ threads | length }}</span>
</a>
<a class="btn" href="#reviews" title="{{ _("Reviews") }}">
<i class="fas fa-star-half-alt"></i>
<span class="count">
+{{ package.reviews | selectattr("recommends") | list | length }}
/
-{{ package.reviews | rejectattr("recommends") | list | length }}
</span>
</a>
{% if package.website %}
<a class="btn" href="{{ package.website }}">
<i class="fas fa-globe-europe"></i>
<span class="count">{{ _("Website") }}</span>
</a>
{% endif %}
{% if package.repo %}
<a class="btn" href="{{ package.repo }}">
<i class="fas fa-code"></i>
<span class="count">{{ _("Source") }}</span>
</a>
{% endif %}
{% if package.forums %}
<a class="btn" href="https://forum.minetest.net/viewtopic.php?t={{ package.forums }}">
<i class="fas fa-comments"></i>
<span class="count">{{ _("Forums") }}</span>
</a>
{% endif %}
{% if package.issueTracker %}
<a class="btn" href="{{ package.issueTracker }}">
<i class="fas fa-bug"></i>
<span class="count">{{ _("Issue Tracker") }}</span>
</a>
{% endif %}
</div>
</div> </div>
</div> </div>
</header> </header>
@ -232,24 +144,37 @@
</section> </section>
{% endif %} {% endif %}
<div class="container d-block d-md-none"> <section class="container mb-4">
{{ self.download_btn() }}
</div>
<section class="container mt-4">
<div class="row"> <div class="row">
<div class="col-md-9" style="padding-right: 45px;"> {% set screenshots = package.screenshots.all() %}
{% set screenshots = package.screenshots.all() %} {% if screenshots or package.checkPerm(current_user, "ADD_SCREENSHOTS") or package.video_url %}
<div id="packageGallery" class="col-md-9 carousel slide my-0" data-ride="carousel" data-interval="7500">
{% if package.checkPerm(current_user, "ADD_SCREENSHOTS") %} <div class="carousel-inner">
<a href="{{ package.getURL('packages.screenshots') }}" class="btn btn-primary float-right"> {% for ss in screenshots %}
<i class="fas fa-images mr-1"></i> <div class="carousel-item {% if loop.index == 1 %}active{% endif %}">
{{ _("Edit") }} <a href="{{ ss.url }}">
</a> <div class="embed-responsive embed-responsive-16by9">
{% endif %} <img class="embed-responsive-item" src="{{ ss.url }}"
alt="{{ ss.title }}">
{% if screenshots or package.checkPerm(current_user, "ADD_SCREENSHOTS") or package.video_url %} </div>
<ul class="gallery"> <div class="carousel-caption text-shadow">
<h3 class="my-0">
{{ ss.title }}
</h3>
</div>
</a>
</div>
{% endfor %}
<a class="carousel-control-prev" href="#packageGallery" role="button" data-slide="prev">
<span class="carousel-control-prev-icon" aria-hidden="true"></span>
<span class="sr-only">{{ _("Previous") }}</span>
</a>
<a class="carousel-control-next" href="#packageGallery" role="button" data-slide="next">
<span class="carousel-control-next-icon" aria-hidden="true"></span>
<span class="sr-only">{{ _("Next") }}</span>
</a>
</div>
<ol class="gallery-thumbnails">
{% if package.video_url %} {% if package.video_url %}
<li> <li>
<a href="{{ package.video_url }}" class="video-embed"> <a href="{{ package.video_url }}" class="video-embed">
@ -263,16 +188,12 @@
{% if screenshots or package.checkPerm(current_user, "ADD_SCREENSHOTS") %} {% if screenshots or package.checkPerm(current_user, "ADD_SCREENSHOTS") %}
{% for ss in screenshots %} {% for ss in screenshots %}
{% if ss.approved or package.checkPerm(current_user, "ADD_SCREENSHOTS") %} <li data-target="#packageGallery" data-slide-to="{{ loop.index - 1 }}" {% if loop.index == 1 %}class="active"{% endif %}>
<li> <img src="{{ ss.getThumbnailURL() }}" alt="{{ ss.title }}" />
<a href="{{ ss.url }}" class="gallery-image"> {% if not ss.approved %}
<img src="{{ ss.getThumbnailURL() }}" alt="{{ ss.title }}" /> <span class="badge bg-dark badge-tr">{{ _("Awaiting review") }}</span>
{% if not ss.approved %} {% endif %}
<span class="badge bg-dark badge-tr">{{ _("Awaiting review") }}</span> </li>
{% endif %}
</a>
</li>
{% endif %}
{% else %} {% else %}
<li> <li>
<a href="{{ package.getURL('packages.create_screenshot') }}"> <a href="{{ package.getURL('packages.create_screenshot') }}">
@ -281,9 +202,76 @@
</li> </li>
{% endfor %} {% endfor %}
{% endif %} {% endif %}
</ul> </ol>
{% endif %} </div>
<div class="col-md-3">
{% else %}
<div class="col">
{% endif %}
<div class="btn-group btn-group-sm mb-3">
{% if package.checkPerm(current_user, "EDIT_PACKAGE") %}
<a class="btn btn-primary" href="{{ package.getURL('packages.create_edit') }}">
<i class="fas fa-pen mr-1"></i>
{{ _("Edit") }}
</a>
{% endif %}
{% if package.checkPerm(current_user, "MAKE_RELEASE") %}
<a class="btn btn-primary" href="{{ package.getURL('packages.create_release') }}">
<i class="fas fa-plus mr-1"></i>
{{ _("Release") }}
</a>
{% endif %}
{% if package.checkPerm(current_user, "DELETE_PACKAGE") or package.checkPerm(current_user, "UNAPPROVE_PACKAGE") %}
<a class="btn btn-danger" href="{{ package.getURL('packages.remove') }}">
<i class="fas fa-trash mr-1"></i>
{{ _("Remove") }}
</a>
{% endif %}
</div>
<p class="lead">
{{ package.short_desc }}
</p>
<p>
{% if package.dev_state.name == "LOOKING_FOR_MAINTAINER" or package.dev_state.name == "DEPRECATED" %}
<span class="badge badge-warning" title="{{ package.dev_state.get_desc() }}">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ package.dev_state.value }}
</span>
{% endif %}
{% if package_warning %}
<a class="badge badge-danger" href="/help/non_free/">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ package_warning }}
</a>
{% endif %}
{% for warning in package.content_warnings %}
<a class="badge badge-warning" rel="nofollow" href="/help/content_flags/"
title="{{ warning.description }}">
<i class="fas fa-exclamation-circle" style="margin-right: 0.3em;"></i>
{{ warning.title }}
</a>
{% endfor %}
{% if package.dev_state.name == "WIP" %}
<span class="badge badge-info" title="{{ package.dev_state.get_desc() }}">
<i class="fas fa-tools" style="margin-right: 0.3em;"></i>
{{ _("Work in Progress") }}
</span>
{% endif %}
{% for t in package.tags %}
<a class="badge badge-primary" rel="nofollow"
title="{{ t.description or '' }}"
href="{{ url_for('packages.list_all', tag=t.name) }}">
{{ t.title }}
</a>
{% endfor %}
</p>
</div>
</div>
</section>
<section class="container">
<div class="row">
<div class="col-md-9" style="padding-right: 45px;">
{% if package.desc %} {% if package.desc %}
<article class="markdown panel mb-5"> <article class="markdown panel mb-5">
{{ package.desc | markdown }} {{ package.desc | markdown }}
@ -296,7 +284,7 @@
{% if current_user.is_authenticated %} {% if current_user.is_authenticated %}
{% if has_review %} {% if has_review %}
<p> <p>
<a class="btn btn-primary" href="{{ package.getURL("packages.review") }}"> <a class="btn btn-primary" href="{{ package.getURL('packages.review') }}">
{{ _("Edit Review") }} {{ _("Edit Review") }}
</a> </a>
</p> </p>
@ -332,7 +320,7 @@
</div> </div>
<aside class="col-md-3 info-sidebar"> <aside class="col-md-3 info-sidebar">
<div class="d-none d-md-block"> <div class="mb-4">
{{ self.download_btn() }} {{ self.download_btn() }}
</div> </div>
@ -378,12 +366,6 @@
</div> </div>
{% endif %} {% endif %}
{% if package.type == package.type.GAME %}
<a href="{{ package.getURL('packages.game_hub') }}" class="btn btn-lg btn-block mb-4 btn-primary">
{{ _("View content for game") }}
</a>
{% endif %}
{% if package.type != package.type.TXP %} {% if package.type != package.type.TXP %}
<h3>{{ _("Dependencies") }}</h3> <h3>{{ _("Dependencies") }}</h3>
<dl> <dl>