Share met le curseur dans le champ 'TAG' - Githubissues

tontof / kriss_feed

A simple and smart (or stupid) feed reader

280 stars 53 forks source link

Share met le curseur dans le champ 'TAG' #327

Open Nono-m0le opened 10 years ago

Nono-m0le commented 10 years ago

Hello, alors plusieurs choses : 1) Quand je partage un lien (et qu'il y a le "via") : Le curseur est dans le champs "tag" ! (j'ai mis à jour à l'instant, mon shaarli et mon kriss_feed).

2) Quand je ne suis pas connecté à mon shaarli, le "via" n'est pas pris en compte. Je dois me loguer, partager et là seulement je le vois ! Narmol ?

tontof commented 10 years ago

Ce sont 2 problèmes liés à shaarli. Je ne sais pas trop pour le 1) mais pour le 2) c'est parce que le paramètre description n'est pas redirigé correctement quand on n'est pas connecté. Je ne savais pas que le via/la description avaient été ajoutés à shaarli, il va falloir que je teste ça pour voir comment ça a été ajouté, mais a priori je ne peux pas faire grand chose à partir de KrISS feed :-)

Nono-m0le commented 10 years ago

Euh, c'est kriss feed qui le rajoute ! J'ai le via uniquement quand je partage un lien depuis là !

tontof commented 10 years ago

Oui KrISS feed envoie les paramètres et shaarli ça les récupère. Le problème c'est que shaarli les récupère mal quand tu n'es pas connecté car il fait une redirection en omettant le paramètre de la description.

Nono-m0le commented 10 years ago

Mon principale problème (vu qu'on peut rien faire pour le 2), c'est le champs TAG par défaut :D Peut on placer le curseur AVANT le via ? ou au pire, après, mais dans le même champs ?

tontof commented 10 years ago

héhé ben c'est pareil pour la 1), c'est la faute de shaarli :-) mais pour le coup je ne sais pas trop pourquoi. Dans le code html c'est écrit

<body onload="document.linkform.lf_tags.focus();" >

Du coup faut regarder dans shaarli d'où sort ce tags.focus

Nono-m0le commented 10 years ago

pourtant, quand j'utilise le bookmarklet, le focus est dans le champs description !

tontof commented 10 years ago

Oui j'imagine que c'est parce qu'il met le focus sur le premier champ vide. Et comme avec le bookmarklet, il n'y a pas de description, ça le met dans description.

Nono-m0le commented 10 years ago

http://shaarli.m0le.net/?mP1LXQ => Voyons si j'ai des réponses :)

tontof commented 10 years ago

Je ne sais pas si tu as regardé dans le lien de sebsauvage que tu fournis, mais https://github.com/sebsauvage/Shaarli/commit/7b2186a63e2c788f1f2622779c5a1c341117eac1

Corrected field focus in bookmarklet
Focus was not properly given to description field when it's empty.

Donc je pense que c'est bien ça, ça met le focus dans le premier qui est vide. Le problème c'est que quand tu as une description via avec KrISS feed tu vas dans le champ tag. Pas vraiment de bug en fait, juste un comportement qu'il faudrait peut-être amélioré en fonction de paramètres particuliers, mais après c'est assez spécifique et je ne sais pas s'il y a une vraie solution simple.

Nono-m0le commented 10 years ago

J'avais compris ca dans le sens ou, dans tous les cas, ca met le focus dans description, mais là, c'etait plus le cas quand description n’était pas vide ?! => Dans le sens, ca devrait toujours (que ce soit vide ou pas) être focus dans la description

tontof commented 10 years ago

Non en fait, le bug c'était lié à la description qui contenait toujours un retour à la ligne (\n) qui n'est donc pas considéré comme vide, du coup ça allait dans tags tout le temps.

Mais peut-être qu'il faut proposer de laisser le comportement actuel en disant de ne pas aller dans tag pour que le focus se mette à la description si tous les champs sont déjà renseignés. Moi je ne trouve pas ça illogique comme proposition, après faut voir avec Seb :-)

Nono-m0le commented 10 years ago

bon, on a sa réponse : http://sebsauvage.net/links/?lS1fKw

tontof commented 10 years ago

Ben voilà, du coup pas grand chose à faire de mon côté :-)

Nono-m0le commented 10 years ago

bon, là, je suis dans un cas ou il n'y a pas de via, et je suis quand même dans le champs TAG ! (et le bookmaklet me met dans le champs description lui) : http://www.commitstrip.com/fr/2013/12/09/le-truc-quun-codeur-sait-reconnaitre-dinstinct/

tontof commented 10 years ago

J'ai oublié de répondre, mais là c'est effectivement la faute de KrISS feed :-) Je sais comment corriger, faut juste que je trouve/prenne le temps

tontof commented 10 years ago

Alors pour corriger en fait, il faut faire un choix. Dans la config, tu peux virer %0A%0A dans l'url shaarli dans la partie description. En fait ça ajoute des retours à la ligne et fait que la description n'est pas vide ce qui place le curseur dans le champ tag même quand il n'y a pas de via.

Je pensais qu'il fallait que je modifie en testant la valeur du via, mais c'est spécifique et il y a aussi la valeur du texte sélectionné. Quand le paramètre via sera ajouté directement en option dans shaarli, on devrait pouvoir faire différemment, mais en attendant, je ne pense pas qu'il est possible de faire mieux à partir de KrISS feed :-)