Skip to content

Commit

Permalink
Translate javascript component
Browse files Browse the repository at this point in the history
  • Loading branch information
Ibtihal Aabdi committed Dec 21, 2022
1 parent 8550f1f commit 6ae54dc
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion js/config.ts
Expand Up @@ -23,7 +23,7 @@ export const defaultConfig: Readonly<Configuration> = {
showCloseButton: true,
minimumQueryLength: 3,
forceOverwrite: false,
resultNoun: { singular: "file", plural: "files" },
resultNoun: { singular: "article", plural: "articles" },
onQueryUpdate: undefined,
onResultSelected: undefined,
onResultsHidden: undefined,
Expand Down
12 changes: 6 additions & 6 deletions js/entity.ts
Expand Up @@ -44,18 +44,18 @@ export class Entity {
if (!this.domManager) return null;
const query = this.domManager.getQuery();
if (this.state === "error") {
return "Error! Check the browser console.";
return "Erreur! Vérifier dans la console...";
} else if (this.state != "ready") {
return "Loading...";
return "Chargement...";
} else if (query?.length < this.config.minimumQueryLength) {
return "Filtering...";
return "Recherche en cours...";
} else if (this.results) {
if (this.totalResultCount === 0) {
return `No ${this.config.resultNoun.plural} found.`;
return `Pas d'${this.config.resultNoun.plural} trouvés.`;

This comment has been minimized.

Copy link
@glepretre

glepretre Jan 9, 2023

@ibtihalaa @gentooboontoo NB : En français comme en anglais, ce cas devrait toujours être au singulier.

=> No file found
=> Pas d'article trouvé (aucun)

Pourrait faire l'objet d'une issue sur le repo stork.

On pourrait également en faire une pour demander un système de traduction de ces messages, pour pouvoir un jour éviter de rebuilder ?

This comment has been minimized.

Copy link
@gentooboontoo

gentooboontoo Jan 11, 2023

Member

@ibtihalaa @gentooboontoo NB : En français comme en anglais, ce cas devrait toujours être au singulier.

Il n'y a pas de règle à ce niveau (source : https://www.lalanguefrancaise.com/orthographe/pas-de-soucis-ou-pas-de-souci). J'aurais tendance à écrire au pluriel car il y a plus de probabilités d'en avoir plusieurs si il y en a (exemple : «je n'ai pas de cheveux» plutôt que «je n'ai pas de cheveu»).

Pourrait faire l'objet d'une issue sur le repo stork.

En anglais, je n'ai pas vérifié la règle. Je te laisse voir @glepretre si c'est pertinent mais ça casserait en français du coup (c'est la clé plural qui est utilisée, selon moi à bon escient).

On pourrait également en faire une pour demander un système de traduction de ces messages, pour pouvoir un jour éviter de rebuilder ?

Why not si ce n'est qu'une issue à faire (l'enjeu pour nous n'est pas suffisamment important pour proposer une implémentation à ce stade).

This comment has been minimized.

Copy link
@glepretre

glepretre Jan 16, 2023

Au temps pour moi pour l'absence de règle à "pas de" ✋

J'ai ouvert une issue sur Stork, si vous voulez bien mettre des pouces bleus 👍 ➡️ jameslittle230#337

@ibtihalaa @gentooboontoo

} else if (this.totalResultCount === 1) {
return `1 ${this.config.resultNoun.singular} found.`;
return `1 ${this.config.resultNoun.singular} trouvé.`;
} else {
return `${this.totalResultCount} ${this.config.resultNoun.plural} found.`;
return `${this.totalResultCount} ${this.config.resultNoun.plural} trouvés.`;
}
}

Expand Down

0 comments on commit 6ae54dc

Please sign in to comment.