Project

General

Profile

Bug #21546

Limiter la largeur du bouton de connexion FC

Added by Frédéric Péters almost 2 years ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
31 Jan 2018
Due date:
% Done:

0%

Hors marché:
No
Patch proposed:
No
Planning:
No

Description

Dans la documentation d'utilisation, https://partenaires.franceconnect.gouv.fr/fournisseur-service :

Pour les boutons en svg, lors de l'utilisation d'une image veuillez preciser la taille du bouton.

On ne fait pas ça et selon les intégrations graphiques on peut se trouver avec des boutons vraiment trop larges (cf capture).

Ils ne fournissent pas d'instruction sur une taille maximale mais leurs PNG sont à 224px.

Screenshot-2018-1-31 Gestion du compte.png View (29.1 KB) Frédéric Péters, 31 Jan 2018 09:19 AM

0001-css-limit-button-width-to-240px-fixes-21546.patch View (889 Bytes) Benjamin Dauvergne, 11 Feb 2018 10:25 PM

22094

History

#1 Updated by Benjamin Dauvergne almost 2 years ago

Au niveau CSS est-ce qu'on peut contraindre la taille d'un SVG ?

#2 Updated by Frédéric Péters almost 2 years ago

Oui. (width: whatever, comme pour le reste).

#4 Updated by Frédéric Péters almost 2 years ago

max-width plutôt, mais ça fait un truc moche parce que l'image est inline, il faudrait la mettre en bloc, et alors ça fait moche quand même et il faudrait essayer en centrant le tout, etc.

Aussi, mais ce n'est pas un problème lié à ce module, à tester pour que ça fonctionne il faudrait que fc.css se trouve chargé sur la page accounts et ça n'est pas le cas.

#5 Updated by Benjamin Dauvergne almost 2 years ago

C'est peut-être bêtement l'absence de fc.css qui explique le problème puisqu'il contient déjà un "height: 50px" qui semble faire l'affaire en page de login. Normalement la directive addtoblock devrait ajouter la CSS, à voir pourquoi ça ne marche pas ici (certainement que la mécanique pour cela n'existe que dans la vue login() d'authentic).

#6 Updated by Benjamin Dauvergne almost 2 years ago

  • Target version set to 0.11

#7 Updated by Benjamin Dauvergne over 1 year ago

  • Target version changed from 0.11 to future

#8 Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Nouveau to Fermé

Le problème a été corrigé par ce commit:

commit 7bffc5011c0b1ea2b6dee4cbb4a141794ac440af
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jul 18 21:39:07 2018 +0200

    misc: don't use addtoblock to insert fc.css (#25356)

Also available in: Atom PDF