Skip to content

Add a paragraph below the item category selection to show its purpose#685

Merged
AerunDev merged 5 commits intodevelopfrom
59-text-helper-for-item-category-purpose-at-creation
Nov 7, 2025
Merged

Add a paragraph below the item category selection to show its purpose#685
AerunDev merged 5 commits intodevelopfrom
59-text-helper-for-item-category-purpose-at-creation

Conversation

@GaburaisuVGC
Copy link
Collaborator

…se (#59)

Thank you for your contribution to the Pokémon Studio repo.

Before submitting this PR into the develop branch, please make sure:

  • Your code builds clean without any errors or warnings
  • You are following the Code guidelines
  • You tested your code to make sure it does what it is supposed to do

Description

Added a text helper below the item category selection (when adding or editing any item) to help the user know what is the purpose of the category. Made english and french text variants for this text helper (that can be improved).

Tests to perform

  • Load a project
  • Go to the Database section > Item
  • Add or Edit an item and check for the text below the item category selection.

closes #59

@GaburaisuVGC GaburaisuVGC self-assigned this Nov 5, 2025
@GaburaisuVGC GaburaisuVGC requested a review from NuriYuri November 5, 2025 16:23
Copy link
Collaborator

@Aelysya Aelysya left a comment

Choose a reason for hiding this comment

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

À part la traduction où je t'ai laissé un commentaire, ç'à l'air de bien marcher, bien joué ^^

@AerunDev AerunDev changed the title Added a paragraph below the item category selection to show its purpo… Add a paragraph below the item category selection to show its purpose Nov 6, 2025
@AerunDev
Copy link
Collaborator

AerunDev commented Nov 6, 2025

Très très cool comme ajout, bien joué ! 😄

Avec Palb et Aelysya, on a juste fait une petite amélioration pour passer le style du helper qui s'affichait dans l'éditeur du bloc "Informations" des Pokémon pour avoir une cohérence de style.
Comme c'est un helper, il vaut mieux qu'il soit affiché dans le contexte du Select, et donc dans son composant et pas qu'il ne donne l'impression d'être "à part", comme le sont certains textes en haut des éditeurs, qui là sont plus des textes avec une visée globale.
Je n'avais pas préparé de design pour ça, donc c'est my bad, clairement !
J'en ai profité pour corriger le texte anglais dans le JSON en.json.

@AerunDev
Copy link
Collaborator

AerunDev commented Nov 6, 2025

Je te laisse nous dire si ça te convient @GaburaisuVGC et on pourra merge 😄

@GaburaisuVGC
Copy link
Collaborator Author

C'est vrai que j'avais pas de design donc j'ai un peu improvisé, mais merci pour les corrections, c'est encore mieux !

Vous pouvez merge

@AerunDev AerunDev merged commit 34491d0 into develop Nov 7, 2025
5 checks passed
@AerunDev AerunDev deleted the 59-text-helper-for-item-category-purpose-at-creation branch November 7, 2025 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a text helper to know what is the purpose of the Item category at the Item creation

5 participants