Projet

Général

Profil

Bug #39031

rendu des endpoints qui n'ont pas de description

Ajouté par Frédéric Péters il y a plus de 4 ans. Mis à jour il y a environ 4 ans.

Statut:
En cours
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
16 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Le chemin est collé au bord du bloc.


Fichiers

Screenshot_2020-01-16 Passerelle.png (7,87 ko) Screenshot_2020-01-16 Passerelle.png Frédéric Péters, 16 janvier 2020 17:10
Screenshot_2020-01-16 Passerelle(3).png (7,24 ko) Screenshot_2020-01-16 Passerelle(3).png Frédéric Péters, 16 janvier 2020 17:16
0001-misc-fix-endpoints-margins-39031.patch (1,37 ko) 0001-misc-fix-endpoints-margins-39031.patch Serghei Mihai, 28 janvier 2020 16:00
0001-patch-thomas-endpoints.patch (1,67 ko) 0001-patch-thomas-endpoints.patch Thomas Jund (congés, retour le 29/04), 30 janvier 2020 16:26

Demandes liées

Lié à Passerelle - Development #41846: phonecalls, description des points d'accèsFermé17 avril 2020

Actions

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 4 ans

(ce côté "collé au bord" a lieu aussi pour les chemins qui wrappent)

#2

Mis à jour par Serghei Mihai il y a environ 4 ans

#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

Sur l'url, si le margin-right sert à éviter une possible collusion avec `.method-name`, un `margin-right de 5.5em` est necessaire (4em + 0.75*2).
je ne sui s pas sur que le padding serve à quelque chose

@@ -212,28 +212,26 @@ ul li .type {
 @media screen and (max-width: 760px) {
     .example-url {
-        padding-right: 0;
-        margin: 0 5em 0 0;
         display: inline-block;
     }
 }

 .example-url {
     border: 0;
-    margin-right: 4.3em;
+    margin-right: 5.5em;
 }

Tu pourrais virer les bullets list sur toutes les listes enfants directement sur toutes les ul

-#endpoints > div > ul {
+#endpoints ul {
     padding: 0;
+    list-style-type: none;
 }

Positionner .method-name direct avec top et bottom plutot que d'ajouter un margin negatif et je ne sais pas à quoi sert le cursor: pointer ?

@@ -289,11 +287,9 @@ ul.get-params {
     width: 4em;
     display: inline-block;
     text-align: center;
-    margin: -1px -1px 0 0;
-    cursor: pointer;
     position: absolute;
-    right: 0;
-    top: 0;
+    right: -1px;
+    top: -1px;
 }

Et je propose aussi une amélioration des margins et paddings (voir patch joint).


Sinon, je pense que tu gagnerais en lisibilité, résolution de conflits en appliquant la convention de nommage CSS.
Par exemple sur

#endpoints > div > ul  li {
        list-style-type: none;
    margin: 1rem 0;
    position: relative;
    padding: 0.3em 0.75em;
}

Tu appliques ces styles sur toutes le balises `li` avec le poids d'un id. Mais tu es dans une structure avec beaucoup de listes imbriquées qui ont besoins de variantes graphiques.
Pour amender les styles d'un item de liste enfant, tu vas devoir surenchérir avec

#endpoints > div > ul  li li {}
//et puis
#endpoints > div > ul  li li li {}

PAr exemple, tes elements pourraient s'identifier :

#endpoints                          =>  .endpoints
#endpoints > div > ul               =>  .endpoints--list

#endpoints > div > ul > li          =>  .endpoint
#endpoints .description             =>  .endpoint--descrition 
#endpoints .long-description        =>  .endpoint--long-description
#endpoints .method-name             =>  .endpoint--method-name
#endpoints .get-method .method-name =>  .endpoint--get-method

#endpoints .params                  =>  .endpoint-params
#endpoints > div > ul li            =>  .endpoint-param
#endpoints .params .params-title    =>  .endpoint-param--title
#endpoints .params .param-name      =>  .endpoint-param--name
#endpoints .params .param-optional  =>  .endpoint-param--optional

Et si tu as besoin d'ajouter les éléments au bloc, une nouvelle class permettra de spécifier des styles sans surcharge

.entpoint-params--body-schemas
#4

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution proposée à En cours
  • Patch proposed changé de Oui à Non
#5

Mis à jour par Frédéric Péters il y a environ 4 ans

Formats disponibles : Atom PDF