Skip to content
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

Enlever le QR code #171

Closed
jeanber opened this issue Feb 28, 2014 · 3 comments
Closed

Enlever le QR code #171

jeanber opened this issue Feb 28, 2014 · 3 comments

Comments

@jeanber
Copy link

jeanber commented Feb 28, 2014

Tout est dans le titre. A part dans le css je ne vois trace de l'affichage du QR code dans aucun fichier.
Où pourrais-je supprimer la génération de ce QR code que je trouve assez inutile et qui doit nécessairement manger un peu de ressources.
Merci pour votre réponse

@Knah-Tsaeb
Copy link
Contributor

C'est dans tpl/linklist.html que ce trouve le code de génération du Qrcode, dans la partie javascript. Pour les ressources, elle ne sont utilisées que lors de la demande d'affichage du Qrcode, c'est à ce moment là qu'il est généré. Donc dans les fait ça consomme que dalle.

@Sbgodin
Copy link
Contributor

Sbgodin commented Feb 28, 2014

De plus, ça ne consomme que les ressources du client. Le serveur n'est pas mis à contribution pour le calcul lors de la génération du QR code.

@nodiscc
Copy link

nodiscc commented Nov 10, 2014

@jeanber Hello, il est prévu de rendre le QR code optionnel dans le fork temporaire sur https://github.com/shaarli/Shaarli (en attendant que @sebsauvage aie un peu de temps pour vérifier et peut être merger les modifs).

Une des solutions prévues est ne le charger que comme "plugin" (voir shaarli#57) si le système de plugins proposé (shaarli#52) est accepté. Il faudra alors avoir qrcode dans la liste des plugins activés pour que celui ci soit chargé.

@jeanber Peut on fermer ce bug et continuer sur shaarli#57 ? Le tracker de bugs de https://github.com/sebsauvage/Shaarli/ est déjà bien encombré et il y a peu de chances que ce soit réglé ici.

Merci!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants