Sublime-Csslisible icon indicating copy to clipboard operation
Sublime-Csslisible copied to clipboard

Appel automatique à la sauvegarde

Open NumEricR opened this issue 9 years ago • 12 comments

Hello, Serait-il possible d'ajouter un paramètre format_on_save pour utiliser CSSLisible automatiquement à l'enregistrement sans raccourci clavier spécifique ?

A titre d'exemple Sublime-HTMLPrettify, SublimeJSCSFormatter ou Sublime-CSS-Format le proposent.

Merci !

NumEricR avatar Jun 11 '15 20:06 NumEricR

Je ne suis pas fermé à l'idée mais je n'ai pas du tout de temps actuellement. Si tu as le temps de faire une PR ou quelqu'un d'autre ça sera merge avec plaisir.

thierrylemoulec avatar Jun 11 '15 21:06 thierrylemoulec

Oh, je viens d'essayer de "bricoler" le truc et en 5min ça fonctionne au moins avec ST2 ! :four_leaf_clover: Du coup je vais vérifier sur ST3 et faire le PR dans les prochains jours.

cc @Darklg

NumEricR avatar Jun 11 '15 22:06 NumEricR

J'ai voulu vérifier sous ST3 ... mais je dois mal m'y prendre. Si je télécharge les sources et les place dans le dossier Packages, sans modifications, le plugin ne fonctionne pas.

J'ai aussi tenté l'install depuis Package Control, fonctionnement ok évidemment. :D Mais comment modifier les sources ? Avec PackageRessourceViewer, les sources ne semblent pas les mêmes !

NumEricR avatar Jun 14 '15 11:06 NumEricR

Une piste pour que je puisse tester ma modif sous ST 3 ?

NumEricR avatar Jun 17 '15 20:06 NumEricR

Non aucune idée ça fait trop longtemps que j'ai pas touché à ça peut-être que @Sheeprider peux t'aider c'est lui qui avait fait les modification pour ST3

Thierry Le Moulec - Co-fondateur et développeur à Smoothie Creative http://smoothie-creative.com/ @thierrylemoulec https://twitter.com/thierrylemoulec | +33 6 81 60 44 41

On Wed, Jun 17, 2015 at 10:37 PM, Eric R [email protected] wrote:

Une piste pour que je puisse tester ma modif sous ST 3 ?

— Reply to this email directly or view it on GitHub https://github.com/thierrylemoulec/Sublime-Csslisible/issues/12#issuecomment-112942103 .

thierrylemoulec avatar Jun 17 '15 20:06 thierrylemoulec

Ok merci j'attends son retour alors :)

NumEricR avatar Jun 17 '15 21:06 NumEricR

@Sheeprider : un peu de temps pour m'aider à tester ma modif sous ST3 ?

NumEricR avatar Jul 12 '15 15:07 NumEricR

@NumEricR Désolé pour le délai, je suis maintenant disponible pour t'aider à tester ta modif.

Sheeprider avatar Jul 18 '15 12:07 Sheeprider

Ok merci.

Du coup j'ai modifié les sources du plugin. Ca fonctionne bien sous ST2 mais je ne comprend pas comment je peux le faire en v3. cf mon précédent commentaire.

Voici le code ajouté :

class CssLisibleEventListeners(sublime_plugin.EventListener):
    @staticmethod
    def on_pre_save(view):
        if view.settings().get('syntax') == 'Packages/CSS/CSS.tmLanguage' and settings.get('format_on_save'):
            view.run_command('csslisible')

Et évidemment le nouveau paramètre dans Csslisible.sublime-settings (même si je pense que par défaut il faudra le laisser à false) :

    "format_on_save": true

NumEricR avatar Jul 18 '15 17:07 NumEricR

J'ai modifié un peu ton code, sachant que settings n'est pas défini globalement sur la branche st3. Je prends aussi en compte le nouveau paramètre défini au niveau du projet (je veux bien une confirmation que cette partie fonctionne).

class CssLisibleEventListeners(sublime_plugin.EventListener):
    @staticmethod
    def on_pre_save(view):
        global_settings = sublime.load_settings('Csslisible.sublime-settings')
        format_on_save = view.settings().get('format_on_save', global_settings.get('format_on_save', False))

        if view.settings().get('syntax') == 'Packages/CSS/CSS.tmLanguage' and format_on_save:
            view.run_command('csslisible')

Sheeprider avatar Jul 18 '15 18:07 Sheeprider

Ok mais comme je le disais avant : comment tester ? :confused: De plus ne serait-il pas possible de conserver le support ST2 en parallèle ?

NumEricR avatar Aug 09 '15 09:08 NumEricR

Pour tester je te conseille de cloner le repo dans ton dossier de plugin puis de changer de branche pour avoir la version ST3. Et nous n'avons pas trouvé de moyen d'avoir une version compatible avec ST2 et ST3, car l'API est différente en plus de la version de python; donc on reste sur deux versions sur des branches git séparées.

Sheeprider avatar Aug 30 '15 13:08 Sheeprider