Skip to content

Update fr.toml: ui_search#929

Merged
chalin merged 1 commit intogoogle:mainfrom
arnaudlevy:patch-3
Jun 29, 2022
Merged

Update fr.toml: ui_search#929
chalin merged 1 commit intogoogle:mainfrom
arnaudlevy:patch-3

Conversation

@arnaudlevy
Copy link
Contributor

The text is too long (or the field is too small)!
Capture d’écran 2022-03-12 à 09 16 04

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D'abord je propose "Chercher dans ce site..." (nous gagnons deux lettres), et apres, nous pourrions ajouter un title au champ de recherche, pour qu'un "hover" montre tout le text, comme ceci:

image

Qu'en pensez-vous?

@chalin chalin self-assigned this Mar 14, 2022
@arnaudlevy
Copy link
Contributor Author

Merci !
Je pense que "Rechercher" est mieux, il ne faut pas que ça sorte du champ du tout, et l'ambiguïté est faible.
On peut aussi élargir le champ.

The text is too long (or the field is too small)!
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est bon, allons-y avec ta proposition! Merci pour la contribution et ta patience!

@chalin chalin changed the title Update fr.toml Jun 29, 2022
@chalin chalin merged commit 4cdc646 into google:main Jun 29, 2022
fekete-robert pushed a commit to fekete-robert/docsy that referenced this pull request Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants