tontof / kriss_feed

A simple and smart (or stupid) feed reader
281 stars 53 forks source link

undo last action button #379

Closed Nono-m0le closed 7 years ago

Nono-m0le commented 7 years ago

Serait-il possible d'ajouter un bouton 'undo last action', car encore une fois, j'ai marqué tous mes flux en "lu", sans trop savoir comment d'ailleurs...

Et dans ce cas précis, avec quelques flux toujours en "non lu" depuis qq semaines/mois sont tout simplement introuvable désormais !

tontof commented 7 years ago

Désolé, mais c'est une action bien trop compliquée à mettre en place. Même si je comprends bien l'idée et le besoin, je ne vois pas comment je pourrais ajouter ça simplement :-( Cette fonctionnalité me manque aussi parfois quand je marque un article comme lu trop vite. Avant la refonte de KriSS feed (version 5) je gardais un historique dans le navigateur qui me permettait de faire un undo sur les 10 derniers éléments. Mais du coup ce n'était pas global, juste au niveau des articles.

Nono-m0le commented 7 years ago

Bon, AGAIN :p Serait-il possible de placer un cartouche de confirmation lorsqu'on tente de "marquer tous les éléments en lu" ? EDIT: Ca semble être exactement la meme chose que la dernière fois : Mes gros doigts sur mon téléphone !!!

tontof commented 7 years ago

En attendant que je regarde pour voir si je peux faire une modif, tu peux peut-être tout simplement supprimer dans le menu la possibilité de marquer comme lu/non lu ? Comme ça plus de problème lié à tes gros doigts ?

Nono-m0le commented 7 years ago

J'ai trouvé !!!

En fait, parfois, l'auto scroll up monte la page tout en haut et affiche donc le compteur total des news. Quand on clic sur ce nombre total => ca passe tout en lu.

Comment je désactive cette action ?

tontof commented 7 years ago

J'suis pas contre une petite capture d'écran pour me montrer de quel nombre total tu parles. Pour moi il n'y a que sur la liste des flux où il y a un nombre qui marque tous les flux comme lus 206 par ex ici : https://tontof.net/feed/ Mais si c'est ça, je n'ai pas trop compris le rapport avec l'auto scroll up. Du coup c'est quel action que tu veux désactiver, l'auto scroll up ou le nombre qui marque tous les flux ?

Nono-m0le commented 7 years ago

C'est bien le 206 oui. J'aimerais enlever le fait de marquer tout comme lu sur ce chiffre (je ne comprends à vrai dire, même pas son intérêt ici !)

tontof commented 7 years ago

Sur celui là ou sur tous ? Disons que c'est un raccourci pour effectuer une action sans cliquer sur plein de boutons. Il m'arrive d'y cliquer dessus quand j'ai beaucoup trop d'articles à lire et que je n'ai pas envie de tout lire. Par contre je ne le fais effectivement pas souvent sur celui là en particulier mais plus sur un dossier de flux ou un flux en particulier.

tontof commented 7 years ago

Si tu ajoutes ou modifies ton style.css en ajoutant ça, ça doit marcher:

Nono-m0le commented 7 years ago

Uniquement celui-là. Les autres (même si que très rarement) ca m'arrive de les utiliser également.

La, celui est genant car quand on a des micro-lag sur le téléphone (et qu'on fait plusieurs "read" à la suite/rapidement).

Ben, il arrive ce qu'il arrive, à la place de tapper sur "read", c'est sur le "206" que ca tombe et ... plus de flux :D

Nono-m0le commented 7 years ago

Normal que je n'ai pas de style.css ? :D (et que j'ai vu passer un this-kfc dans ton code ? :p )

tontof commented 7 years ago

https://tontof.net/kriss/feed/#style Il faut créer un dossier inc qui contient un fichier user.css (pas style.css comme je disais)

Du coup j'suis pas sûr d'avoir compris ton this-kfc .

Nono-m0le commented 7 years ago

Dans index.php: 41 public function __construct($dataFile, $cacheDir, $kfc) 42 { 43 $this->kfc = $kfc;

Merci pour le css, ca semble marcher (j'ai peur de tester sur mon mobile :p)

Nono-m0le commented 7 years ago

Bon, il se trouve que ce n'est pas seulement le "206" qui n'est plus "cliquable" mais le "All Feed" également, et celui-là, il m'arrive de l'utiliser :) Une solution ?

tontof commented 7 years ago
#all-subscriptions > h4 > a:first-child {
     pointer-events: none;
}

à la place :-)