Remplacer les liens d'actions des commentaires par un menu contextuel #100

Merged

Lephenixnoir merged 5 commits from context-menu into dev

2021-07-12 18:37:10 +02:00

Lephenixnoir commented

2021-07-10 21:57:52 +02:00

Owner

Cette PR ajoute un menu contextuel pour remplacer les liens d'action sur les messages, sauf le permalien qui est mis sur la date (une technique courante, eg. sur Twitter).

Avantages :

Visuellement plus compact, ça tient sur une ligne et ça perturbe moins le flot quand un message commence par un élément de largeur complète (code/quote).
On peut rajouter plus d'actions sans se préoccuper de la place que ça prend :
- Suppression avec pénalité de points (voir ci-dessous) ;
- Suppression de topic vs. post principal (qui est sur la TODO list) ;
- Déplacer des messages vers un nouveau topic (TODO list aussi) ;
- Ma suggestion perso : un raccourci pour supprimer + kick/ban, en cas de spam.

Inconvénients :

Un clic de plus est nécessaire pour accéder à ces actions (pas ultra courantes mais un peu quand même).
Un peu moins naturel sur mobile (et encore, les boutons du menu contextuel sont plus gros que les liens qui précédaient).

La base du menu fonctionne en pur HTML/CSS, et est compatible avec le thème. Le menu est un <details> dont les trois points sont le <summary>. Le positionnement est relatif à la division contenant la date et le menu.

Il y a un bout de Javascript dans pc-utils.js qui ferme les menus ouverts si on clique en-dehors (comme le menu principal), mais c'est optionnel ; on peut toujours fermer en re-cliquant sur l'icône si JS est désactivé.

Cette PR ajoute un menu contextuel pour remplacer les liens d'action sur les messages, sauf le permalien qui est mis sur la date (une technique courante, eg. sur Twitter). Avantages : * Visuellement plus compact, ça tient sur une ligne et ça perturbe moins le flot quand un message commence par un élément de largeur complète (code/quote). * On peut rajouter plus d'actions sans se préoccuper de la place que ça prend : - Suppression avec pénalité de points (voir ci-dessous) ; - Suppression de topic vs. post principal (qui est sur la TODO list) ; - Déplacer des messages vers un nouveau topic (TODO list aussi) ; - Ma suggestion perso : un raccourci pour supprimer + kick/ban, en cas de spam. Inconvénients : * Un clic de plus est nécessaire pour accéder à ces actions (pas *ultra courantes* mais un peu quand même). * Un peu moins naturel sur mobile (et encore, les boutons du menu contextuel sont plus gros que les liens qui précédaient). ![](https://www.planet-casio.com/storage/forums/v5-menu-2-183659.gif) La base du menu fonctionne en pur HTML/CSS, et est compatible avec le thème. Le menu est un `<details>` dont les trois points sont le `<summary>`. Le positionnement est relatif à la division contenant la date et le menu. Il y a un bout de Javascript dans `pc-utils.js` qui ferme les menus ouverts si on clique en-dehors (comme le menu principal), mais c'est optionnel ; on peut toujours fermer en re-cliquant sur l'icône si JS est désactivé.

Lephenixnoir added the

enhancement

proposal

labels 2021-07-10 21:57:52 +02:00

Darks was assigned by Lephenixnoir

2021-07-10 21:57:52 +02:00

Lephenixnoir force-pushed context-menu from 2764223a21 to 5505ee0b75

2021-07-11 11:00:19 +02:00

Compare

Lephenixnoir commented

2021-07-11 11:00:42 +02:00

Author

Owner

J'ai rebasé, corrigé un bug, et ajouté la penalité d'XP comme discuté.

Lephenixnoir force-pushed context-menu from 5505ee0b75 to 79289146a2

2021-07-12 17:50:32 +02:00

Compare

Lephenixnoir added 1 commit 2021-07-12 18:31:40 +02:00

00813bf71c

forum: enable topic deletion

Topic modification does not work in this snapshot, this is normal.

Lephenixnoir merged commit b8ed0bba99 into dev

2021-07-12 18:37:10 +02:00

Lephenixnoir deleted branch context-menu

2021-07-12 18:37:25 +02:00

No reviewers

No Label

No Milestone

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#100

No description provided.