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

About #40

Open
wants to merge 7 commits into
base: master
from

Conversation

2 participants
@B4rb3rouss
Contributor

B4rb3rouss commented Dec 28, 2017

Cet addon rétabli l'idée d'une page "à propos", disponible en suivant le lien URL-du-blog/?about

Quelques éléments de configuration son proposés, mais pouvoir mettre du code html serait bien pratique (pas échappé, cf #34 )

Sinon, c'est nouveau pour moi les hooks, et je ne suis pas très à l'aide avec libxml, donc si vous voulez tailler à la hache, faîtes-vous plaisir.

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Feb 18, 2018

Contributor

Je viens de jeter un coup d'oeil dans le détail de l'addon.
Tu utilises libxml pour parser le contenu et intégrer le contenu de la page addon, je pense que ce n'est pas une approche valide car tu te réfères à #main ($main = $doc->getElementById('main');), le problème que je vois c'est si BT tourne avec un template fait maison, il n'y aura pas forcément de #main.

Je vois pour trouver une autre approche dès que j'ai un peu de temps...
Et je vois aussi pour donner plus de possibilités pour les settings des addons ;)

Contributor

remrem commented Feb 18, 2018

Je viens de jeter un coup d'oeil dans le détail de l'addon.
Tu utilises libxml pour parser le contenu et intégrer le contenu de la page addon, je pense que ce n'est pas une approche valide car tu te réfères à #main ($main = $doc->getElementById('main');), le problème que je vois c'est si BT tourne avec un template fait maison, il n'y aura pas forcément de #main.

Je vois pour trouver une autre approche dès que j'ai un peu de temps...
Et je vois aussi pour donner plus de possibilités pour les settings des addons ;)

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Feb 18, 2018

Contributor

Si on peut mettre du html dans les settings d'un addons, alors là ça sera hyper simple, en tout cas beaucoup plus !
Il y a plus urgent que cet addons de toutes façons, comme les failles de sécu à corriger pour la 3.7.7.

Contributor

B4rb3rouss commented Feb 18, 2018

Si on peut mettre du html dans les settings d'un addons, alors là ça sera hyper simple, en tout cas beaucoup plus !
Il y a plus urgent que cet addons de toutes façons, comme les failles de sécu à corriger pour la 3.7.7.

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Feb 18, 2018

Contributor

+1, mais là n'étant pas à domicile, je suis limité dans mes usages, du coup, je jette un oeil sur les addons...

Contributor

remrem commented Feb 18, 2018

+1, mais là n'étant pas à domicile, je suis limité dans mes usages, du coup, je jette un oeil sur les addons...

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