Skip to content

Fix some modal centering due to responsive issues#471

Merged
AerunDev merged 4 commits intodevelopfrom
fix-modal-not-centered
May 7, 2025
Merged

Fix some modal centering due to responsive issues#471
AerunDev merged 4 commits intodevelopfrom
fix-modal-not-centered

Conversation

@AntoinePoree
Copy link
Collaborator

#470

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

Fix a bug from positioning a modal

Tests to perform

Test on bigger and smaller modal, and some different screen.

@AntoinePoree AntoinePoree self-assigned this Mar 25, 2025
@Palbolsky
Copy link
Collaborator

Palbolsky commented Mar 26, 2025

C'est mieux (car au moins maintenant la dialog est utilisable) mais ce n'est toujours pas centré :/

image

Et ce n'est plus correctement centré pour les dialogues de suppression :

image

Après avoir utilisé une dialog centrée, si on veut utiliser un éditeur latéral, ça bug complétement.

Pokemon.Studio.2025-03-26.13-15-01.mp4

@AerunDev
Copy link
Collaborator

Hello, effectivement je confirme les retours de Palb. Je pense que ça vient des valeurs en pourcentage forcées sur le CSS.

@AntoinePoree
Copy link
Collaborator Author

Merci pour vos retours, ce n'était pas totalement terminé, j'aurais du la mettre ne draft.
Excusez-moi, et d'un autre coté, j'ai pushé les corrections voulu et demandé !

@Palbolsky
Copy link
Collaborator

Palbolsky commented Apr 6, 2025

C'est beaucoup mieux. :)

Mais j'ai encore trouvé un problème, si je réduis la fenêtre, le dialog n'est plus centré horizontalement :

image

@AntoinePoree AntoinePoree force-pushed the fix-modal-not-centered branch from fdf9ace to 9df6a54 Compare April 9, 2025 18:45
@AntoinePoree
Copy link
Collaborator Author

Merci Palb ><

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.

Hello ! J'ai pu tester ta branche suite aux derniers retours et pour moi ça fonctionne nickel.
Sur la partie code par contre, je me demande si on ne devrait pas faire une passe supplémentaire pour s'assurer de ne pas avoir du CSS directement dans la logique du code ou des valeurs un peu "magic number" pour le responsive. Je laisse les pros me corriger si je dis une bêtise ! 😄

@AerunDev AerunDev changed the title fix(modal): modal centering was'nt working well on small screen Fix some modal centering due to responsive issues May 5, 2025
@AerunDev AerunDev merged commit 47a3a30 into develop May 7, 2025
4 checks passed
@AerunDev AerunDev deleted the fix-modal-not-centered branch May 7, 2025 14:52
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.

3 participants