Project

General

Profile

Développement #60056

Essayer d'afficher les espaces insécables dans l'inspect

Added by Valentin Deniaud almost 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
23 December 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Dans #60002#note-7, on remarque qu'on fait l'effort de traduire les messages d'erreur renvoyés par chrono mais dans l'inspect ça donne des «\xa0...\xa0» et c'est dommage.

Mais peut-être qu'en dehors de ce contexte « message d'erreur en français » on a envie de distinguer les espaces insécables, auquel cas ce ticket peut s'avérer complexe.


Files

Associated revisions

Revision 1530f586 (diff)
Added by Frédéric Péters almost 3 years ago

backoffice: avoid escaping of non-breaking spaces in inspect page (#60056)

History

#1

Updated by Frédéric Péters almost 3 years ago

Pour reprendre de #60002#note-7,

 ['Il est nécessaire de passer un paramètre «\xa0agendas\xa0» ou «\xa0subscribed\xa0».'] (list)

i.e. ça concerne l'affichage d'éléments de type liste (ce qui amène python à un repr() de chacun de ses éléments) et le repr() d'une chaine et bien il échappe l'espace insécable et autres.

Sans tout chambouler et après avoir testé différentes choses, je serais pour juste viser le cas précis ici, quelque chose comme :

                 r += htmltext('<li>%s') % (k, k)
- r = htmltext(' <div class="value"><span>%s</span>') % ellipsize(safe(v), 10000)
if isinstance(v, list):
+ # custom behaviour for lists
+ r = htmltext('<div class="value"><span>[')
def custom_repr(var):
+ if isinstance(var, str):
+ var = var.replace('\xa0', ' ') # avoid escaped non breaking space
+ return repr(var)
+ r = ', '.join(custom_repr(x) for x in v)
r = htmltext(']</span>')
else:
+ r += htmltext(' <div class="value"><span>%s</span>') % ellipsize(safe(v), 10000)
#2

Updated by Frédéric Péters almost 3 years ago

#3

Updated by Thomas Noël (congés → 5 décembre) almost 3 years ago

  • Status changed from Solution proposée to Solution validée
#4

Updated by Frédéric Péters almost 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 1530f58646c5f0d3e402e585e47b5b7cd78e4ed4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Dec 24 12:32:22 2021 +0100

    backoffice: avoid escaping of non-breaking spaces in inspect page (#60056)
#5

Updated by Frédéric Péters almost 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#6

Updated by Transition automatique almost 3 years ago

Automatic expiration

Also available in: Atom PDF