Skip to content

Add a database subpage to quickly manage techitem moves#494

Merged
AerunDev merged 18 commits intodevelopfrom
408-add-a-database-subpage-to-quickly-manage-techitem-moves
May 8, 2025
Merged

Add a database subpage to quickly manage techitem moves#494
AerunDev merged 18 commits intodevelopfrom
408-add-a-database-subpage-to-quickly-manage-techitem-moves

Conversation

@Aelysya
Copy link
Collaborator

@Aelysya Aelysya commented Apr 21, 2025

Description

This PR adds a new database page accessible form the Items page. It lists all the Technique items and allows modifying the move it teaches.

closes #408

Note before testing

Go to the "Items" database page

Tests to perform

On the items main page

  • A new button "Manage Technique items" is shown in the control bar
  • Clicking on the button show the Tech Items list page

On the Tech items list page

  • Clicking on the arrow button at the top of the page returns to the Items main page
  • Only Tech items are listed
  • The items are sorted based on their name in this order: HMs first, then TMs, then TRs and finally others
  • Clicking on the "Edit button" on the right of the line shows the page of the related item
  • Modifying the move linked to an item correctly modify the type/category/etc... data
  • Modifying the move linked to an item correctly changes the move shown on this items main page and its description
  • Creating a new Tech item adds it to the list with "Move deleted" in the select

@Aelysya Aelysya self-assigned this Apr 21, 2025
@Aelysya Aelysya linked an issue Apr 21, 2025 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@Palbolsky Palbolsky left a comment

Choose a reason for hiding this comment

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

J'ai fait des remarques sur Discord concernant le problème d'optimisation : https://discord.com/channels/143824995867557888/1363966739658047550

@Palbolsky
Copy link
Collaborator

GG pour l'amélioration des performances 🥳

Une toute dernière chose. Il faut faire un choix entre mettre une majuscule à techniques ou non (pour la cohérence) :p
image
image

@Palbolsky
Copy link
Collaborator

Palbolsky commented Apr 23, 2025

Je viens de remarquer que quand on crée un objet depuis la page de la liste des techniques, l'objet est bien créé mais on ne se retrouve pas automatiquement sur la page de l'objet (on reste sur la page des techniques)

Pour moi il faudrait aller automatiquement sur la page de l'objet créé, mais à voir avec @AerunDev

@invatorzen
Copy link

On the items main page

  • [* ] A new button "Manage Technique items" is shown in the control bar
  • [ *] Clicking on the button show the Tech Items list page

On the Tech items list page

  • [ *] Clicking on the arrow button at the top of the page returns to the Items main page
  • [ *] Only Tech items are listed
  • [ *] The items are sorted based on their name in this order: HMs first, then TMs, then TRs and finally others
  • [ *] Clicking on the "Edit button" on the right of the line shows the page of the related item
  • [ *] Modifying the move linked to an item correctly modify the type/category/etc... data
  • [ *] Modifying the move linked to an item correctly changes the move shown on this items main page and its description
  • [* ] Creating a new Tech item adds it to the list with "Move deleted" in the select

Creating a new item did add it to the list, but since mine was at the bottom of the list it was not able to select any moves.

Two possible suggestions:
I also think we could add a delete button next to the edit button, to quickly delete TMs.
Maybe the dropdown can hyperlink or the HM/TM/TR row be clickable (similar to pokemon in the pokedex tab) to quickly jump to that move.

@AerunDev
Copy link
Collaborator

Creating a new item did add it to the list, but since mine was at the bottom of the list it was not able to select any moves.

Two possible suggestions: I also think we could add a delete button next to the edit button, to quickly delete TMs. Maybe the dropdown can hyperlink or the HM/TM/TR row be clickable (similar to pokemon in the pokedex tab) to quickly jump to that move.

We can't add an hyperlink into a dropdown. That's not accessible at all.
For the delete button, we'll see with a new issue later. 👍

@invatorzen
Copy link

invatorzen commented Apr 30, 2025

We can't add an hyperlink into a dropdown. That's not accessible at all.

Fair, but I believe some type of short cut to edit the move and not just the item would be nice, whether it be quick modifications or a shortcut to that move~

Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

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

Je viens de tester cette PR et pour moi le fonctionnel est super, bien joué ! 😄

Problèmes rencontrés

  • Si on a des icônes qui sont plus grandes que 32px (dans mon cas 64px), l'icône présente un défaut d'affichage. Ce genre de défaut n'est pas présent sur le Pokédex si ça peut aider à la résolution, de mémoire il est possible de forcer la taille via le CSS.

Capture d'écran 2025-05-06 194419

  • Lorsqu'on se trouve au bas de la liste des CTs, on a la liste déroulante du Select des capacités qui est coupé, sûrement à cause d'une problématique de z index.

Capture d'écran 2025-05-06 195706

Notes pour plus tard

  • À l'utilisation, je me suis dis que devoir aller sur la page de l'objet d'une CT pour le supprimer était effectivement pénible. On pourra rajouter ça plus tard à la demande.
  • Également, je me demande si avoir un bouton de création avec une modale adaptée ne serait pas mieux à termes. En gros la création d'un TechItem directement.
  • Pour la partie "redirection vers la page de l'item créé", on va voir ce que les utilisateurs nous font comme retour. D'un côté je me dis que c'est top de revenir sur la page de l'item, mais si tu crées ta liste de CT en bulk c'est mieux de rester sur la liste.

@AerunDev
Copy link
Collaborator

AerunDev commented May 6, 2025

Autre sujet peut-être à discuter ensemble.
Lorsqu'on a des CS qui ont un ordre d'ID dans le désordre (par exemple CS 1, 2, 5, 3, 4), elles ne sont pas affichées dans l'ordre dans la liste. Est-ce qu'on ne trierait pas par ordre alphabétique plutôt ?

@AerunDev AerunDev merged commit 6c86681 into develop May 8, 2025
5 checks passed
@AerunDev AerunDev deleted the 408-add-a-database-subpage-to-quickly-manage-techitem-moves branch May 8, 2025 21:32
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 database subpage to quickly manage TechItem moves

4 participants