Development #34790
améliorer la présentation des endpoints
0%
Description
Mettre en valeur la méthode http, avoir un peu plus de séparation entre l'url et la description.
En bossant sur un connecteur je me suis dit que s'il y a une multitude de endpoints avoir une distriction visuelle permet plus facilement de s'y retrouver.
Un peu dans le style de Swagger.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
Mettre en valeur la méthode http
Pourquoi ?
Est-ce vraiment l'information la plus importante ?
(aussi, reléguer la description fonctionnelle, qui me semble bien plus utile, en bout à droite, pas ok)
(+ des détails qu'on verra plus tard)
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
C'est mignon j'aime bien, maintenant je ne sais pas à quel point on ne devrait pas juste produire du swagger plutôt que de le refaire en CSS, swagger c'est aussi les widgets pour tester les appels, c'est quand même bien pratique.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
Est-ce vraiment l'information la plus importante ?
Je ne crois pas non plus, l'intérêt que j'y vois est plus de se rapprocher d'une présentation devenue 'standard' dans le domaine.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Lié à Autre #10854: expérimenter un connecteur basé sur djangorestframework ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
Benjamin Dauvergne a écrit :
Frédéric Péters a écrit :
Pourquoi ?
Est-ce vraiment l'information la plus importante ?Je ne crois pas non plus
Pourquoi, ben pour savoir quelles méthode utiliser pour appeler le endpoint ... je ne comprends pourquoi vous tenez vous passer de cette information.
Mis à jour par Frédéric Péters il y a plus de 4 ans
vous passer de cette information.
Mon commentaire concerne la mise en valeur excessive de l'information, façon "super je peux faire un GET, et en regardant tout à droite et en plissant les yeux je pourrai peut-être savoir ce que ce GET ferait".
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Je soutiens ce ticket juste pour le coté uniformisation de la présentation qui donne une information explicite et non équivoque et les couleurs c'est sympa.
Mis à jour par Serghei Mihai il y a plus de 4 ans
J'avais par erreur poussé (b8e84681fe277eb43bd972a4bcd480e0339d92a4) et reverté.
Donc je peux reprendre en intégrant la remarque: placer la description de l'endpoint à gauche et la méthode HTTP à droite.
Puis aussi prendre en compte les long_description, etc.
Mis à jour par Serghei Mihai il y a plus de 4 ans
- Fichier passerelle.png passerelle.png ajouté
- Fichier 0001-misc-beautify-endpoints-pr-sentation-34790.patch 0001-misc-beautify-endpoints-pr-sentation-34790.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Nouvelle version intégrant les remarques.
Il faudrait mieux formatter les paramètres des requetes, mais déjà voir si la distinction des endpoints convient.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
J'ai poussé #38276 si tu veux rebase et refaire un screenshot.
Mis à jour par Serghei Mihai il y a plus de 4 ans
- Fichier passerelle-1.png passerelle-1.png ajouté
Voilà.
ThomasJ va donner son avis aussi sur l'affaire.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
Les types sont stylés en gris et en italique normalement, tu n'as pas dû recharger ton css
Mis à jour par Serghei Mihai il y a plus de 4 ans
- Fichier passerelle-2.png passerelle-2.png ajouté
Effectivement, zappé de lancer collectstatic
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans
En vrac.
Quelques améliorations graphiques possibles
- Aligner les marges blanches sur celles du tableau "Sécurité" affiché en dessous.
- Ne pas mettre l'information "Query parameters" en bold qui est une information redondante (et déjà mis en exergue par le fond blanc) mais plutôt la description de l'endpoint
- Réduire l'espace de tabulation des paramètres
- Aligner description & long-description sur le bloc blanc qui liste les paramètres
- Ajouter un padding d'un espace mot au bloc blanc des paramètres
- augmenter légèrement le margin entre les blocs endpoints
- Caler le bloc .method-name "vraiment" en haut à gauche : sur la bordure du bloc
- Uniformiser l'utilisation des doubles points pour les paramètres
- Peut-être réduire l'impact visuel des blocs blanc parameters en ajoutant une opacité réduite.
Code CSS
- Préférer em à px pour la gestion des espaces, basé sur le blanc typographique d'un espace mot (.33em ou .66em pour le double)
- Retirer les tags des selecteurs
- Préférer HSL pour les couleurs à la place de HEX permettrait de gagner en maintenance et mettre à jour plus simplement les couleurs
- Vérifier si tout s'affiche bien en mobile : superposition methode / description; checker les line-height si affichage des textes en plusieurs lignes
:)
Mis à jour par Serghei Mihai il y a plus de 4 ans
- Fichier 0001-misc-beautify-endpoints-pr-sentation-34790.patch 0001-misc-beautify-endpoints-pr-sentation-34790.patch ajouté
- Fichier passerelle-2020.png passerelle-2020.png ajouté
- Fichier passerelle-2020-mobile.png passerelle-2020-mobile.png ajouté
Ok.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Je verrais bien le nom de la méthode mis en exergue et pas au même niveau que l'URL, via le même principe que la méthode HTTP, avec un encadré collé au coin gauche de la box (Récupérer / Liste / Remplacer), quand on arrive sur la page c'est l'information la plus importante.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Benjamin Dauvergne a écrit :
Je verrais bien le nom de la méthode mis en exergue et pas au même niveau que l'URL, via le même principe que la méthode HTTP, avec un encadré collé au coin gauche de la box (Récupérer / Liste / Remplacer), quand on arrive sur la page c'est l'information la plus importante.
Remarque de mon camarade Manu, pour une première version c'est très bien, oublie; Thomas J. repassera dessus et fera ça à sa sauce.
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Alors gogo.
Mis à jour par Serghei Mihai il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
commit 010fabec811fafa7d3a93d3c7a965071a7967cdd (origin/master, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Jul 11 17:08:51 2019 +0200 misc: beautify endpoints présentation (#34790)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: beautify endpoints présentation (#34790)