Icone des Itruc #2745

Closed
opened 11 years ago by miros · 15 comments
miros commented 11 years ago

Visiblement les ITruc cherchent une icone (en fait deux) ... visible dans l'errorlog apache

[Fri Jun 01 15:54:46 2012] [error] [client 82.233.183.57] File does not exist: public_html/apple-touch-icon.png
[Fri Jun 01 15:54:46 2012] [error] [client 82.233.183.57] File does not exist: public_html/apple-touch-icon-precomposed.png

faut il en fournir une par défaut, la calculer ?

Visiblement les ITruc cherchent une icone (en fait deux) ... visible dans l'errorlog apache [Fri Jun 01 15:54:46 2012] [error] [client 82.233.183.57] File does not exist: public_html/apple-touch-icon.png [Fri Jun 01 15:54:46 2012] [error] [client 82.233.183.57] File does not exist: public_html/apple-touch-icon-precomposed.png faut il en fournir une par défaut, la calculer ?
Owner

Fournir la même fonctionnalité que pour les favicon.ico non?

Je ne connais pas les particularités par contre de ces icones...

Fournir la même fonctionnalité que pour les favicon.ico non? Je ne connais pas les particularités par contre de ces icones...
Owner

http://www.alsacreations.com/astuce/lire/1080-apple-touch-icon-iphone-ipad-ipod-icone.html

114x114 px en png

apple-touch-icon.png devrait suffir...

Donc une règle dans le htaccess :

RewriteRule ^apple-touch-icon[.]png$ spip.php?page=apple-touch-icon.png [QSA,L]

et une page adéquate

http://www.alsacreations.com/astuce/lire/1080-apple-touch-icon-iphone-ipad-ipod-icone.html 114x114 px en png apple-touch-icon.png devrait suffir... Donc une règle dans le htaccess : RewriteRule ^apple-touch-icon[.]png$ spip.php?page=apple-touch-icon.png [QSA,L] et une page adéquate
Owner

En fait si on veut fournir la bonne taille il faut 4 icones (variantes iphone/ipad, pas retina/retina)
apple-touch-icon-57x57.png
apple-touch-icon-72x72.png
apple-touch-icon-114x114.png
apple-touch-icon-144x144.png

en attendant une prochaine variante.

On doit pouvoir tout traiter en une regexp+1skel qui ajuste aux bonne dimensions ?

En fait si on veut fournir la bonne taille il faut 4 icones (variantes iphone/ipad, pas retina/retina) apple-touch-icon-57x57.png apple-touch-icon-72x72.png apple-touch-icon-114x114.png apple-touch-icon-144x144.png en attendant une prochaine variante. On doit pouvoir tout traiter en une regexp+1skel qui ajuste aux bonne dimensions ?
Owner

C'est un comportement spécifique d'une marque en particulier, on ne devrait pas encourager ce genre d'"initiatives" dans le core. La dist actuelle a déjà des trucs spécifiques Apple, il faudrait plutôt penser à les enlever qu'en ajouter d'autres.

Par ailleurs, il est très facile pour quelqu'un qui voudrait gérer ces icônes d'en faire un plugin. À l'heure actuelle, même sous forme de plugin personne ne l'a fait, ce qui veut dire que ça n'intéresse pas beaucoup de monde.

Laissons un plugin apparaître dans un premier temps.

C'est un comportement spécifique d'une marque en particulier, on ne devrait pas encourager ce genre d'"initiatives" dans le core. La dist actuelle a déjà des trucs spécifiques Apple, il faudrait plutôt penser à les enlever qu'en ajouter d'autres. Par ailleurs, il est très facile pour quelqu'un qui voudrait gérer ces icônes d'en faire un plugin. À l'heure actuelle, même sous forme de plugin personne ne l'a fait, ce qui veut dire que ça n'intéresse pas beaucoup de monde. Laissons un plugin apparaître dans un premier temps.
Owner

Par ailleurs, il est très facile pour quelqu'un qui voudrait gérer ces icônes d'en faire un plugin. À l'heure actuelle, même sous forme de plugin personne ne l'a fait, ce qui veut dire que ça n'intéresse pas beaucoup de monde.

Laissons un plugin apparaître dans un premier temps.

Non pas tellement car tu ne peux jouer sur le htaccess par exemple depuis un plugin donc ne pas utiliser la même solution que pour la favicon

<quote>Par ailleurs, il est très facile pour quelqu'un qui voudrait gérer ces icônes d'en faire un plugin. À l'heure actuelle, même sous forme de plugin personne ne l'a fait, ce qui veut dire que ça n'intéresse pas beaucoup de monde. Laissons un plugin apparaître dans un premier temps.</quote> Non pas tellement car tu ne peux jouer sur le htaccess par exemple depuis un plugin donc ne pas utiliser la même solution que pour la favicon
Owner

intégré par r70268
Version cible mise à 3.0

intégré par r70268 **Version cible mise à 3.0**
Owner

Statut changé à Fermé

**Statut changé à Fermé**
b_b commented 9 years ago
Owner

Hop, je viens de répondre à une question à ce sujet sur IRC et je vois que dans le log de commit Cédric indique :

Regle htacces a ajouter à la main, à moins qu'on ne la mette par défaut ?

On a toujours pas réglé cette question et le ticket est fermé. Alors, on ajoute cette règle au htaccess de base ou pas ? (sans quoi le squelette est inactif par défaut)

Hop, je viens de répondre à une question à ce sujet sur IRC et je vois que dans le log de commit Cédric indique : > Regle htacces a ajouter à la main, à moins qu'on ne la mette par défaut ? On a toujours pas réglé cette question et le ticket est fermé. Alors, on ajoute cette règle au htaccess de base ou pas ? (sans quoi le squelette est inactif par défaut)
b_b commented 9 years ago
Owner

Pour info, la règle à ajouter au htaccess :

RewriteRule ^apple-touch-icon(-(\d+x\d+))?(-precomposed)?[.]png$ spip.php?page=apple-touch-icon.png&size=$2 [QSA,L]

Pour info, la règle à ajouter au htaccess : RewriteRule ^apple-touch-icon(-(\d+x\d+))?(-precomposed)?[.]png$ spip.php?page=apple-touch-icon.png&size=$2 [QSA,L]

Ben oui faut l'ajouter !
Et puis le squelette est là, dans plugin-dist, alors pourquoi en priver les utilisateurs d'itrucs ? (plus de 15% si j'en crois la presse)

Ben oui faut l'ajouter ! Et puis le squelette est là, dans plugin-dist, alors pourquoi en priver les utilisateurs d'itrucs ? (plus de 15% si j'en crois la presse)
b_b commented 7 years ago
Owner

Comme rasta le proposait sur seenthis (http://seenthis.net/messages/418114#message421236), je pense qu'on peut ajouter la règle en question au htaccess de SPIP, commentée par défaut + un ty texte explicatif.

Comme rasta le proposait sur seenthis (http://seenthis.net/messages/418114#message421236), je pense qu'on peut ajouter la règle en question au htaccess de SPIP, commentée par défaut + un ty texte explicatif.
b_b commented 7 years ago
Owner

Pour ne pas oublier, je repasse le ticket en cours, et je le bascule sur la 3.1 (on pourra faire le report en 3.0 aussi amha).
Version cible mise à 3.1
Statut changé à En cours

Pour ne pas oublier, je repasse le ticket en cours, et je le bascule sur la 3.1 (on pourra faire le report en 3.0 aussi amha). **Version cible mise à 3.1** **Statut changé à En cours**
Owner

Bonjour

Pourquoi s'embeter à commenter le truc ? Autant mettre le truc par défaut. Si je ne dis pas de conneries on a 2 cas de figure :

  • les personnes qui ne sont pas sous apple, l'activation ne le pénalisent pas
  • les personnes sous apple, l'activation leur font plaisir

Donc selon la regle du "on ne prive personne tout en offrant un plus aux autres" autant mettre la règle par défaut. De plus ça simplifiera la vie des dev qui n'auront pas expliquer pourquoi c'est un regle à activer à la main alors que c'est natif, ... et avoir un ticket long comme l'amazone.

Bonjour Pourquoi s'embeter à commenter le truc ? Autant mettre le truc par défaut. Si je ne dis pas de conneries on a 2 cas de figure : * les personnes qui ne sont pas sous apple, l'activation ne le pénalisent pas * les personnes sous apple, l'activation leur font plaisir Donc selon la regle du "on ne prive personne tout en offrant un plus aux autres" autant mettre la règle par défaut. De plus ça simplifiera la vie des dev qui n'auront pas expliquer pourquoi c'est un regle à activer à la main alors que c'est natif, ... et avoir un ticket long comme l'amazone.
b_b commented 7 years ago
Owner

On commente le truc car le modèle ne fonctionne pas tant qu'aucune lib de traitement des images n'est configurée ;)

On commente le truc car le modèle ne fonctionne pas tant qu'aucune lib de traitement des images n'est configurée ;)
Poster

Appliqué par commit r22482.
Statut changé à Fermé

Appliqué par commit r22482. **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
7 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.