évolution du squelette
Bonjour,
Quelques évolutions pour ce squelette afin de donner la possibilité :
- d'ajouter du logo de site en page d'accueil (nouveau champ dans le formulaire de config du squelette)
- d'utiliser les champs #CHAPO
- d'utiliser le plugin Menus avec le site : un menu avec l'identifiant "principal" remplacera le menu principal
- d'utiliser le plugin Agenda (des boucles événements sur les pages articles)
- d'utiliser le plugin Rang
- de permettre un clic sur les images des élements cliquables
Et aussi :
- Micro-optimisations de la feuille de style
- Amélioration des portfolios
- Ajout du bouton accueil dans la version smartphone du menu
- On n'utilise pas #LOGO_ARTICLE_RUBRIQUE sur la page rubrique (sinon le même image est repetée trop de fois)
Rapports de requête de fusion
Activité
j'oubliais, un aperçu d'un site utilisant cette version : http://ad2f.fr (Seules les couleurs ont été modifié par rapport au plugin proposé)
Bonjour,
J'ai proposé il y a un mois et demi l'évolution de ce plugin ci-dessus. Comme personne n'a réagit, plusieurs questions :
- La méthode utilisée est-elle la bonne ?
- L'évolution proposée du plugin est-il hors sujet ?
- L'évolution proposée du plugin est-elle trop dense pour permettre aux contributeurs de ce plugin de donner rapidement leur avis ? (aurai-je dû le découper en plusieurs contributions ?)
- Lorsqu'il n'y a pas de réponse des contributeurs, dois-je publier la proposition ?
Je viens de faire quelques modifs : 1./ Une évolution permettant d'afficher automatiquement les videos sous forme de player lorsqu'elles sont associées à un article 2./ Quelques modifs concernant la boucle événement (en cas de présence du plugin agenda)
Cette évolution interresse t-elle quelqu'un•e ?
je ne sais pas. Il arrive que les mainteneurs/euses ne soit plus actif·ves mais je ne crois pas que ce soit le cas de @jeanmarie
cela étant il n'est pas abonné au suivi de ce dépôt. Sans doute un bug lors de la migration svn>git.
(cela fait à peine 1 an qu'on utilise vraiment git).
Du coup je l'ai bipée. S'il répond pas dans les prochains jours, écrit lui.
Salut,
pour la petite histoire, c'est surtout @Bennyb qui a porté le thème. Et je crois que @b_b l'utilise aussi... (allez, on ping tout le monde, youhou :) ) Pour ma part, je ne l'utilise plus et j'ai pas trop suivi depuis donc bon...
J'ai regardé rapidement, c'est super ces nouvelles évolutions !
Juste un retour/question (comme je n'ai pas testé) pour les forums : j'ai l'impression que ta modif conditionne l'affichage des commentaires existants à l'activation des forums (cf échanges par ici https://contrib.spip.net/HTML5UP-Alpha#comment499905 ) or, il faudrait les garder visible même s'il n'y a pas de
#FORMULAIRE_FORUM
.Je ne sais pas si tu as eu l'occasion de regarder cet article qui sert de "cadre" pour essayer (autant que possible) d'homogénéiser les fonctionnalités : https://contrib.spip.net/Pour-un-portage-efficace-des-themes-HTML5UP
Yop, de mon côté je surcharge pas mal le squelette en question, donc ça ne devrait pas me géner des masses dans mon utilisation. Amha, ça peut être intégré tant que ça n'impacte pas le fonctionnement des sites qui utilisent déjà le squelette, sinon il faut créer une nouvelle branche pour archiver l'actuel et passer cette PR dans le trunk/master.
PS : je doute grandement sur ces ajouts pour les vidéos 064e657e le plugin oembed fait déjà ça très bien, et je trouve que c'est trop spécifique pour être dans ce squelette.
Une petite remarque :
le linear-gradient sur le menu, bof. Le design est flat, y mettre un dégradé c'est moyen. Si c'est pour une histoire de contraste avec un fond, soit utiliser une photo adéquate, soit mettre le linear-gradient dans un perso.css par exemple. (Là sur mon site ça le fait pas du tout ce linear gradient, ça m'oblige à le surcharger pour l'enlever).
Sinon j'ai pas testé l'agenda ni les rubriques que je n'utilise pas (pour l'instant).
Donc hormis linear-gradient pour moi c'est ok.
Pour le menu, il n'y a pas moyen de faire un "si oui/sinon" ?
[(#PLUGIN{menus}|oui|?{ <INCLURE{fond=inclure/header_plugin_menus,env} /> , <INCLURE{fond=inclure/header,env} /> })]
Et pour l'inclure, appeler ça
inclure/header_perso
? (<- je chipotte hein, c'est juste pour essayer de trouver un truc qui pourrait être généralisé)Bonjour,
@fa_b : j'ai viré le linear gradient et ait laissé un style commenté pour améliorer la lecture du menu en cas de bandeau clair
@jeanmarie : c'est OK pour le oui / sinon
@b_b : les modèles contenus dans oembed ne permettent pas de présenter les vidéos avec le lecteur d'origine. l'interêt d'utiliser par exemple le lecteur youtube officiel (et non une vignette qui laisse place au lecteur en mode autoplay une fois celle-ci cliquée) est de permettre à Youtube de comptabiliser le nombre de vues (celle-ci ne sont pas prises en compte en mode autoplay). Enfin, je n'ai pas trouvé comment jongler avec les dimensions des lecteurs et faire disparaître les titres, descriptifs et credits des vidéos avec les modèles d'oembed. Voilà pourquoi j'utilise cette inclusion de lecteurs vidéos qui présente la limite de ne concerner que les 3 services les plus connus ainsi que les vidéos hébergées en local.
@yohoo : sans avoir d'avis sur le fond quand aux lecteurs officiels ou pas, il me semble que si dérogation il devait y avoir, ce ne serait pas à mettre dans un squelette, mais soit directement dans le plugin oEmbed, soit dans un plugin à part. A voir selon quelles mdoalités. Etant une quiche en vidéo, je n'en dirai pas plus.
@Yohooo Super pour le si oui/sinon.
Pour la vidéo, justement, l'intérêt de OEmbded est de faire "proxy" entre le visiteur et le service de vidéo peu regardant sur la confidentialité, comme ça, on respecte les données personnelles des visiteur·ses qui ne lancent pas la vidéo :) Et comme le dit @maieul, je ne pense que ça soit la bonne solution de le faire par défaut dans le squelette.
Décidément, le fonctionnement de Git reste pour moi un mystère sur de nombreux points.
Les 2 dernières modifications effectuées sur ce projet (visibles ici) n'apparaissent pas ici. Ni dans l'onglet révision, ni sous forme d'un petit message "Yohooo added 1 commit ..."
J'ai pourtant utilisé la même méthode pour mon pousser mon code. Visiblement, la seule différence avec mes autres push est que celui-ci comporte la suppression d'un fichier (le fameux "inclure/video.html" qui posait problème).
Quelqu'un peut-il m'aider sur la méthode ?
Si on résume les interventions:
- perso je ne l'utilise plus et j'ai fait mes retours
- @b_b surcharge tellement qu'il ne verra pas la différence
- @Jack31 devait tester et s'adapter :)
- @Bennyb n'est pas intervenu (qui ne dis mot consent)
Donc si vous avez testé, toi et @Jack31, et c'est ok, gogogo, nan ?
Idem, c'est tellement surchargé que ca devrait pas trop bouger. Par contre le framework maison de @ajlkn a changé, il est passé en flex. Une nouvelle branche en perspective, voire une nouvelle version du plugin car ce ne sont plus des, par exemple,
.u6(mobile)
mais du classique.col-6
qui remplace tout ca.Je suis en train d'essayer cette version sur un site temporaire que je monte en SPIP 4.0-beta (en php 7.3)
J'ai un souci avec cette ligne sur la page sommaire.html <INCLURE{fond=inclure/header_plugin_menus,alpha_class='alt'} />, qui génère l'affichage de ce texte en haut de la page
<?php echo recuperer_fond( 'inclure/header', array('alpha_class' => 'alt', 'lang' => 'fr'), array("compil"=>array('plugins/html5up_alpha/sommaire.html','html_99283b489a226c4a6c4cdc9d6b6cc21b','',0,$GLOBALS['spip_lang'])), _request("connect")); ?>
Sinon ça tombe bien, j'avais justement besoin d'intégrer un logo sur la page d'accueil :)
Est-ce qu'il serait possible d'en profiter pour moderniser le formulaire de config avec des div au lieu des ul/li ?
Enfin sur la validité du squelette sans doute il faudrait qu'il ne soit valide qu'à partir de SPIP 3.2 ? voire SPIP 4.0 ?
En ce qui concerne l'erreur générée par l'inclure du plugins menus il y a des squelettes qui utilisent Menus de façon conditionnelle ce serait bien de s'en inspirer. Par exemple spipr-educ avec ce code :
[(#PLUGIN{menus}|oui)<div class="navbar navbar-inverse" id="nav"><div class="menu-conteneur navbar-inner"> #INCLURE{fond=inclure/menu, env, identifiant=barrenav} </div></div>] [(#PLUGIN{menus}|non)<div class="navbar navbar-inverse" id="nav"><INCLURE{fond=noisettes/commun/nav-secteurs-spipr,env} /></div>]
N'utilisant pas menus là je n'ai pas testé jusqu'au bout...
Ensuite je note un souci chez moi sur les logos des articles en page d'accueil : les images ne sont plus centrées et ne prennent plus toute la largeur du bloc...une personnalisation malheureuse qui trainait chez moi...ah my bad, je testais "alpha", et j’ai vu que ça utilisait pas "chapo" et j’ai envoyé !11 … puis… je vois cette PR… hum
added 51 commits
-
ff8869cc...cbcc2e6c - 22 commits from branch
master
- cbcc2e6c...4ccf82bf - 19 earlier commits
- cb051fe9 - build: Nouvelles fonctionnalités : On incrémente la version
- 32e5352d - Mini-modif de style : Mise en commentaire d'un dégradé trop glossy sur les menus.
- c64df3f2 - Debug css
- 4e39b496 - fix: Présentation des vidéos associées à un article
- 2e13e366 - fix: Inclusion de vidéos moins hétérodoxes.
- 07a33f8f - Menu dans la page 404
- d0f48b28 - fix: Menus, boucle événements, et doublons de documents lorsque ceux-ci sont dans le corps de texte
- 231139c2 - fix: Correction et simplification de l’appel du menu
- 15b6220f - fix: L’accueil doit être toujours visible, sinon les boutons suivants ne sont...
- 86da9cc0 - fix: affichage d’un cadre de forum vide
Afficher/masquer la liste des validations-
ff8869cc...cbcc2e6c - 22 commits from branch
added 1 commit
- ceb10f56 - fix: doublon suite à conflit de fusion mal corrigé
Je vois quelques soucis sur les événements notamment
-
- s’il y a un seul événement, le titre indique, dans l’article "Événements Passés", même si c’est un événement dans le futur
-
- il y a un lien "Lire la suite" sur les événements qui amène à une page (/evenement.html) qui présente moins d’information que sur la liste : pas la date ou le lieu, notamment.
-
added 1 commit
- fd03e341 - fix: Si un seul événement, ne pas indiquer qu’il est passé. Vérifier sa date
added 1 commit
- 68de64b9 - fix: Passage de classe au résumé d’événement