search

FreederTeam / Freeder

An easy-to-use free feed reader
Other
5 stars 1 forks source link

Intégrer l'installation dans le thème #17

Closed tmos closed 10 years ago

tmos commented 10 years ago

On fait comment pour styler la page d'installation ?

eliemichel commented 10 years ago

On fait une page install.html dans le thème !

eliemichel commented 10 years ago

Hum, problème évident : Rain TPL n'est pas encore fonctionnel avant installation… Du coup l'idéal serait de pouvoir intégrer le thème automatiquement dans la page d'installation. Un truc du genre un install.html dans le thème dont on récupère le fichier de cache de Rain TPL côté dev et on l'ajoute au dépôt. Je vois juste pas bien comment automatiser le truc… Un lien dûr peut-être, mais ça passe pas au dépôt donc chacun doit faire le lien de son côté, c'est pas top.

edit: Dans tous les cas faut pas laisser ça en vrac dans inc/install.php… Et du coup on peut peut-être faire un design dans le même style mais épuré pour la page d'installation, à l'instar de WordPress.

eliemichel commented 10 years ago

93fad87 ajoute un début de thème. C'est un peu trop hacky à mon goût, surtout au niveau des chemins (j'ai ajouté du tpl/default/ un peu partout, mais c'est mal). Le logo s'affiche pas par exemple…

En gros j'ai repris le design de la page de login (d'ailleurs, j'ai utilisé l'id loginform, faudra régler ça). Et on peut pas mettre des placeholders comme pour le login vu qu'il y a une valeur par défaut à timezone. Faudra voir si on peut pas mettre une infobulle pour timezone et les placeholders pour le reste.

edit: Une image : https://raw.githubusercontent.com/FreederTeam/Freeder/master/doc/screenshots/install.png

Phyks commented 10 years ago

J'ai une idée pour ça si vous voulez pour inclure l'install.HTML du thème.

En gros, on fait une variable en dur pour spécifier le thème et on draw le install associé.

Phyks

On 25 juillet 2014 23:10:42 UTC+02:00, Elie Michel notifications@github.com wrote:

Preview: https://lut.im/7VbQKmwi/lCRzFdah

Reply to this email directly or view it on GitHub: https://github.com/FreederTeam/Freeder/issues/17#issuecomment-50205043

eliemichel commented 10 years ago

Oui je pense qu'on va faire ça, surtout que je lie déjà quelques variables à la page pour les erreurs et les valeurs des champs donc l'envoi de variables est déjà en place. C'est juste que du coup si on change le layout général ça change pas celui de l'installation, mais c'est pas un drame !

edit: 664462c fait ça. Y a une variable en haut de inc/install.php pour changer le thème par défaut.

Phyks commented 10 years ago

Closed by https://github.com/FreederTeam/Freeder/commit/664462c8cc9d5d381b872a4b22f2680f0b7b3d09.