[spip-dev] Modifications HTML cosmétiques dans 'spip_loader.php3'

Bonjour,

Voici une proposition de patch pour spip_loader.php3.
$ diff -up spip_loader-org.php3 spip_loader.php3 > spip_loader-patch

Je ne m'attends pas à ce que ce patch soit accepté en l'état. En voici les
points principaux (certains litigieux):

- Je sais que changer ce seul fichier (en plus marginal) est un peu
inutile. Je crois cependant qu'il vaut mieux commencer par de 'petites
choses' simple pour me faire la main (si ca passe, je continuerai donc).

- Le patch n'est pas complet, il ne couvre que la premiere page HTML alors
que le loader en affiche d'autres.

- Le but du patch est de faire en sorte que le HTML généré sur la page web
soit très simple et soit du HTM valide.

- Tous les tags que j'ai ajouté ou retouchés sont en minuscule (choix
personnel, puisque spip n'en a pas).

- Le rendu visuel peut etre légerement différent du rendu original.
  -> Il est possible de completer la CSS pour mieux le mimer (en
     particulier, usage de smemes fontes, etc).

- Le patch fait donc:

  * Produit une en-tete HTML 4.01 valide
  * Inclus les CSS correctement (style type="text/css">)
  * Retiré toute indication de taille et type de fonte.
  * Structuré le document par la présence d'un titre de premier niveau
    (plutot qu'un effet de fonte).
  * Remplacé la largeur fixe du tableau par une largeur dépendant de la
    taille de la page (pourrait etre plutot indexé sur la taille de la
    fonte).
  * Corrigé des erreurs html variés (tags ouverts non fermés, caracteres
    incorrects (tels que '>'), etc.

- Je comprends bien que cela peut paraitre 'inutile' (ca l'est
probablement en partie), mais vuq ue je le fais pour vous...

Voilà, c'est en attaché (3ko)... Qu'en pensez-vous ?

  Yannick

spip_loader-patch (3.03 KB)

Je sais que changer ce seul fichier (en plus marginal) est un peu
inutile. Je crois cependant qu'il vaut mieux commencer par de
'petites choses' simple pour me faire la main (si ca passe, je
continuerai donc)

C'est un très bon début, qui devrait motiver la généralisation à tout
le reste, et peut-être lancer les réflexions sur la refonte de
'ecrire/' sans texte dans les images.

Si un système à onglets vous tente, je vous propose à nouveau le code
que j'ai utilisé sur http://www.lecercle.com/actualites/ dont la
partie 'actualités' est gérée par SPIP ... :wink:

Nicolas.

C'est un très bon début, qui devrait motiver la généralisation à tout
le reste, et peut-être lancer les réflexions sur la refonte de
'ecrire/' sans texte dans les images.

Il y a peut-être d'autres priorités que la présentation ou la conformité du
code html, cf le fichier TODO :
    - les documents attachés ne sont pas terminés
    - l'aspect "liste de diffusion" mériterait des efforts importants
    - les statuts des spip_auteurs (manquent "modérateur" en remplacement
      de l'admin restreint, et "visiteur", pour un site public restreint
      et pour les futures "listes de diff")
    - etc.

Par ailleurs il serait plus que nécessaire de refondre le code autour d'API
un peu propres : un ensemble de fonctions pour traiter/mettre à jour des
articles dans la base, par exemple. Et dans le reste du code on
s'abstiendrait d'appeler mysql directement, en préférant appeler cet API. Ca
nous libérerait pour écrire ensuite des interfaces privées différentes...

Ne pas hésiter à se "faire la main" sur des trucs difficiles ! En ce qui me
concerne, vérifier et intégrer des patches qui ne servent à rien, comme
celui de Yannick pour spip_loader, ça ne m'amuse pas tellement.

Côté technique, je préférerais recevoir non pas des patches (que je ne sais
pas intégrer sous Mac) mais un URL (ftp anonyme ou web avec fichiers en
.txt) où récuperer les fichiers modifiés (oui, je sais, c'est moins bien).

Si un système à onglets vous tente, je vous propose à nouveau le code
que j'ai utilisé sur http://www.lecercle.com/actualites/ dont la
partie 'actualités' est gérée par SPIP ... :wink:

Oui, c'est assez joliment fait, et ça paraît fonctionnel sous links comme
sous lynx.

-- Fil

Il y a peut-être d'autres priorités que la présentation ou la
conformité du code html

Pour la présentation, c'est un chantier nécessaire avant toute
réflexion i18n, c'est juste pour ça que ça me semble intéressant ... :wink:

Par ailleurs il serait plus que nécessaire de refondre le code
autour d'API un peu propres : un ensemble de fonctions pour
traiter/mettre à jour des articles dans la base, par exemple. Et
dans le reste du code on s'abstiendrait d'appeler mysql directement,
en préférant appeler cet API. Ca nous libérerait pour écrire ensuite
des interfaces privées différentes...

C'est clair que ce serait beaucoup mieux. Ce qu'il faut, c'est créer
ses API, sans s'en servir tout de suite, et une fois que tous les
outils sont prêts, migration progressive des pages ...

Il me semble qu'il y a trop de code spécifique à MySQL pour envisager
l'utilisation d'une abstraction de base de données permettant
d'utiliser par exemple PostgreSQL, non ?

Ne pas hésiter à se "faire la main" sur des trucs difficiles ! En
ce qui me concerne, vérifier et intégrer des patches qui ne servent
à rien, comme celui de Yannick pour spip_loader, ça ne m'amuse pas
tellement.

C'est un peu excessif de dire que ça "ne sert à rien", mais je vois
bien où tu veux en venir. Par contre, il y a certainement des
personnes qui sont prêtes à aider sur la partie présentation alors
qu'elles n'y comprennent rien (ou pas assez) au fonctionnel/technique
de SPIP ...

Si un système à onglets vous tente, je vous propose à nouveau le
code que j'ai utilisé sur http://www.lecercle.com/actualites/ dont
la partie 'actualités' est gérée par SPIP ... :wink:

Oui, c'est assez joliment fait, et ça paraît fonctionnel sous links
comme sous lynx.

Je donnerais le code (tout simple) et il faudra éventuellement refaire
une ou deux images aux couleurs de SPIP ...

Nicolas.

Salut,

> C'est un très bon début, qui devrait motiver la généralisation à tout
> le reste, et peut-être lancer les réflexions sur la refonte de
> 'ecrire/' sans texte dans les images.

Il y a peut-être d'autres priorités que la présentation ou la conformité du
code html,

Seulement, je ne connais pas le PHP (enfin, disons pas beaucoup) et encore
moins spip...

Je ne peux pas contribuer sur les 'grands chantiers' que tu proposes, de
plus, je ne suis aps convaincu que si j'avais les moyen de proposer un
spip completement réécrit et réorganisé demain, ca plaise vraiment aux
développeurs de spip :wink:

C'est pourquoi je ne peux faire que de 'petites choses' (dans qq mois, si
je fais cela régulierement, je commencerai à vaoir une vue d'ensemble et
onen reparlera).

Ne pas hésiter à se "faire la main" sur des trucs difficiles ! En ce qui me
concerne, vérifier et intégrer des patches qui ne servent à rien, comme
celui de Yannick pour spip_loader, ça ne m'amuse pas tellement.

Bon, ca je comprends. Mais comme ces patch ne touchent pas une ligne de
code PHP mais juste le HTML, tu as juste à vérifier que la page générée
semble correcte, et basta.

Côté technique, je préférerais recevoir non pas des patches (que je ne sais
pas intégrer sous Mac) mais un URL (ftp anonyme ou web avec fichiers en
.txt) où récuperer les fichiers modifiés (oui, je sais, c'est moins bien).

OK, je te mets ca sur le web.

Je regarderai la liste des 'todo' et je verai ce que je peux faire...

  Yannick

Bonsoir,

Hum, finalement, on fait quoi avec ce patch ?

  Yannick