New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom banner #44

Closed
wants to merge 10 commits into
base: master
from

Conversation

2 participants
@B4rb3rouss
Contributor

B4rb3rouss commented Jan 2, 2018

Ajoute une option pour définir l'image d'en-tête.
Quelques commentaires :

  • Ça ne fonctionne qu'avec le thème par défaut
  • J'aurais voulu passer par un import de feuille CSS, mais impossible d'accéder à l'option configurée par l'utilisateur
  • J'ai tenté d'utiliser un style.php, mais de même, impossible d'accéder aux options de l'addons.
    On pourrait faire plus propre (plutôt que de se servir d'une recherche de ''), mais c'est tout aussi rapide.

Peut-être cet addon sera vite obsolète avec les changements qui arrivent en 3.8 ou 4.0, mais d'ici là, à voir ;)

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Jan 30, 2018

Contributor

Je ne peux pas pousser dans l'état Il y a 2 addons dans ton PR :/
Je pense que le mieux et que dans ton fork, tu crées 1 branche par addon, dès que tu as un addon de bon, tu peux pousser ici depuis la branche de ton addon.
Je sais que ça peux être pénible, mais c'est plus propre et ça évite que je passe trop de temps à vérifier qu'il n'y ait pas de régression ou d'autres problèmes sur un autre addon lorsque tu pousses.

Contributor

remrem commented Jan 30, 2018

Je ne peux pas pousser dans l'état Il y a 2 addons dans ton PR :/
Je pense que le mieux et que dans ton fork, tu crées 1 branche par addon, dès que tu as un addon de bon, tu peux pousser ici depuis la branche de ton addon.
Je sais que ça peux être pénible, mais c'est plus propre et ça évite que je passe trop de temps à vérifier qu'il n'y ait pas de régression ou d'autres problèmes sur un autre addon lorsque tu pousses.

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 31, 2018

Contributor

Ce n'est pas pénible, c'est moi qui suis un boulet. Désolé.

Nouvelle PR ici : #50
en espérant avoir réussi à utiliser git correctement.

Contributor

B4rb3rouss commented Jan 31, 2018

Ce n'est pas pénible, c'est moi qui suis un boulet. Désolé.

Nouvelle PR ici : #50
en espérant avoir réussi à utiliser git correctement.

@B4rb3rouss B4rb3rouss closed this Jan 31, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment