Projet

Général

Profil

Development #34790

améliorer la présentation des endpoints

Ajouté par Serghei Mihai il y a presque 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Version cible:
-
Début:
12 juillet 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

passerelle-endpoints.png (268 ko) passerelle-endpoints.png Serghei Mihai, 12 juillet 2019 14:09
passerelle.png (233 ko) passerelle.png Serghei Mihai, 16 décembre 2019 18:22
0001-misc-beautify-endpoints-pr-sentation-34790.patch (4,91 ko) 0001-misc-beautify-endpoints-pr-sentation-34790.patch Serghei Mihai, 16 décembre 2019 18:22
passerelle-1.png (233 ko) passerelle-1.png Serghei Mihai, 17 décembre 2019 10:20
passerelle-2.png (235 ko) passerelle-2.png Serghei Mihai, 17 décembre 2019 10:35
0001-misc-beautify-endpoints-pr-sentation-34790.patch (5,21 ko) 0001-misc-beautify-endpoints-pr-sentation-34790.patch Serghei Mihai, 06 janvier 2020 11:04
passerelle-2020-mobile.png (162 ko) passerelle-2020-mobile.png Serghei Mihai, 06 janvier 2020 11:05
passerelle-2020.png (217 ko) passerelle-2020.png Serghei Mihai, 06 janvier 2020 11:05

Demandes liées

Lié à Passerelle - Autre #10854: expérimenter un connecteur basé sur djangorestframeworkNouveau06 mai 2016

Actions

Révisions associées

Révision 010fabec (diff)
Ajouté par Serghei Mihai il y a plus de 4 ans

misc: beautify endpoints présentation (#34790)

Historique

#1

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)

#2

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.

#3

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.

#4

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é
#5

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.

#6

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".

#7

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.

#8

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.

#9

Mis à jour par Serghei Mihai il y a plus de 4 ans

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.

#10

Mis à jour par Valentin Deniaud il y a plus de 4 ans

J'ai poussé #38276 si tu veux rebase et refaire un screenshot.

#11

Mis à jour par Serghei Mihai il y a plus de 4 ans

Voilà.
ThomasJ va donner son avis aussi sur l'affaire.

#12

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

#13

Mis à jour par Serghei Mihai il y a plus de 4 ans

Effectivement, zappé de lancer collectstatic

#14

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

  1. Aligner les marges blanches sur celles du tableau "Sécurité" affiché en dessous.
  2. 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
  3. Réduire l'espace de tabulation des paramètres
  4. Aligner description & long-description sur le bloc blanc qui liste les paramètres
  5. Ajouter un padding d'un espace mot au bloc blanc des paramètres
  6. augmenter légèrement le margin entre les blocs endpoints
  7. Caler le bloc .method-name "vraiment" en haut à gauche : sur la bordure du bloc
  8. Uniformiser l'utilisation des doubles points pour les paramètres
  9. Peut-être réduire l'impact visuel des blocs blanc parameters en ajoutant une opacité réduite.

Code CSS

  1. 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)
  2. Retirer les tags des selecteurs
  3. Préférer HSL pour les couleurs à la place de HEX permettrait de gagner en maintenance et mettre à jour plus simplement les couleurs
  4. Vérifier si tout s'affiche bien en mobile : superposition methode / description; checker les line-height si affichage des textes en plusieurs lignes

:)

#16

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.

#17

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.

#18

Mis à jour par Emmanuel Cazenave il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée

Alors gogo.

#19

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)
#20

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

Formats disponibles : Atom PDF