Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Unités] Fixe et améliore plusieurs points #1133

Merged
merged 15 commits into from
Jan 25, 2024
Merged

[Unités] Fixe et améliore plusieurs points #1133

merged 15 commits into from
Jan 25, 2024

Conversation

ivangabriele
Copy link
Contributor

@ivangabriele ivangabriele commented Jan 18, 2024 •
edited
Loading

@ivangabriele ivangabriele force-pushed the ivan/fix-multiple-control-units-glitches branch 2 times, most recently from 5b6f535 to e7e0e35 Compare January 19, 2024 10:18
@ivangabriele ivangabriele marked this pull request as ready for review January 22, 2024 10:39
@ivangabriele ivangabriele force-pushed the ivan/fix-multiple-control-units-glitches branch 3 times, most recently from 0b2db0c to 58f31ae Compare January 22, 2024 12:30
@ivangabriele ivangabriele added bug Something isn't working feat. enhancement Amélioration/évolution d'une fonctionnalité labels Jan 22, 2024
@ivangabriele ivangabriele force-pushed the ivan/fix-multiple-control-units-glitches branch from fccde92 to 83499d9 Compare January 23, 2024 08:13
claire2212
claire2212 reviewed Jan 23, 2024
View reviewed changes
frontend/src/features/SideWindow/SideWindowLauncher.tsx
@@ -42,7 +43,9 @@ export function SideWindowLauncher() {
const onUnload = async () => {
dispatch(sideWindowActions.close())
dispatch(reportingActions.resetReportings())
// TODO Why?
Copy link
Collaborator

Choose a reason for hiding this comment

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

alors je me rends compte qu'il y a un souci, quand on ferme la sidewindow ça ferme aussi les signalements ouverts sur la map. Je vais fixer ça.

Sinon ici c'était pour cleaner l'état et qu'on est pas le fond noir comme s'il y avait un formulaire ouvert (sur la side-window)

@claire2212
Copy link
Collaborator

claire2212 commented Jan 23, 2024 •
edited
Loading

@ivangabriele :

  • j'ai l'impression que la zone permettant de décaler le formulaire de signalement (sur la carte) pour faire apparaître les boutons de droite est plus large qu'avant. Là même dans le formulaire (quand la souris passe sur la droite du formulaire) ça la décale. Je trouve ça un peu pénible quand tu saisis des données dans le formulaire, on peut pas réduire cette zone?

  • Quand tu ouvres un signalement sur la carte, et que tu cliques sur le boutons des bases, le signalement devrait se réduire ce n'est pas le cas

  • Quand tu ouvres un signalement sur la carte -> tu ouvres un autre signalement depuis la side_window -> l'affichage côté carte n'est pas bon

@ivangabriele ivangabriele force-pushed the ivan/fix-multiple-control-units-glitches branch from de9248d to 81afddc Compare January 24, 2024 09:14
@ivangabriele
Copy link
Contributor Author

ivangabriele commented Jan 24, 2024 •
edited
Loading

@claire2212 Du coup je pense qu'il reste seulement la question des signalements sur la carte lorsque la side window est fermée.

@claire2212 claire2212 self-requested a review January 25, 2024 14:56
@ivangabriele ivangabriele merged commit 779ffb4 into main Jan 25, 2024
17 checks passed
@ivangabriele ivangabriele deleted the ivan/fix-multiple-control-units-glitches branch January 25, 2024 15:05
@claire2212 claire2212 mentioned this pull request Jan 29, 2024
Merged
1 task
claire2212 added a commit that referenced this pull request Feb 2, 2024
@claire2212
[Signalement] Ajout d'un bouton radio "Pas de réponse VHF" (#1158)
214ea78 
## Related Pull Requests & Issues

- Resolve #1108
- Résolution du bug remonté ici :
#1133 (comment)

----

- [x] Tests E2E (Cypress)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@claire2212 claire2212 claire2212 approved these changes

Assignees
No one assigned
Labels
bug Something isn't working feat. enhancement Amélioration/évolution d'une fonctionnalité
Projects
None yet
Milestone
No milestone
2 participants
@ivangabriele @claire2212