ademcan / canSnippet

An open source web-based snippets management tool
MIT License
48 stars 7 forks source link

Feature.prism css selector #11

Closed bawaaaaah closed 10 years ago

bawaaaaah commented 10 years ago

ajout d'une feature permetant de changer le theme prism les instruction sont dans le README partie action d'admin et liée avec le fichier config.php

pour mettre a jour le sqlite sans reinstaller: ALTER TABLE "main"."settings" ADD COLUMN "prismtheme" longtext;

ademcan commented 10 years ago

Ca c'est cool, c'est le truc le plus important sur lequel je voulais travailler, et ajouter les plugins de prism.js aussi. Je teste ce soir :)

bawaaaaah commented 10 years ago

la c'est que la gestion du css pas des plugin

ademcan commented 10 years ago

oui oui, je disais que j'aimerais ajouter les plugins aussi, je vais certainement travailler cette partie prochainement ^^

bawaaaaah commented 10 years ago

j'avais vu ça dans ta todo list sur ton blog si j'ai un peut de temps (faut quand même que je bosse un peut) je jetterai un coup d'oeil sauf si quelqu'un a déjà commencé

baptistedonaux commented 10 years ago

oui d'ailleurs le CSS aurait besoin d'un petit coup de jeune :)

baptistedonaux commented 10 years ago

Pour la personne qui s'en occupera, je vous conseille de mettre le nom du thème dans $parameters, et de stocker le path des styles en BDD.

ademcan commented 10 years ago

@baptistedonaux : tout à fait d'accord pour le css :) ok merci @bawaaaaah : oui tu as déjà pas mal de PR en attente, prends le temps de bosser aussi comme ça je pourrai aussi prendre le temps de checker les PR :D

baptistedonaux commented 10 years ago

Je ne suis pas la jusqu'à la fin de la semaine mais si personne ne s'y est attaqué, je peux te faire un petit design qui serait responsive ce qui peut être pas mal, je pense.

bawaaaaah commented 10 years ago

@baptistedonaux +1

ademcan commented 10 years ago

Normalement le CSS actuel devrait passer sur mobile aussi non ? J'avais fait en sorte qu'il soit un minimum responsive, même si je dois avouer que ce n'est pas optimal.

baptistedonaux commented 10 years ago

Je n'ai pas testé. Il faudrat le faire si tu veux refondre ton CSS.

ademcan commented 10 years ago

Alors voilà ce que je propose :) Je pense qu'il faut créer une branche "dev" où l'on pourra faire nos PR et tester sans problèmes, du coup on laisse la branche "master" pour les versions. On pourrai alors avoir un plan pour la v2.0 ,v3.0... On mettra tout sur la branche master quand on aura rempli le "cahier des charges" de la version suivante. Tout cela demande à ce que j'organise le dépôt, je vais voir ce que je vais faire, car je n'ai pas envie de donner fin à l'aventure Github, ça serait dommage de se priver des dev comme vous, j'ai besoin d'aide dans pas mal de domaine. Mais tout va très vite, je vais donc me poser ce week-end et voir comment on pourra avancer. Merci les amis :)

baptistedonaux commented 10 years ago

Pas de soucis, pour le coup je vais voir comment je fais également avec ma branche master sur mon fork (PR en attente) car je suis bloqué pour le moment. On en reparle lundi dans ce cas.

ademcan commented 10 years ago

Ok ça marche. D'ailleurs au début j'avais prévu d'attendre le week-end pour me lancer dans github mais la tentation était trop forte, merci encore

bawaaaaah commented 10 years ago

@ademcan +1 pour une branch dev, aprés tkt pas on peut attendre que les PR soit intégrer plus tard surtout certain le mien proposant une new feature (la gestion des theme pour prism) il peut attendre aprés ceux niveau sécu c'est autre chose.

ademcan commented 10 years ago

Ouai tout à fait d'accord, comme ça on fait les choses correctement ;) Merci, je vous tiens au courant de l'avancement

ademcan commented 10 years ago

@bawaaaaah est-ce que tu pourrai me faire une PR de ce commit sur la branche "dev" quand tu aura le temps ? merci

bawaaaaah commented 10 years ago

ok, j'ai par contre l'impression qu'il va falloir que je supprime et recrée le PR

ademcan commented 10 years ago

Ouep pas de soucis, ce n'est pas urgent ;)