Ajout du carousel sur la page projet (issue #104) #312
Conversation
En fait c’est (5+3) commits :
|
| <!--<button class="button next">></button> --> | ||
| </div> | ||
| <div class="carousel column is-6"> | ||
| <div class="carousel-item"> |
There was a problem hiding this comment.
There was a problem hiding this comment.
bon ca me semble coolcoolcool. Ça sera coolcoolcool d'ailleurs de pouvoir réutiliser les parties concernant le carrousel pour les intégrer dans le code source d'Apiviz... Du coup on pourra importer des fichiers html tels quels depuis GitHub grâce au composant dynamicstatic, mais profiter globalement du carrousel Bulma en important les libs de bulma carrousel qq part ds spa.html ou ds le composant dynamicstatic lui-mm
|
|
||
| //@import "../node_modules/bulma-checkradio/dist/css/bulma-checkradio.sass"; | ||
| //@import "../node_modules/bulma-extensions/bulma-carousel/dist/css/bulma-carousel.sass" | ||
| @import "./git-bulma-carousel-4.0.1.sass"; // temporary fix, seems that the npm package is not up-to-date? |
There was a problem hiding this comment.
pe faire une issue à ce propos si ce n'est pas déjà fait du coup ?
| {#<!-- | ||
| <script type="text/javascript" src="{{ url_for('static', filename='bulma-carousel/dist/js/bulma-carousel.js') }}"></script> | ||
| --> #} | ||
| <script src="https://cdn.jsdelivr.net/npm/bulma-carousel@4.0.1/dist/js/bulma-carousel.min.js"></script> |
There was a problem hiding this comment.
| --> #} | ||
| <script src="https://cdn.jsdelivr.net/npm/bulma-carousel@4.0.1/dist/js/bulma-carousel.min.js"></script> | ||
| <!-- <link rel="stylesheet" href="{{ url_for('static', filename='bulma-carousel/dist/css/bulma-carousel.min.css') }}"> --> | ||
| <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bulma-carousel@4.0.1/dist/css/bulma-carousel.min.css"> |
There was a problem hiding this comment.
| @@ -36,6 +41,18 @@ | |||
|
|
|||
| <script defer src="https://use.fontawesome.com/releases/v5.0.6/js/all.js"></script> | |||
|
|
|||
| <!-- Initialize carousels --> | |||
| <script> | |||
| window.onload = function(){ | |||
|
g relu, c tt bon pr moi sauf le petit conflit à résoudre avant de pouvoir faire le merge. |
|
@JulienParis normalement c’est bon, j’ai liquidé le petit conflit avec l’outil en ligne, ça marche pas mal ! |
Ajout du carousel sur la page projet (issue #104)
Modifications :
Testé avec Firefox 65 et Chromium 71 (Ubuntu)
Ping @JulienParis pour la review