-
Notifications
You must be signed in to change notification settings - Fork 11
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
Créer un gestionnaire de notes #82
Comments
Pour l'affichage du menu, il y plusieurs solutions :
ça donne un truc comme ça sur desktop : Icônes utilisées :
Variante : le menu de navigation en bas de l'écran.
Efficace sur mobile, beaucoup moins sur desktop. Or l'usage sur desktop est prédominant pour cette fonctionnalité donc je suis pas non plus convaincu.
|
La couleur de l'interface (top bar + barre du menu) pourrait changer selon le mode actif (Carte / Liste) pour bien marquer la différence. Ou pas. |
La disposition actuelles des notes dans NotesReview créé des blocs de hauteur homogène, qui est la disposition n°1 (telle que définie par le material design) : Le scroll infini sans avoir à cliquer sur un bouton me paraît une bonne idée. Etant donné qu'on est sur un projet du mois avec un nombre limité de notes, on peut charger la totalité des notes. Par ailleurs, NotesReview n'affiche que 3 colonne sur grand écran (le mien a une résolution de 1920x1080). Je crois qu'on peut en mettre au moins 4 sur un écran aussi large. |
Il faut une 404 : aucune note existante. Je propose une belle photo avec la mention "On n'a rien pour vous pour l'instant. Repassez plus tard." |
On n'en a pas parlé mais j'ai l'impression qu'on peut garder le CTA principal même sur la page des notes : le bouton (+) qui permet de créer une note. S'il est par dessus des notes existantes, je pense que c'est très bien comme ça. |
Une première version est à tester ici : https://projetdumois.fr/projects/2020-09_aed/issues À noter que :
Preneur de retours au-delà de ces points là 😉 |
Pourquoi réinventer la roue ? 😉 |
Parce qu'à terme on va aussi afficher dans cette page les signalements de tuveuxmaphoto.fr et les remontées Osmose (si ça s'y prête). Et aussi qu'on a plus facilement le lien avec l'éditeur intégré, et que ça filtre automatiquement sur les notes du projet du mois. |
ça peut se faire aussi sous forme de notes (le stockage de photo de StreetComplete est disponible aussi pour PdM 😉 )
Ce lien aussi 😜 |
Sauf si les signalements n'ont pas de géolocalisation. Ou que l'utilisateur de tuveuxmaphoto envoie une photo qui n'est pas à partager publiquement (uniquement à destination du contributeur OSM car on y voit des visages, des plaques d'immatriculation...). |
Une ville c'est bien, à défaut la France.
Ce sont des notes avec des photos temporaires. |
La barre latérale avec les filtres mange trop d'espace. |
On peut la cacher la barre latérale, sur le même mode de fonctionnement que celle de la carte. |
Je trouve que 4 colonnes c'est bien avec un grand écran. Lorsque l'écran devient plus petit, j'ai l'impression qu'on peut le faire disparaître s'il atteint 280px ou moins. |
La place que prend la barre latérale ne me choque pas. Mais si on n'y affiche que des filtres, ça peut valoir le coup de les mettre en haut, question d'ergonomie; je pense qu'on pourra voir un peu plus tard ce point précis. |
Le nombre de colonne se change uniquement en cas de rechargement de la page. Actuellement c'est :
|
OK, ça va. J'avais testé de redimensionner la page pour voir si elle était full responsive, d'où mon retour. Mais ta solution convient bien également. |
En fait c'est géré comme ça pour avoir un affichage sur une logique ligne à ligne (et pas colonne par colonne, qui permettrait d'être full responsive), car ça me semble plus sensé d'avoir une progression des dates sur une logique de pseudo lignes que de colonnes. |
très bonne idée |
OK j'ai un peu joué avec les cards et voici ma proposition :
Seul problème de ma proposition : sur mobile je pense que ça prendra une place verticale beaucoup trop importante. On pourra virer des trucs superflus à l'occaz' si besoin. |
Ça rend top comme ça 😄 Juste pour le 3, ça va nous apporter une complication technique pour une valeur ajoutée faible ? On y mettrait pas plutôt le nom du contributeur ? Et pour le 5 la carte c'est design mais ça risque d'être lourd à charger. À moins de se lancer dans un système qui charge la carte seulement si elle est visible à l'écran... Pas le plus simple. |
Ouais je m'attendais à un retour sur ces deux points lol. La différence entre le moment où tu design un beau truc et où il doit être implémenté. |
Le titre pourrait être la nature du signalement : via tuveuxmaphoto, note OSM, Osmose... OK pour le défilement photo. |
OK, essayons la nature du signalement pour voir ce que ça donne |
PS : pour moi, le comportement attendu de cliquer sur le nom d'un utilisateur est de filtrer toutes les notes qu'il a créé, et pourquoi pas celles sur lesquelles il a répondu |
|
👍 pour rester cohérent avec le filtrage des couches |
quand aux autres filtres, on peut leur mettre des icônes : |
NoteReview le garde, mais comme c'est affiché dans une boite qui occupe presque tout l'écran, il y a de la place. Et ce que je trouve long c'est "… par" 😉 |
👍 |
Que signifie exactement Prêt à intégrer ? |
PS : après avoir cliqué sur "éditer" sur une note, quand la carte s'ouvre elle se recentre sur la position du GPS de l'utilisateur. |
Pour les photos, c'est normal qu'elles ne s'affichent pas, ce ne sont pas les liens directs vers les fichiers images... |
Mais bon j'imagine qu'il faut ajouter le bout de code pour deviner l'URL de la photo si c'est une image qui vient de Commons. |
Et comment font les autres applications OSM ? Cf. note de test. |
Elles sont obligées de gérer un cas particulier avec du code spécifique 😓 |
Il est même ici le code spécifique pour gérer ça https://github.com/ENT8R/NotesReview/blob/ef416315669d5aa7b6ed85816184cb61612ad654/js/linkify.js |
J'ai ouvert un ticket pour qu'il affiche aussi les photos Mapillary. |
C'est cool @PanierAvide tu fais la discussion tout seul : tu poses le problème et trouves la solution; il ne manque plus que l'implémentation :D |
Je reviens sur ma position : survoler et cliquer sur le nom permet de savoir à qui on a affaire. |
Je ne suis pas certain de comprendre ta proposition @pyrog. Peux-tu stp la détailler ? |
@pyrog mais en fait le bouton sera le même que sur la liste des notes, c'est juste qu'il est caché pour l'instant vu que la fonctionnalité n'est pas rendue publique à cause des ajustements à réaliser... |
linkify.js affiche désormais les photos Mapillary 😃 |
PdM va afficher une liste de notes quelque soit les couches visualisées sur la carte.
Oui je m'en doute. Mais suivant sa position et son libellé, l'utilisateur va comprendre que c'est :
Peux-tu bricoler stp une "copie" d'écran ? |
Je viens de faire une mise à jour, toujours en test donc pas de lien depuis la carte :
|
Pas de nouvelles = bonne nouvelle ? En tout cas c'est désormais en prod 😁 |
Premier retours La paire de boutons Carte/Liste est clair 😃 cf. #100 |
Sur une suggestion de @nlehuby, on pourrait intégrer un système de gestion des notes liées au projet du mois en cours.
Il s'agit donc d'intégrer le mode expert de Notes Review https://ent8r.github.io/NotesReview/expert/
A terme le gestionnaire pourrait intégrer en plus des notes 1. les signalements issus de tuveuxmaphoto.fr 2. les signalements OSMOSE.
Fonctionnalités clef :
The text was updated successfully, but these errors were encountered: