[Beta] LMDI Multilinks 2.0.0

Pour les extensions destinées à phpBB 3.1.x

Modérateur: Equipe

Règles du forum
A lire impérativement : Règlement de phpBB-fr.com

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 17 Fév 2017 à 21:04

Ah...
Quand as-tu téléchargé cette version ?
Parce que je n'ai pas arrêté de publier des modifications.
Je vérifie de mon côté.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar Mazeltof » 17 Fév 2017 à 21:09

j'ai téléchargé quelques minutes avant mon précédent message :wink:
Image Image Image
Avatar de l’utilisateur
Mazeltof
Resp. des graphistes
Resp. des graphistes
 
Messages: 2154
Enregistré le: 09 Mar 2009 à 22:02
Localisation: Arras

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 17 Fév 2017 à 21:16

Bon, tout cela n'est pas aussi propre et net que je le voudrais.
J'y reviens demain (à cette heure-ci, mon neurone est fatigué).
Merci pour le signalement.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar Mazeltof » 17 Fév 2017 à 21:23

service :wink:
(bon repos à ton neurone) :mrgreen:
Image Image Image
Avatar de l’utilisateur
Mazeltof
Resp. des graphistes
Resp. des graphistes
 
Messages: 2154
Enregistré le: 09 Mar 2009 à 22:02
Localisation: Arras

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 18 Fév 2017 à 09:50

Voilà, j'ai téléchargé une version à jour.
J'ai un peu testé toutes les fonctions, donc cela devrait être bon.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar Mazeltof » 18 Fév 2017 à 16:47

pour cette version,

  • pour le choix d'une icône FontAwesome, je trouverai plus pratique d'indiquer l'adresse http://fontawesome.io/icons/ plutôt que le fichier assets/css/font-awesome.min.css. qui lui ne propose pas d'aperçu.
  • pour le choix d'une icône FontAwesome, bonne idée que d'indiquer : Par défaut, fa-external-link. j'aurai été tenté de mettre l'unicode f08e
  • pour le choix d'icône graphique, si on utilise une icône plus grande que 20px de coté, l'image n'est pas réduite comme indiqué.

:wink:
Image Image Image
Avatar de l’utilisateur
Mazeltof
Resp. des graphistes
Resp. des graphistes
 
Messages: 2154
Enregistré le: 09 Mar 2009 à 22:02
Localisation: Arras

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 18 Fév 2017 à 17:34

Merci pour tes commentaires.

1) Bonne idée d'indiquer le site FA, mais la chose à mettre dans la case, c'est le nom de l'icône dans la feuille de style. Est-ce qu'il suffit toujours d'ajouter fa- devant le nom FA ? C'est jouable.
2) Je vais essayer de visualiser l'icône.
3) Je vérifie ce qui se passe.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar cabot » 18 Fév 2017 à 17:39

Mazeltof a écrit:pour le choix d'une icône FontAwesome, je trouverai plus pratique d'indiquer l'adresse http://fontawesome.io/icons/ plutôt que le fichier assets/css/font-awesome.min.css. qui lui ne propose pas d'aperçu.

+1 :)

Mazeltof a écrit:pour le choix d'icône graphique, si on utilise une icône plus grande que 20px de coté, l'image n'est pas réduite comme indiqué.

Pierre, tu as remis l'ancien code HTML sur les balises img, aux lignes 11 et 20 de overall_header_navigation_append.html ainsi qu'aux lignes 11 et 20 de overall_header_navigation_prepend.html.
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26717
Enregistré le: 13 Fév 2011 à 19:42
Localisation: Rarement sur la plaque

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 18 Fév 2017 à 18:23

Je viens de vérifier et ça réduit bien l'image, mais si je comprends bien, je n'ai pas mis le fichier à jour sur le dépôt de github. Je vais le faire.

J'ai corrigé le texte pour inviter à aller voir le site FA.
Et j'ai trouvé comment afficher l'icône dans du texte. Il faut retyper la définition de dl sous la forme
Code: Tout sélectionner
dl	{
	font-family: Verdana,Helvetica,Arial,sans-serif,FontAwesome;
}

que j'inclus au début du gabarit.

C'est à noter si l'on veut utiliser ce genre de police dans des textes explicatifs.

PS
C'est fait. Le dépôt est mis à jour.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar Mazeltof » 18 Fév 2017 à 19:35

  • La réduction de l'icône image est ok
  • Dans l'explicatif tu as laissé :
    Pour rechercher des exemples d’icônes utilisées dans le logiciel phpBB, consultez le fichier assets/css/font-awesome.min.css.

    à la suite de :
    Voir le site Font Awesome pour les possibilités offertes par cette police. Il suffit d’ajouter le préfixe ‘fa-’ devant le nom.

    est-ce volontaire ? c'est un peu redondant non ?
  • Bien vu l'affichage de l'icone dans l'explicatif :P
  • Je viens de voir, dans l'ACP, la légende chevauche la fenêtre de configuration. Peut-être supprimer le </br > qui renvoie à la ligne.
:wink:
Image Image Image
Avatar de l’utilisateur
Mazeltof
Resp. des graphistes
Resp. des graphistes
 
Messages: 2154
Enregistré le: 09 Mar 2009 à 22:02
Localisation: Arras

Re: LMDI Multilinks 1.1.0

Messagepar cabot » 18 Fév 2017 à 20:13

Peut-être aussi ajouter un target="_blank" sur le lien pour éviter que l'utilisateur quitte la page en cours ?

Dans language/en/acp_multilinks.php et language/fr/acp_multilinks.php

Remplacer :
Code: Tout sélectionner
<a href="http://fontawesome.io/icons/">

Par :
Code: Tout sélectionner
<a href="http://fontawesome.io/icons/" target="_blank">
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26717
Enregistré le: 13 Fév 2011 à 19:42
Localisation: Rarement sur la plaque

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 18 Fév 2017 à 21:04

Bonsoir à tous les deux.

D'accord pour le target = "_blank".

Pour les explications, je ne sais pas si l'on doit en faire trop ou pas assez.

Je n'ai pas compris le 3e point de Mazeltof.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 1.1.0

Messagepar Mazeltof » 18 Fév 2017 à 21:13

'soir,

pierredu a écrit:Pour les explications, je ne sais pas si l'on doit en faire trop ou pas assez.

je pense que pour l'utilisation de ton extension l'indication avec le lien FontAwesome est amplement suffisante. et du coup la recherche d'icônes dans assets/css/font-awesome.min.css devient, de fait, pratiquement inutile. (et ça éviterai toute modification accidentelle du fichier .css) :wink:

pierredu a écrit:Je n'ai pas compris le 3e point de Mazeltof.

en 3, je parlais de ça :
pierredu a écrit:Et j'ai trouvé comment afficher l'icône dans du texte. Il faut retyper la définition de dl sous la forme

Code: Tout sélectionner

dl {
font-family: Verdana,Helvetica,Arial,sans-serif,FontAwesome;
}

que j'inclus au début du gabarit.

en 4, par contre et je reconnais je n'ai pas été très clair, je parlais de ça :
LMDI-multilinks.jpg
Image Image Image
Avatar de l’utilisateur
Mazeltof
Resp. des graphistes
Resp. des graphistes
 
Messages: 2154
Enregistré le: 09 Mar 2009 à 22:02
Localisation: Arras

Re: LMDI Multilinks 1.1.0

Messagepar pierredu » 19 Fév 2017 à 08:44

Pour le point 4, il est nécessaire d'avoir quelque chose parce qu'on peut très bien ne plus savoir si on est en train de saisir quelque chose à gauche ou à droite de la FAQ. En supprimant le retour à la ligne, on a une ligne trop longue, donc peu lisible.

Il faut que je remanie ça. De toute manière, il faut que je change le texte suivant qu'il s'agit d'une création ou d'une édition.

PS :
C'est fait. La version sur github a été mise à jour.
Addition d'une capture d'écran dans le premier message.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: LMDI Multilinks 2.0.0

Messagepar pierredu » 14 Avr 2017 à 14:36

L'extension a été rejetée sur phpbb.com pour différents problèmes mineurs, mais surtout parce que j'utilisais deux lignes de la table config_text pour stocker les données (étant donné que l'on ne peut pas connaître la longueur des URL que les utilisateurs vont saisir). On m'a indiqué qu'il fallait mieux utiliser une table. Les avis peuvent diverger sur ce point, mais...

J'ai donc installé une nouvelle version 2.0.0 sur github (branch2). La précédente reste (branch1).

La gestion des données est naturellement complètement différente. Avec une table, la guillotine des 5 liens avant et 5 liens après n'a plus lieu d'être, même si c'est une limite pratique. Cela permet par exemple de garder des liens au chaud que l'on n'utilise qu'à certaines occasions ou événements.

Cette version n'est pas compatible avec la précédente. Je ne me suis pas lancé dans l'écriture d'un fichier de migration pour supprimer les deux rubriques de la table config_text et il faut donc déasactiver l'extension, puis supprimer les données, avant d'installer la nouvelle version. Éventuellement, il est possible de sauvegarder ces deux lignes dans phpMyAdmin ou Adminer pour s'en servir comme modèle (URL, texte d'ancrage, fichiers, etc.) dans la resaisie des liens.

Dernier point, ces tables possèdent souvent un mécanisme de montée et de descente des rubriques. Je l'avais codé dans la première version. Je ne l'ai pas encore recodé. Je me demande si cela a vraiment un sens. Merci d'avance pour vos commentaires.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 922
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

PrécédenteSuivante

Retourner vers Extensions en développement

 


  • Articles en relation
    Réponses
    Vues
    Dernier message

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 1 invité