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

Landmark #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Landmark #105

wants to merge 1 commit into from

Conversation

totetmatt
Copy link

Mise en place de "Landmark". Permet de partager des géo-positions ou des
adresses (Foursquare mais plus orienté partage et carnet d'adresses). Il
suffit de cocher Landmark lors de l'ajout de lien (via le bookmarklet)
pour que le snippet javascript prenne votre position actuelle et change

le lien en lien openstreetmap (Jquery est requis)

Implementing "Landmarks". Allow to share geo-positoons / addresses
(Foursquare-like but more sharing oriented). Just check the Landmark
checkbox when you add a link (with the bookmarklet) and it will get your
position and change the link to an openstreetmap link (Jquery is
requiered)

Mise en place de "Landmark". Permet de partager des géo-positions ou des
adresses (Foursquare mais plus orienté partage et carnet d'adresses). Il
suffit de cocher Landmark lors de l'ajout de lien (via le bookmarklet)
pour que le snippet javascript prenne votre position actuelle et change
le lien en lien openstreetmap (Jquery est requis)
==
Implementing "Landmarks". Allow to share geo-positoons / addresses
(Foursquare-like but more sharing oriented). Just check the Landmark
checkbox when you add a link (with the bookmarklet) and it will get your
position and change the link to an openstreetmap link (Jquery is
requiered)
@Nabellaleen
Copy link

Il faudrait que ce genre de "feature" soit sous forme de "plugins", pour ne pas encombrer le code central afin de rester dans une philosophie KISS.
Reste la question : comment mettre en place un système de plugin qui reste dans cette philosophie KISS ;) Un avis, @sebsauvage ?

@totetmatt
Copy link
Author

Tout a fesse d'accort. Mais bon y'a pas vraiment de système de plugins... on est bloqué. ça peux rester une branche indépendante pour l'instant....

@nodiscc
Copy link

nodiscc commented Nov 18, 2014

Hello tous. @totetmatt @Nabellaleen il y a une proposition pour un système de plugins sur le fork communautaire shaarli#52, les contributions sont bienvenues. C'est encore incomplet mais on peut sans doute l'étendre.

Si je comprends bien, cette pull request ajoute une checkbox au dialogue "Edit link", un script, et écrit la valeur du landmark dans la liste des liens? La valeur du landmark étant quoi? La position actuelle depuis laquelle on post, obtenue depuis la fonction de géolocalisation du navigateur?

Allez jeter un coup d'oeil à shaarli#52 pour voir comment on pourrait ajouter ça.

Petite note, jQuery a été retiré de la page principale par seb il y a quelque temps. C'est toujours présent dans le dialogue "Edit link" par contre.

Vos commentaires sur le système de plugins sont bienvenus, ou une autre pull request sur lé dépôt shaarli/Shaarli (le fork est sans doute temporaire, on attend une décision de @sebsauvage concernant un éventuel merge. Pour l'instant il semble qu'il n'aie pas le temps de maintenir Shaarli. On a déjà fait un peu le ménage dans les nombreux forks - voir https://github.com/shaarli/Shaarli/network - et dans les bugs https://github.com/shaarli/Shaarli/issues)

Sinon, @totetmatt je pense que tu peux raisonnablement fermer cette PR, elle ne merge plus.

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

Successfully merging this pull request may close these issues.

None yet

3 participants